Permalink
Jump to Line
Branch:
master
Switch branches/tags
dollar-ignores
feature/new-rule-require-new-lines-in-array
gh-pages
issues/1242
issues/1271-related
master
mdevils/cst
Nothing to show
Nothing to show
Fetching contributors…
![]()
Cannot retrieve contributors at this time
| /** | |
| * Requires newline after blocks | |
| * | |
| * Type: `Boolean` or `Object` | |
| * | |
| * Values: | |
| * - `true`: always require a newline after blocks | |
| * - `Object`: | |
| * - `"allExcept"`: `Array` | |
| * - `"inCallExpressions"` Blocks don't need a line of padding in function argument lists | |
| * - `"inNewExpressions"` Blocks don't need a line of padding in constructor argument lists | |
| * - `"inArrayExpressions"` Blocks don't need a line of padding in arrays | |
| * - `"inProperties"` Blocks don't need a line of padding as object properties | |
| * | |
| * #### Example | |
| * | |
| * ```js | |
| * "requirePaddingNewLinesAfterBlocks": true | |
| * "requirePaddingNewLinesAfterBlocks": { | |
| * "allExcept": ["inCallExpressions", "inNewExpressions", "inArrayExpressions", "inProperties"] | |
| * } | |
| * ``` | |
| * | |
| * ##### Valid | |
| * | |
| * ```js | |
| * function () { | |
| * for (var i = 0; i < 2; i++) { | |
| * if (true) { | |
| * return false; | |
| * } | |
| * | |
| * continue; | |
| * } | |
| * | |
| * var obj = { | |
| * foo: function() { | |
| * return 1; | |
| * }, | |
| * | |
| * bar: function() { | |
| * return 2; | |
| * } | |
| * }; | |
| * | |
| * func( | |
| * function() { | |
| * } | |
| * ); | |
| * | |
| * var a = [ | |
| * function() { | |
| * }, | |
| * | |
| * function() { | |
| * } | |
| * ] | |
| * | |
| * } | |
| * ``` | |
| * | |
| * ##### Valid for `{ "allExcept": ["inCallExpressions"] }` | |
| * | |
| * ```js | |
| * func( | |
| * 2, | |
| * 3, | |
| * function() { | |
| * } | |
| * ); | |
| * ``` | |
| * | |
| * ##### Valid for `{ "allExcept": ["inNewExpressions"] }` | |
| * | |
| * ```js | |
| * new SomeClass( | |
| * 2, | |
| * 3, | |
| * function() { | |
| * } | |
| * ); | |
| * ``` | |
| * | |
| * ##### Valid for `{ "allExcept": ["inArrayExpressions"] }` | |
| * | |
| * ```js | |
| * var foo = [ | |
| * 2, | |
| * 3, | |
| * function() { | |
| * } | |
| * ]; | |
| * ``` | |
| * ##### Valid for `{ "allExcept": ["inProperties"] }` | |
| * | |
| * ```js | |
| * var foo = { | |
| * a: 2, | |
| * b: function() { | |
| * }, | |
| * c: 3 | |
| * ]; | |
| * ``` | |
| * | |
| * ##### Invalid | |
| * | |
| * ```js | |
| * function () { | |
| * for (var i = 0; i < 2; i++) { | |
| * if (true) { | |
| * return false; | |
| * } | |
| * continue; | |
| * } | |
| * } | |
| * ``` | |
| */ | |
| var assert = require('assert'); | |
| var excludes = { | |
| 'IfStatement': ['else'], | |
| 'DoWhileStatement': ['while'], | |
| 'TryStatement': ['catch', 'finally'], | |
| 'CatchClause': ['finally'], | |
| 'FunctionExpression': ['.'], | |
| 'ArrowFunctionExpression': [')'] | |
| }; | |
| function isException(parent, exceptions) { | |
| var grandpa = parent.parentNode; | |
| // Check if this block is used in call or array expression | |
| if (exceptions[grandpa.type]) { | |
| return true; | |
| } | |
| return false; | |
| } | |
| module.exports = function() {}; | |
| module.exports.prototype = { | |
| configure: function(value) { | |
| this.exceptions = { | |
| 'CallExpression': false, | |
| 'NewExpression': false, | |
| 'ArrayExpression': false, | |
| 'Property': false | |
| }; | |
| var optionName = this.getOptionName(); | |
| if (typeof value === 'object') { | |
| assert(Array.isArray(value.allExcept), optionName + ' option requires "allExcept" ' + | |
| 'to be an array'); | |
| assert(value.allExcept.length > 0, optionName + ' option requires "allExcept" ' + | |
| 'to have at least one item or be set to `true`'); | |
| value.allExcept.forEach(function(except) { | |
| if (except === 'inCallExpressions') { | |
| this.exceptions.CallExpression = true; | |
| } else if (except === 'inNewExpressions') { | |
| this.exceptions.NewExpression = true; | |
| } else if (except === 'inArrayExpressions') { | |
| this.exceptions.ArrayExpression = true; | |
| } else if (except === 'inProperties') { | |
| this.exceptions.Property = true; | |
| } else { | |
| assert(false, optionName + ' option requires "allExcept" to only have ' + | |
| 'one of "inCallExpressions", "inNewExpressions", "inArrayExpressions" or "inProperties'); | |
| } | |
| }, this); | |
| } else { | |
| assert(value === true, | |
| optionName + ' option requires true value or object' | |
| ); | |
| } | |
| }, | |
| getOptionName: function() { | |
| return 'requirePaddingNewLinesAfterBlocks'; | |
| }, | |
| check: function(file, errors) { | |
| file.iterateNodesByType('BlockStatement', (function(node) { | |
| var endToken = file.getLastNodeToken(node); | |
| var parentNode = node.parentNode; | |
| if (isException(parentNode, this.exceptions)) { | |
| return; | |
| } | |
| var nextToken = file.getNextToken(endToken); | |
| while (nextToken.type !== 'EOF') { | |
| var excludeValues = excludes[parentNode.type]; | |
| if (excludeValues && excludeValues.indexOf(nextToken.value) !== -1) { | |
| return; | |
| } | |
| if (endToken.loc.end.line === nextToken.loc.start.line) { | |
| endToken = nextToken; | |
| nextToken = file.getNextToken(nextToken); | |
| continue; | |
| } | |
| if (nextToken.type === 'Punctuator' && ( | |
| nextToken.value === '}' || | |
| nextToken.value === ']' || | |
| nextToken.value === ')') | |
| ) { | |
| return; | |
| } | |
| errors.assert.linesBetween({ | |
| token: endToken, | |
| nextToken: nextToken, | |
| atLeast: 2, | |
| message: 'Missing newline after block' | |
| }); | |
| return; | |
| } | |
| }).bind(this)); | |
| } | |
| }; |