diff options
| author | Timothée Jeannin | 2014-03-04 20:39:10 +0100 | 
|---|---|---|
| committer | Tobias Bosch | 2014-03-05 16:30:51 -0800 | 
| commit | 933537860247de35c21db060ae9ae1f87bb30d53 (patch) | |
| tree | 3a5ae308aa246de8e3d48d544b5250f1c80effc8 | |
| parent | de2ecb8a969aa4c96da59e94c0aa9fa84ab232e0 (diff) | |
| download | angular.js-933537860247de35c21db060ae9ae1f87bb30d53.tar.bz2 | |
style: enable jscs requireLeftStickedOperators rule
Closed #6544.
| -rw-r--r-- | .jscs.json | 3 | ||||
| -rw-r--r-- | .jscs.json.todo | 1 | ||||
| -rw-r--r-- | src/jqLite.js | 2 | ||||
| -rw-r--r-- | src/ngSanitize/sanitize.js | 2 | ||||
| -rw-r--r-- | test/auto/injectorSpec.js | 2 | ||||
| -rw-r--r-- | test/ng/sceSpecs.js | 2 | ||||
| -rw-r--r-- | test/ngTouch/directive/ngSwipeSpec.js | 4 | ||||
| -rw-r--r-- | test/ngTouch/swipeSpec.js | 4 | 
8 files changed, 10 insertions, 10 deletions
| @@ -1,5 +1,6 @@  {    "disallowKeywords": ["with"],    "disallowTrailingWhitespace": true, -  "requireRightStickedOperators": ["!"] +  "requireRightStickedOperators": ["!"], +  "requireLeftStickedOperators": [","]  } diff --git a/.jscs.json.todo b/.jscs.json.todo index 96326dce..1fb8c28d 100644 --- a/.jscs.json.todo +++ b/.jscs.json.todo @@ -8,7 +8,6 @@    "requireSpaceAfterKeywords": ["if", "else", "for", "while", "do", "switch", "return", "try", "catch"],    "disallowLeftStickedOperators": ["?", "+", "-", "/", "*", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="],    "disallowRightStickedOperators": ["?", "+", "/", "*", ":", "=", "==", "===", "!=", "!==", ">", ">=", "<", "<="], -  "requireLeftStickedOperators": [","],    "disallowImplicitTypeConversion": ["string"],    "disallowMultipleLineBreaks": true,    "disallowKeywordsOnNewLine": ["else"], diff --git a/src/jqLite.js b/src/jqLite.js index 809ede03..ba613f21 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -457,7 +457,7 @@ forEach({      return jqLite(element).data('$isolateScope') || jqLite(element).data('$isolateScopeNoTemplate');    }, -  controller: jqLiteController , +  controller: jqLiteController,    injector: function(element) {      return jqLiteInheritedData(element, '$injector'); diff --git a/src/ngSanitize/sanitize.js b/src/ngSanitize/sanitize.js index 35dc8f10..38d088bb 100644 --- a/src/ngSanitize/sanitize.js +++ b/src/ngSanitize/sanitize.js @@ -254,7 +254,7 @@ function htmlParser( html, handler ) {          match = html.match( DOCTYPE_REGEXP );          if ( match ) { -          html = html.replace( match[0] , ''); +          html = html.replace( match[0], '');            chars = false;          }        // end tag diff --git a/test/auto/injectorSpec.js b/test/auto/injectorSpec.js index a70cd763..4b967978 100644 --- a/test/auto/injectorSpec.js +++ b/test/auto/injectorSpec.js @@ -163,7 +163,7 @@ describe('injector', function() {        function $f_n0 /*            */(            $a, // x, <-- looks like an arg but it is a comment -          b_ , /* z, <-- looks like an arg but it is a +          b_, /* z, <-- looks like an arg but it is a                   multi-line comment                   function (a, b) {}                   */ diff --git a/test/ng/sceSpecs.js b/test/ng/sceSpecs.js index e2a16c1c..40e79c57 100644 --- a/test/ng/sceSpecs.js +++ b/test/ng/sceSpecs.js @@ -109,7 +109,7 @@ describe('SCE', function() {      }));      it('should NOT wrap unknown contexts', inject(function($sce) { -      expect(function() { $sce.trustAs('unknown1' , '123'); }).toThrowMinErr( +      expect(function() { $sce.trustAs('unknown1', '123'); }).toThrowMinErr(            '$sce', 'icontext', 'Attempted to trust a value in invalid context. Context: unknown1; Value: 123');      })); diff --git a/test/ngTouch/directive/ngSwipeSpec.js b/test/ngTouch/directive/ngSwipeSpec.js index 2aa1a8fe..b46a9384 100644 --- a/test/ngTouch/directive/ngSwipeSpec.js +++ b/test/ngTouch/directive/ngSwipeSpec.js @@ -222,6 +222,6 @@ var swipeTests = function(description, restrictBrowsers, startEvent, moveEvent,    });  } -swipeTests('touch', true  /* restrictBrowers */, 'touchstart', 'touchmove', 'touchend'); -swipeTests('mouse', false /* restrictBrowers */, 'mousedown',  'mousemove', 'mouseup'); +swipeTests('touch', /* restrictBrowers */ true, 'touchstart', 'touchmove', 'touchend'); +swipeTests('mouse', /* restrictBrowers */ false, 'mousedown',  'mousemove', 'mouseup'); diff --git a/test/ngTouch/swipeSpec.js b/test/ngTouch/swipeSpec.js index 435dda8c..1eb53e45 100644 --- a/test/ngTouch/swipeSpec.js +++ b/test/ngTouch/swipeSpec.js @@ -387,6 +387,6 @@ var swipeTests = function(description, restrictBrowsers, startEvent, moveEvent,    });  } -swipeTests('touch', true  /* restrictBrowers */, 'touchstart', 'touchmove', 'touchend'); -swipeTests('mouse', false /* restrictBrowers */, 'mousedown',  'mousemove', 'mouseup'); +swipeTests('touch', /* restrictBrowers */ true, 'touchstart', 'touchmove', 'touchend'); +swipeTests('mouse', /* restrictBrowers */ false, 'mousedown',  'mousemove', 'mouseup'); | 
