Skip to content

Commit 7f65f97

Browse files
hzoocaitp
hzoo
authored and
caitp
committed
style(*): add rule requireSpacesInFunction beforeOpeningCurlyBrace
This rule enforces a space after the curly brace for function declarations, anonymous function expressions, and named function expressions.
1 parent 9221628 commit 7f65f97

72 files changed

Lines changed: 324 additions & 321 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.jscs.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,8 @@
1616
"disallowTrailingComma": true,
1717
"disallowTrailingWhitespace": true,
1818
"requireSpaceAfterKeywords": ["if", "else", "for", "while", "do", "switch", "return", "try", "catch"],
19+
"requireSpacesInFunction": {
20+
"beforeOpeningCurlyBrace": true
21+
},
1922
"validateParameterSeparator": ", "
2023
}

‎src/Angular.js‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ var VALIDITY_STATE_PROPERTY = 'validity';
123123
* @param {string} string String to be converted to lowercase.
124124
* @returns {string} Lowercased string.
125125
*/
126-
var lowercase = function(string){return isString(string) ? string.toLowerCase() : string;};
126+
var lowercase = function(string) {return isString(string) ? string.toLowerCase() : string;};
127127
var hasOwnProperty = Object.prototype.hasOwnProperty;
128128

129129
/**
@@ -136,7 +136,7 @@ var hasOwnProperty = Object.prototype.hasOwnProperty;
136136
* @param {string} string String to be converted to uppercase.
137137
* @returns {string} Uppercased string.
138138
*/
139-
var uppercase = function(string){return isString(string) ? string.toUpperCase() : string;};
139+
var uppercase = function(string) {return isString(string) ? string.toUpperCase() : string;};
140140

141141

