Fix regressions with use of arguments in fat arrow functions.
Review Request #8190 — Created May 26, 2016 and submitted
Fat arrow functions don't get the
argumentsobject like normal functions do.
There were a few places in the recent ES6 conversion of the base resource model
that didn't account for this, and caused a regression. This change fixes those
up and adds some unit testing to make sure we're doing the right thing.
Ran js-tests.
-
Looks good overall. Might be good to have tests for the other places where the callbacks are called, but I'd rather get this in now.
- Change Summary:
-
Use rest parameters instead of listing them all out.
- Commit:
-
014301789e0ffda77d22c9e7861f15f29b5469a704c10034f81ef16938c1a5bcf92732b6d7b2fe77
-
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js reviewboard/static/rb/js/resources/models/draftResourceModelMixin.es6.js reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js reviewboard/static/rb/js/resources/models/draftResourceModelMixin.es6.js reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.js
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js reviewboard/static/rb/js/resources/models/draftResourceModelMixin.es6.js reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.js Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js reviewboard/static/rb/js/resources/models/draftResourceModelMixin.es6.js reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.js