style(jscs): fix jscs errors

Author: adamdbradley

Diff for: .jscs.json

@@ -18,7 +18,6 @@
     "beforeOpeningRoundBrace": true
   },
   "disallowSpacesInsideArrayBrackets": true,
-  "disallowSpacesInsideParentheses": true,
   "disallowTrailingComma": true,
   "disallowTrailingWhitespace": true,
   "requireCommaBeforeLineBreak": true,

Diff for: gulpfile.js

@@ -61,7 +61,7 @@ if (argv.dist) {
 
 gulp.task('default', ['build']);
 gulp.task('build', ['bundle', 'sass']);
-gulp.task('validate', ['eslint', 'ddescribe-iit', 'karma']);
+gulp.task('validate', ['jscs', 'eslint', 'ddescribe-iit', 'karma']);
 
 var IS_WATCH = false;
 gulp.task('watch', ['build'], function() {

Diff for: js/angular/controller/listController.js

@@ -59,7 +59,7 @@ IonicModule.service('$ionicListDelegate', ionic.DelegateService([
    * @name $ionicListDelegate#closeOptionButtons
    * @description Closes any option buttons on the list that are swiped open.
    */
-  'closeOptionButtons',
+  'closeOptionButtons'
   /**
    * @ngdoc method
    * @name $ionicListDelegate#$getByHandle
@@ -91,7 +91,7 @@ function($scope, $attrs, $ionicListDelegate, $ionicHistory) {
   $scope.$on('$destroy', deregisterInstance);
 
   self.showReorder = function(show) {
-    if (arguments.length) {
+    if(arguments.length) {
       isReorderShown = !!show;
     }
     return isReorderShown;

Diff for: js/angular/directive/collectionRepeat.js

@@ -323,7 +323,7 @@ function CollectionRepeatDirective($ionicCollectionManager, $parse, $window, $$r
       // Try to just parse the plain attr value
       try {
         parsedValue = $parse(attrValue);
-      } catch(e) {
+      } catch (e) {
         // If the parse fails and the value has `px` or `%` in it, surround the attr in
         // quotes, to attempt to let the user provide a simple `attr="100%"` or `attr="100px"`
         if (attrValue.trim().match(/\d+(px|%)$/)) {
@@ -520,7 +520,7 @@ function RepeatManagerFactory($rootScope, $window, $$rAF) {
       var current = containerNode;
       do {
         repeaterBeforeSize += current[isVertical ? 'offsetTop' : 'offsetLeft'];
-      } while( ionic.DomUtil.contains(scrollView.__content, current = current.offsetParent) );
+      } while ( ionic.DomUtil.contains(scrollView.__content, current = current.offsetParent) );
 
       var containerPrevNode = containerNode.previousElementSibling;
       var beforeStyle = containerPrevNode ? $window.getComputedStyle(containerPrevNode) : {};

Diff for: js/angular/directive/exposeAsideWhen.js

@@ -64,7 +64,7 @@ IonicModule.directive('exposeAsideWhen', ['$window', function($window) {
 
       ionic.on('resize', onResize, $window);
 
-      $scope.$on('$destroy', function(){
+      $scope.$on('$destroy', function() {
         ionic.off('resize', onResize, $window);
       });
 

Diff for: js/angular/directive/itemReorderButton.js

@@ -68,8 +68,8 @@ IonicModule
         };
 
         // prevent clicks from bubbling up to the item
-        if(!$attr.ngClick && !$attr.onClick && !$attr.onclick){
-          $element[0].onclick = function(e){
+        if (!$attr.ngClick && !$attr.onClick && !$attr.onclick) {
+          $element[0].onclick = function(e) {
             e.stopPropagation();
             return false;
           };

Diff for: js/angular/directive/popover.js

@@ -8,7 +8,7 @@ IonicModule
     restrict: 'E',
     transclude: true,
     replace: true,
-    controller: [function(){}],
+    controller: [function() {}],
     template: '<div class="popover-backdrop">' +
                 '<div class="popover-wrapper" ng-transclude></div>' +
               '</div>'

Diff for: js/angular/directive/scroll.js

@@ -77,11 +77,11 @@ function($timeout, $controller, $ionicBind) {
             innerElement.toggleClass('padding', !!newVal);
           });
         }
-        if($scope.$eval($scope.paging) === true) {
+        if ($scope.$eval($scope.paging) === true) {
           innerElement.addClass('scroll-paging');
         }
 
-        if(!$scope.direction) { $scope.direction = 'y'; }
+        if (!$scope.direction) { $scope.direction = 'y'; }
         var isPaging = $scope.$eval($scope.paging) === true;
 
         var scrollViewOptions = {

Diff for: js/angular/service/bind.js

@@ -5,7 +5,7 @@ IonicModule
 .factory('$ionicBind', ['$parse', '$interpolate', function($parse, $interpolate) {
   var LOCAL_REGEXP = /^\s*([@=&])(\??)\s*(\w*)\s*$/;
   return function(scope, attrs, bindDefinition) {
-    forEach(bindDefinition || {}, function (definition, scopeName) {
+    forEach(bindDefinition || {}, function(definition, scopeName) {
       //Adapted from angular.js $compile
       var match = definition.match(LOCAL_REGEXP) || [],
         attrName = match[3] || scopeName,