142142
var manualLowercase = function(s) {
@@ -419,7 +419,7 @@ function valueFn(value) {return function() {return value;};}
419419
* @param {*} value Reference to check.
420420
* @returns {boolean} True if `value` is undefined.
421421
*/
422-
function isUndefined(value){return typeof value === 'undefined';}
422+
function isUndefined(value) {return typeof value === 'undefined';}
423423

424424

425425
/**
@@ -434,7 +434,7 @@ function isUndefined(value){return typeof value === 'undefined';}
434434
* @param {*} value Reference to check.
435435
* @returns {boolean} True if `value` is defined.
436436
*/
437-
function isDefined(value){return typeof value !== 'undefined';}
437+
function isDefined(value) {return typeof value !== 'undefined';}
438438

439439

440440
/**
@@ -450,7 +450,7 @@ function isDefined(value){return typeof value !== 'undefined';}
450450
* @param {*} value Reference to check.
451451
* @returns {boolean} True if `value` is an `Object` but not `null`.
452452
*/
453-
function isObject(value){
453+
function isObject(value) {
454454
// http://jsperf.com/isobject4
455455
return value !== null && typeof value === 'object';
456456
}
@@ -468,7 +468,7 @@ function isObject(value){
468468
* @param {*} value Reference to check.
469469
* @returns {boolean} True if `value` is a `String`.
470470
*/
471-
function isString(value){return typeof value === 'string';}
471+
function isString(value) {return typeof value === 'string';}
472472

473473

474474
/**
@@ -483,7 +483,7 @@ function isString(value){return typeof value === 'string';}
483483
* @param {*} value Reference to check.
484484
* @returns {boolean} True if `value` is a `Number`.
485485
*/
486-
function isNumber(value){return typeof value === 'number';}
486+
function isNumber(value) {return typeof value === 'number';}
487487

488488

489489
/**
@@ -529,7 +529,7 @@ var isArray = Array.isArray;
529529
* @param {*} value Reference to check.
530530
* @returns {boolean} True if `value` is a `Function`.
531531
*/
532-
function isFunction(value){return typeof value === 'function';}
532+
function isFunction(value) {return typeof value === 'function';}
533533

534534

535535
/**

‎src/AngularPublic.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ var version = {
110110
};
111111

112112

113-
function publishExternalAPI(angular){
113+
function publishExternalAPI(angular) {
114114
extend(angular, {
115115
'bootstrap': bootstrap,
116116
'copy': copy,

‎src/auto/injector.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -705,7 +705,7 @@ function createInjector(modulesToLoad, strictDi) {
705705
////////////////////////////////////
706706
// Module Loading
707707
////////////////////////////////////
708-
function loadModules(modulesToLoad){
708+
function loadModules(modulesToLoad) {
709709
var runBlocks = [], moduleFn;
710710
forEach(modulesToLoad, function(module) {
711711
if (loadedModules.get(module)) return;

‎src/jqLite.js‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ function jqLiteClone(element) {
251251
return element.cloneNode(true);
252252
}
253253

254-
function jqLiteDealoc(element, onlyDescendants){
254+
function jqLiteDealoc(element, onlyDescendants) {
255255
if (!onlyDescendants) jqLiteRemoveData(element);
256256

257257
if (element.querySelectorAll) {
@@ -493,7 +493,7 @@ var JQLitePrototype = JQLite.prototype = {
493493
},
494494
toString: function() {
495495
var value = [];
496-
forEach(this, function(e){ value.push('' + e);});
496+
forEach(this, function(e) { value.push('' + e);});
497497
return '[' + value.join(', ') + ']';
498498
},
499499

@@ -584,7 +584,7 @@ forEach({
584584
}
585585
},
586586

587-
attr: function(element, name, value){
587+
attr: function(element, name, value) {
588588
var lowercasedName = lowercase(name);
589589
if (BOOLEAN_ATTR[lowercasedName]) {
590590
if (isDefined(value)) {
@@ -658,7 +658,7 @@ forEach({
658658
},
659659

660660
empty: jqLiteEmpty
661-
}, function(fn, name){
661+
}, function(fn, name) {
662662
/**
663663
* Properties: writes return selection, reads return first value
664664
*/
@@ -766,7 +766,7 @@ function createEventHandler(element, events) {
766766
forEach({
767767
removeData: jqLiteRemoveData,
768768

769-
on: function jqLiteOn(element, type, fn, unsupported){
769+
on: function jqLiteOn(element, type, fn, unsupported) {
770770
if (isDefined(unsupported)) throw jqLiteMinErr('onargs', 'jqLite#on() does not support the `selector` or `eventData` parameters');
771771

772772
// Do not add event handlers to non-elements because they will not be cleaned up.
@@ -836,7 +836,7 @@ forEach({
836836
replaceWith: function(element, replaceNode) {
837837
var index, parent = element.parentNode;
838838
jqLiteDealoc(element);
839-
forEach(new JQLite(replaceNode), function(node){
839+
forEach(new JQLite(replaceNode), function(node) {
840840
if (index) {
841841
parent.insertBefore(node, index.nextSibling);
842842
} else {
@@ -848,7 +848,7 @@ forEach({
848848

849849
children: function(element) {
850850
var children = [];
851-
forEach(element.childNodes, function(element){
851+
forEach(element.childNodes, function(element) {
852852
if (element.nodeType === NODE_TYPE_ELEMENT)
853853
children.push(element);
854854
});
@@ -874,7 +874,7 @@ forEach({
874874
prepend: function(element, node) {
875875
if (element.nodeType === NODE_TYPE_ELEMENT) {
876876
var index = element.firstChild;
877-
forEach(new JQLite(node), function(child){
877+
forEach(new JQLite(node), function(child) {
878878
element.insertBefore(child, index);
879879
});
880880
}
@@ -911,7 +911,7 @@ forEach({
911911

912912
toggleClass: function(element, selector, condition) {
913913
if (selector) {
914-
forEach(selector.split(' '), function(className){
914+
forEach(selector.split(' '), function(className) {
915915
var classCondition = condition;
916916
if (isUndefined(classCondition)) {
917917
classCondition = !jqLiteHasClass(element, className);
@@ -976,7 +976,7 @@ forEach({
976976
});
977977
}
978978
}
979-
}, function(fn, name){
979+
}, function(fn, name) {
980980
/**
981981
* chaining functions
982982
*/

‎src/ng/asyncCallback.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
function $$AsyncCallbackProvider(){
3+
function $$AsyncCallbackProvider() {
44
this.$get = ['$$rAF', '$timeout', function($$rAF, $timeout) {
55
return $$rAF.supported
66
? function(fn) { return $$rAF(fn); }

‎src/ng/browser.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ function Browser(window, document, $log, $sniffer) {
7272
// force browser to execute all pollFns - this is needed so that cookies and other pollers fire
7373
// at some deterministic time in respect to the test runner's actions. Leaving things up to the
7474
// regular poller would result in flaky tests.
75-
forEach(pollFns, function(pollFn){ pollFn(); });
75+
forEach(pollFns, function(pollFn) { pollFn(); });
7676

7777
if (outstandingRequestCount === 0) {
7878
callback();
@@ -114,7 +114,7 @@ function Browser(window, document, $log, $sniffer) {
114114
*/
115115
function startPoller(interval, setTimeout) {
116116
(function check() {
117-
forEach(pollFns, function(pollFn){ pollFn(); });
117+
forEach(pollFns, function(pollFn) { pollFn(); });
118118
pollTimeout = setTimeout(check, interval);
119119
})();
120120
}
@@ -449,9 +449,9 @@ function Browser(window, document, $log, $sniffer) {
449449

450450
}
451451

452-
function $BrowserProvider(){
452+
function $BrowserProvider() {
453453
this.$get = ['$window', '$log', '$sniffer', '$document',
454-
function($window, $log, $sniffer, $document){
454+
function($window, $log, $sniffer, $document) {
455455
return new Browser($window, $document, $log, $sniffer);
456456
}];
457457
}

‎src/ng/compile.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1145,7 +1145,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
11451145
}
11461146
// We can not compile top level text elements since text nodes can be merged and we will
11471147
// not be able to attach scope data to them, so we will wrap them in <span>
1148-
forEach($compileNodes, function(node, index){
1148+
forEach($compileNodes, function(node, index) {
11491149
if (node.nodeType == NODE_TYPE_TEXT && node.nodeValue.match(/\S+/) /* non-empty */ ) {
11501150
$compileNodes[index] = jqLite(node).wrap('<span></span>').parent()[0];
11511151
}
@@ -1155,7 +1155,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
11551155
maxPriority, ignoreDirective, previousCompileContext);
11561156
compile.$$addScopeClass($compileNodes);
11571157
var namespace = null;
1158-
return function publicLinkFn(scope, cloneConnectFn, transcludeControllers, parentBoundTranscludeFn, futureParentElement){
1158+
return function publicLinkFn(scope, cloneConnectFn, transcludeControllers, parentBoundTranscludeFn, futureParentElement) {
11591159
assertArg(scope, 'scope');
11601160
if (!namespace) {
11611161
namespace = detectNamespaceForChildElements(futureParentElement);
@@ -2504,15 +2504,15 @@ function nodesetLinkingFn(
25042504
/* NodeList */ nodeList,
25052505
/* Element */ rootElement,
25062506
/* function(Function) */ boundTranscludeFn
2507-
){}
2507+
) {}
25082508

25092509
function directiveLinkingFn(
25102510
/* nodesetLinkingFn */ nodesetLinkingFn,
25112511
/* angular.Scope */ scope,
25122512
/* Node */ node,
25132513
/* Element */ rootElement,
25142514
/* function(Function) */ boundTranscludeFn
2515-
){}
2515+
) {}
25162516

25172517
function tokenDifference(str1, str2) {
25182518
var values = '',

‎src/ng/directive/a.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ var htmlAnchorDirective = valueFn({
2121
// SVGAElement does not use the href attribute, but rather the 'xlinkHref' attribute.
2222
var href = toString.call(element.prop('href')) === '[object SVGAnimatedString]' ?
2323
'xlink:href' : 'href';
24-
element.on('click', function(event){
24+
element.on('click', function(event) {
2525
// if we have no href url, then don't navigate anywhere.
2626
if (!element.attr(href)) {
2727
event.preventDefault();

‎src/ng/directive/ngShowHide.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ var ngShowDirective = ['$animate', function($animate) {
162162
restrict: 'A',
163163
multiElement: true,
164164
link: function(scope, element, attr) {
165-
scope.$watch(attr.ngShow, function ngShowWatchAction(value){
165+
scope.$watch(attr.ngShow, function ngShowWatchAction(value) {
166166
// we're adding a temporary, animation-specific class for ng-hide since this way
167167
// we can control when the element is actually displayed on screen without having
168168
// to have a global/greedy CSS selector that breaks when other animations are run.
@@ -323,7 +323,7 @@ var ngHideDirective = ['$animate', function($animate) {
323323
restrict: 'A',
324324
multiElement: true,
325325
link: function(scope, element, attr) {
326-
scope.$watch(attr.ngHide, function ngHideWatchAction(value){
326+
scope.$watch(attr.ngHide, function ngHideWatchAction(value) {
327327
// The comment inside of the ngShowDirective explains why we add and
328328
// remove a temporary class for the show/hide animation
329329
$animate[value ? 'addClass' : 'removeClass'](element,NG_HIDE_CLASS, {

0 commit comments

Comments
 (0)