/
home
/
techb158
/
balavpn.abdallabala.com
/
node_modules
/
eslint
/
lib
/
rules
/
/home/techb158/balavpn.abdallabala.com/node_modules/eslint/lib/rules
mkdir
upload
Name
Size
Mode
Actions
utils/
-
0777
rm
accessor-pairs.js
11210
0666
edit
dl
rm
array-bracket-newline.js
8123
0666
edit
dl
rm
array-bracket-spacing.js
8071
0666
edit
dl
rm
array-callback-return.js
12815
0666
edit
dl
rm
array-element-newline.js
9070
0666
edit
dl
rm
arrow-body-style.js
10365
0666
edit
dl
rm
arrow-parens.js
7119
0666
edit
dl
rm
arrow-spacing.js
4618
0666
edit
dl
rm
block-scoped-var.js
3501
0666
edit
dl
rm
block-spacing.js
5159
0666
edit
dl
rm
brace-style.js
7592
0666
edit
dl
rm
callback-return.js
5879
0666
edit
dl
rm
camelcase.js
10363
0666
edit
dl
rm
capitalized-comments.js
9285
0666
edit
dl
rm
class-methods-use-this.js
6393
0666
edit
dl
rm
comma-dangle.js
11652
0666
edit
dl
rm
comma-spacing.js
5892
0666
edit
dl
rm
comma-style.js
10241
0666
edit
dl
rm
complexity.js
5194
0666
edit
dl
rm
computed-property-spacing.js
6703
0666
edit
dl
rm
consistent-return.js
5739
0666
edit
dl
rm
consistent-this.js
4429
0666
edit
dl
rm
constructor-super.js
12051
0666
edit
dl
rm
curly.js
11758
0666
edit
dl
rm
default-case-last.js
1103
0666
edit
dl
rm
default-case.js
2311
0666
edit
dl
rm
default-param-last.js
1690
0666
edit
dl
rm
dot-location.js
3404
0666
edit
dl
rm
dot-notation.js
5192
0666
edit
dl
rm
eol-last.js
3566
0666
edit
dl
rm
eqeqeq.js
4874
0666
edit
dl
rm
for-direction.js
4305
0666
edit
dl
rm
func-call-spacing.js
8290
0666
edit
dl
rm
func-name-matching.js
8835
0666
edit
dl
rm
func-names.js
5135
0666
edit
dl
rm
func-style.js
5337
0666
edit
dl
rm
function-call-argument-newline.js
4222
0666
edit
dl
rm
function-paren-newline.js
10198
0666
edit
dl
rm
generator-star-spacing.js
6319
0666
edit
dl
rm
getter-return.js
5756
0666
edit
dl
rm
global-require.js
2853
0666
edit
dl
rm
grouped-accessor-pairs.js
7185
0666
edit
dl
rm
guard-for-in.js
1924
0666
edit
dl
rm
handle-callback-err.js
3247
0666
edit
dl
rm
id-blacklist.js
6700
0666
edit
dl
rm
id-denylist.js
6357
0666
edit
dl
rm
id-length.js
5613
0666
edit
dl
rm
id-match.js
9208
0666
edit
dl
rm
implicit-arrow-linebreak.js
3147
0666
edit
dl
rm
indent-legacy.js
35743
0666
edit
dl
rm
indent.js
66474
0666
edit
dl
rm
index.js
17913
0666
edit
dl
rm
init-declarations.js
3974
0666
edit
dl
rm
jsx-quotes.js
3147
0666
edit
dl
rm
key-spacing.js
21268
0666
edit
dl
rm
keyword-spacing.js
20417
0666
edit
dl
rm
line-comment-position.js
3736
0666
edit
dl
rm
linebreak-style.js
3569
0666
edit
dl
rm
lines-around-comment.js
15497
0666
edit
dl
rm
lines-around-directive.js
7023
0666
edit
dl
rm
lines-between-class-members.js
10129
0666
edit
dl
rm
logical-assignment-operators.js
19047
0666
edit
dl
rm
max-classes-per-file.js
1858
0666
edit
dl
rm
max-depth.js
3633
0666
edit
dl
rm
max-len.js
13652
0666
edit
dl
rm
max-lines-per-function.js
6050
0666
edit
dl
rm
max-lines.js
4403
0666
edit
dl
rm
max-nested-callbacks.js
2691
0666
edit
dl
rm
max-params.js
3075
0666
edit
dl
rm
max-statements-per-line.js
6738
0666
edit
dl
rm
max-statements.js
4575
0666
edit
dl
rm
multiline-comment-style.js
18395
0666
edit
dl
rm
multiline-ternary.js
6541
0666
edit
dl
rm
new-cap.js
7204
0666
edit
dl
rm
new-parens.js
3167
0666
edit
dl
rm
newline-after-var.js
8852
0666
edit
dl
rm
newline-before-return.js
7126
0666
edit
dl
rm
newline-per-chained-call.js
3883
0666
edit
dl
rm
no-alert.js
3843
0666
edit
dl
rm
no-array-constructor.js
4995
0666
edit
dl
rm
no-async-promise-executor.js
1007
0666
edit
dl
rm
no-await-in-loop.js
2494
0666
edit
dl
rm
no-bitwise.js
2948
0666
edit
dl
rm
no-buffer-constructor.js
1868
0666
edit
dl
rm
no-caller.js
1086
0666
edit
dl
rm
no-case-declarations.js
1867
0666
edit
dl
rm
no-catch-shadow.js
2515
0666
edit
dl
rm
no-class-assign.js
1566
0666
edit
dl
rm
no-compare-neg-zero.js
1662
0666
edit
dl
rm
no-cond-assign.js
4837
0666
edit
dl
rm
no-confusing-arrow.js
3232
0666
edit
dl
rm
no-console.js
6233
0666
edit
dl
rm
no-const-assign.js
1848
0666
edit
dl
rm
no-constant-binary-expression.js
18150
0666
edit
dl
rm
no-constant-condition.js
4469
0666
edit
dl
rm
no-constructor-return.js
1191
0666
edit
dl
rm
no-continue.js
784
0666
edit
dl
rm
no-control-regex.js
3830
0666
edit
dl
rm
no-debugger.js
793
0666
edit
dl
rm
no-delete-var.js
852
0666
edit
dl
rm
no-div-regex.js
1245
0666
edit
dl
rm
no-dupe-args.js
2387
0666
edit
dl
rm
no-dupe-class-members.js
2905
0666
edit
dl
rm
no-dupe-else-if.js
4197
0666
edit
dl
rm
no-dupe-keys.js
3963
0666
edit
dl
rm
no-duplicate-case.js
1765
0666
edit
dl
rm
no-duplicate-imports.js
10024
0666
edit
dl
rm
no-else-return.js
13405
0666
edit
dl
rm
no-empty-character-class.js
1938
0666
edit
dl
rm
no-empty-function.js
5754
0666
edit
dl
rm
no-empty-pattern.js
1841
0666
edit
dl
rm
no-empty-static-block.js
1589
0666
edit
dl
rm
no-empty.js
3202
0666
edit
dl
rm
no-eq-null.js
1117
0666
edit
dl
rm
no-eval.js
7542
0666
edit
dl
rm
no-ex-assign.js
1301
0666
edit
dl
rm
no-extend-native.js
5421
0666
edit
dl
rm
no-extra-bind.js
6007
0666
edit
dl
rm
no-extra-boolean-cast.js
10590
0666
edit
dl
rm
no-extra-label.js
4265
0666
edit
dl
rm
no-extra-parens.js
43596
0666
edit
dl
rm
no-extra-semi.js
4628
0666
edit
dl
rm
no-fallthrough.js
6966
0666
edit
dl
rm
no-floating-decimal.js
2568
0666
edit
dl
rm
no-func-assign.js
1861
0666
edit
dl
rm
no-global-assign.js
2426
0666
edit
dl
rm
no-implicit-coercion.js
12600
0666
edit
dl
rm
no-implicit-globals.js
4626
0666
edit
dl
rm
no-implied-eval.js
4339
0666
edit
dl
rm
no-import-assign.js
6179
0666
edit
dl
rm
no-inline-comments.js
2691
0666
edit
dl
rm
no-inner-declarations.js
3221
0666
edit
dl
rm
no-invalid-regexp.js
5962
0666
edit
dl
rm
no-invalid-this.js
4488
0666
edit
dl
rm
no-irregular-whitespace.js
7853
0666
edit
dl
rm
no-iterator.js
1110
0666
edit
dl
rm
no-label-var.js
2063
0666
edit
dl
rm
no-labels.js
3365
0666
edit
dl
rm
no-lone-blocks.js
3320
0666
edit
dl
rm
no-lonely-if.js
3303
0666
edit
dl
rm
no-loop-func.js
7035
0666
edit
dl
rm
no-loss-of-precision.js
7393
0666
edit
dl
rm
no-magic-numbers.js
10580
0666
edit
dl
rm
no-misleading-character-class.js
16369
0666
edit
dl
rm
no-mixed-operators.js
7147
0666
edit
dl
rm
no-mixed-requires.js
6383
0666
edit
dl
rm
no-mixed-spaces-and-tabs.js
3465
0666
edit
dl
rm
no-multi-assign.js
1327
0666
edit
dl
rm
no-multi-spaces.js
4588
0666
edit
dl
rm
no-multi-str.js
1610
0666
edit
dl
rm
no-multiple-empty-lines.js
5581
0666
edit
dl
rm
no-native-reassign.js
2765
0666
edit
dl
rm
no-negated-condition.js
2431
0666
edit
dl
rm
no-negated-in-lhs.js
1338
0666
edit
dl
rm
no-nested-ternary.js
960
0666
edit
dl
rm
no-new-func.js
2393
0666
edit
dl
rm
no-new-native-nonconstructor.js
1754
0666
edit
dl
rm
no-new-object.js
1866
0666
edit
dl
rm
no-new-require.js
1588
0666
edit
dl
rm
no-new-symbol.js
1739
0666
edit
dl
rm
no-new-wrappers.js
1460
0666
edit
dl
rm
no-new.js
905
0666
edit
dl
rm
no-nonoctal-decimal-escape.js
4566
0666
edit
dl
rm
no-obj-calls.js
2466
0666
edit
dl
rm
no-object-constructor.js
2923
0666
edit
dl
rm
no-octal-escape.js
1175
0666
edit
dl
rm
no-octal.js
832
0666
edit
dl
rm
no-param-reassign.js
6363
0666
edit
dl
rm
no-path-concat.js
2012
0666
edit
dl
rm
no-plusplus.js
2672
0666
edit
dl
rm
no-process-env.js
1646
0666
edit
dl
rm
no-process-exit.js
1735
0666
edit
dl
rm
no-promise-executor-return.js
6675
0666
edit
dl
rm
no-proto.js
1093
0666
edit
dl
rm
no-prototype-builtins.js
4721
0666
edit
dl
rm
no-redeclare.js
4425
0666
edit
dl
rm
no-regex-spaces.js
5276
0666
edit
dl
rm
no-restricted-exports.js
5636
0666
edit
dl
rm
no-restricted-globals.js
6229
0666
edit
dl
rm
no-restricted-imports.js
23880
0666
edit
dl
rm
no-restricted-modules.js
6128
0666
edit
dl
rm
no-restricted-properties.js
5948
0666
edit
dl
rm
no-restricted-syntax.js
1698
0666
edit
dl
rm
no-return-assign.js
2259
0666
edit
dl
rm
no-return-await.js
4457
0666
edit
dl
rm
no-script-url.js
1512
0666
edit
dl
rm
no-self-assign.js
4666
0666
edit
dl
rm
no-self-compare.js
1705
0666
edit
dl
rm
no-sequences.js
4011
0666
edit
dl
rm
no-setter-return.js
5725
0666
edit
dl
rm
no-shadow-restricted-names.js
2672
0666
edit
dl
rm
no-shadow.js
17353
0666
edit
dl
rm
no-spaced-func.js
2603
0666
edit
dl
rm
no-sparse-arrays.js
1483
0666
edit
dl
rm
no-sync.js
1887
0666
edit
dl
rm
no-tabs.js
2449
0666
edit
dl
rm
no-template-curly-in-string.js
989
0666
edit
dl
rm
no-ternary.js
785
0666
edit
dl
rm
no-this-before-super.js
9043
0666
edit
dl
rm
no-throw-literal.js
1088
0666
edit
dl
rm
no-trailing-spaces.js
6110
0666
edit
dl
rm
no-unassigned-vars.js
1964
0666
edit
dl
rm
no-undef-init.js
2409
0666
edit
dl
rm
no-undef.js
1899
0666
edit
dl
rm
no-undefined.js
1861
0666
edit
dl
rm
no-underscore-dangle.js
9535
0666
edit
dl
rm
no-unexpected-multiline.js
3477
0666
edit
dl
rm
no-unmodified-loop-condition.js
10035
0666
edit
dl
rm
no-unneeded-ternary.js
5935
0666
edit
dl
rm
no-unreachable-loop.js
5076
0666
edit
dl
rm
no-unreachable.js
7498
0666
edit
dl
rm
no-unsafe-finally.js
3339
0666
edit
dl
rm
no-unsafe-negation.js
3972
0666
edit
dl
rm
no-unsafe-optional-chaining.js
5389
0666
edit
dl
rm
no-unused-expressions.js
5520
0666
edit
dl
rm
no-unused-labels.js
3878
0666
edit
dl
rm
no-unused-private-class-members.js
6277
0666
edit
dl
rm
no-unused-vars.js
48552
0666
edit
dl
rm
no-use-before-define.js
11866
0666
edit
dl
rm
no-useless-assignment.js
19334
0666
edit
dl
rm
no-useless-backreference.js
7662
0666
edit
dl
rm
no-useless-call.js
2779
0666
edit
dl
rm
no-useless-catch.js
1310
0666
edit
dl
rm
no-useless-computed-key.js
5936
0666
edit
dl
rm
no-useless-concat.js
2966
0666
edit
dl
rm
no-useless-constructor.js
6986
0666
edit
dl
rm
no-useless-escape.js
11065
0666
edit
dl
rm
no-useless-rename.js
4929
0666
edit
dl
rm
no-useless-return.js
11954
0666
edit
dl
rm
no-var.js
11461
0666
edit
dl
rm
no-void.js
1366
0666
edit
dl
rm
no-warning-comments.js
5670
0666
edit
dl
rm
no-whitespace-before-property.js
3883
0666
edit
dl
rm
no-with.js
757
0666
edit
dl
rm
nonblock-statement-body-position.js
4418
0666
edit
dl
rm
object-curly-newline.js
10115
0666
edit
dl
rm
object-curly-spacing.js
10399
0666
edit
dl
rm
object-property-newline.js
3758
0666
edit
dl
rm
object-shorthand.js
18543
0666
edit
dl
rm
one-var-declaration-per-line.js
2995
0666
edit
dl
rm
one-var.js
19182
0666
edit
dl
rm
operator-assignment.js
7536
0666
edit
dl
rm
operator-linebreak.js
8901
0666
edit
dl
rm
padded-blocks.js
8910
0666
edit
dl
rm
padding-line-between-statements.js
16989
0666
edit
dl
rm
prefer-arrow-callback.js
11667
0666
edit
dl
rm
prefer-const.js
15780
0666
edit
dl
rm
prefer-destructuring.js
8931
0666
edit
dl
rm
prefer-exponentiation-operator.js
6727
0666
edit
dl
rm
prefer-named-capture-group.js
5048
0666
edit
dl
rm
prefer-numeric-literals.js
4504
0666
edit
dl
rm
prefer-object-has-own.js
3804
0666
edit
dl
rm
prefer-object-spread.js
9489
0666
edit
dl
rm
prefer-promise-reject-errors.js
4614
0666
edit
dl
rm
prefer-reflect.js
3828
0666
edit
dl
rm
prefer-regex-literals.js
14379
0666
edit
dl
rm
prefer-rest-params.js
3032
0666
edit
dl
rm
prefer-spread.js
2555
0666
edit
dl
rm
prefer-template.js
10537
0666
edit
dl
rm
preserve-caught-error.js
16269
0666
edit
dl
rm
quote-props.js
10129
0666
edit
dl
rm
quotes.js
10934
0666
edit
dl
rm
radix.js
5549
0666
edit
dl
rm
require-atomic-updates.js
9293
0666
edit
dl
rm
require-await.js
4564
0666
edit
dl
rm
require-unicode-regexp.js
7765
0666
edit
dl
rm
require-yield.js
1718
0666
edit
dl
rm
rest-spread-spacing.js
3739
0666
edit
dl
rm
semi-spacing.js
7384
0666
edit
dl
rm
semi-style.js
5246
0666
edit
dl
rm
semi.js
13324
0666
edit
dl
rm
sort-imports.js
8603
0666
edit
dl
rm
sort-keys.js
5880
0666
edit
dl
rm
sort-vars.js
3320
0666
edit
dl
rm
space-before-blocks.js
6328
0666
edit
dl
rm
space-before-function-paren.js
5253
0666
edit
dl
rm
space-in-parens.js
9314
0666
edit
dl
rm
space-infix-ops.js
5890
0666
edit
dl
rm
space-unary-ops.js
10644
0666
edit
dl
rm
spaced-comment.js
11246
0666
edit
dl
rm
strict.js
9141
0666
edit
dl
rm
switch-colon-spacing.js
4701
0666
edit
dl
rm
symbol-description.js
1701
0666
edit
dl
rm
template-curly-spacing.js
4337
0666
edit
dl
rm
template-tag-spacing.js
3059
0666
edit
dl
rm
unicode-bom.js
1716
0666
edit
dl
rm
use-isnan.js
7130
0666
edit
dl
rm
valid-typeof.js
4262
0666
edit
dl
rm
vars-on-top.js
4304
0666
edit
dl
rm
wrap-iife.js
6914
0666
edit
dl
rm
wrap-regex.js
2167
0666
edit
dl
rm
yield-star-spacing.js
3992
0666
edit
dl
rm
yoda.js
9713
0666
edit
dl
rm
Edit:
/home/techb158/balavpn.abdallabala.com/node_modules/eslint/lib/rules/no-else-return.js
(13405B)
/** * @fileoverview Rule to flag `else` after a `return` in `if` * @author Ian Christian Myers */ "use strict"; //------------------------------------------------------------------------------ // Requirements //------------------------------------------------------------------------------ const astUtils = require("./utils/ast-utils"); const FixTracker = require("./utils/fix-tracker"); //------------------------------------------------------------------------------ // Rule Definition //------------------------------------------------------------------------------ /** @type {import('../types').Rule.RuleModule} */ module.exports = { meta: { type: "suggestion", defaultOptions: [{ allowElseIf: true }], docs: { description: "Disallow `else` blocks after `return` statements in `if` statements", recommended: false, frozen: true, url: "https://eslint.org/docs/latest/rules/no-else-return", }, schema: [ { type: "object", properties: { allowElseIf: { type: "boolean", }, }, additionalProperties: false, }, ], fixable: "code", messages: { unexpected: "Unnecessary 'else' after 'return'.", }, }, create(context) { const [{ allowElseIf }] = context.options; const sourceCode = context.sourceCode; //-------------------------------------------------------------------------- // Helpers //-------------------------------------------------------------------------- /** * Checks whether the given names can be safely used to declare block-scoped variables * in the given scope. Name collisions can produce redeclaration syntax errors, * or silently change references and modify behavior of the original code. * * This is not a generic function. In particular, it is assumed that the scope is a function scope or * a function's inner scope, and that the names can be valid identifiers in the given scope. * @param {string[]} names Array of variable names. * @param {eslint-scope.Scope} scope Function scope or a function's inner scope. * @returns {boolean} True if all names can be safely declared, false otherwise. */ function isSafeToDeclare(names, scope) { if (names.length === 0) { return true; } const functionScope = scope.variableScope; /* * If this is a function scope, scope.variables will contain parameters, implicit variables such as "arguments", * all function-scoped variables ('var'), and block-scoped variables defined in the scope. * If this is an inner scope, scope.variables will contain block-scoped variables defined in the scope. * * Redeclaring any of these would cause a syntax error, except for the implicit variables. */ const declaredVariables = scope.variables.filter( ({ defs }) => defs.length > 0, ); if (declaredVariables.some(({ name }) => names.includes(name))) { return false; } // Redeclaring a catch variable would also cause a syntax error. if (scope !== functionScope && scope.upper.type === "catch") { if ( scope.upper.variables.some(({ name }) => names.includes(name), ) ) { return false; } } /* * Redeclaring an implicit variable, such as "arguments", would not cause a syntax error. * However, if the variable was used, declaring a new one with the same name would change references * and modify behavior. */ const usedImplicitVariables = scope.variables.filter( ({ defs, references }) => defs.length === 0 && references.length > 0, ); if ( usedImplicitVariables.some(({ name }) => names.includes(name)) ) { return false; } /* * Declaring a variable with a name that was already used to reference a variable from an upper scope * would change references and modify behavior. */ if (scope.through.some(t => names.includes(t.identifier.name))) { return false; } /* * If the scope is an inner scope (not the function scope), an uninitialized `var` variable declared inside * the scope node (directly or in one of its descendants) is neither declared nor 'through' in the scope. * * For example, this would be a syntax error "Identifier 'a' has already been declared": * function foo() { if (bar) { let a; if (baz) { var a; } } } */ if (scope !== functionScope) { const scopeNodeRange = scope.block.range; const variablesToCheck = functionScope.variables.filter( ({ name }) => names.includes(name), ); if ( variablesToCheck.some(v => v.defs.some( ({ node: { range } }) => scopeNodeRange[0] <= range[0] && range[1] <= scopeNodeRange[1], ), ) ) { return false; } } return true; } /** * Checks whether the removal of `else` and its braces is safe from variable name collisions. * @param {Node} node The 'else' node. * @param {eslint-scope.Scope} scope The scope in which the node and the whole 'if' statement is. * @returns {boolean} True if it is safe, false otherwise. */ function isSafeFromNameCollisions(node, scope) { if (node.type === "FunctionDeclaration") { // Conditional function declaration. Scope and hoisting are unpredictable, different engines work differently. return false; } if (node.type !== "BlockStatement") { return true; } const elseBlockScope = scope.childScopes.find( ({ block }) => block === node, ); if (!elseBlockScope) { // ecmaVersion < 6, `else` block statement cannot have its own scope, no possible collisions. return true; } /* * elseBlockScope is supposed to merge into its upper scope. elseBlockScope.variables array contains * only block-scoped variables (such as let and const variables or class and function declarations) * defined directly in the elseBlockScope. These are exactly the only names that could cause collisions. */ const namesToCheck = elseBlockScope.variables.map( ({ name }) => name, ); return isSafeToDeclare(namesToCheck, scope); } /** * Display the context report if rule is violated * @param {Node} elseNode The 'else' node * @returns {void} */ function displayReport(elseNode) { const currentScope = sourceCode.getScope(elseNode.parent); context.report({ node: elseNode, messageId: "unexpected", fix(fixer) { if (!isSafeFromNameCollisions(elseNode, currentScope)) { return null; } const startToken = sourceCode.getFirstToken(elseNode); const elseToken = sourceCode.getTokenBefore(startToken); const source = sourceCode.getText(elseNode); const lastIfToken = sourceCode.getTokenBefore(elseToken); let fixedSource, firstTokenOfElseBlock; if ( startToken.type === "Punctuator" && startToken.value === "{" ) { firstTokenOfElseBlock = sourceCode.getTokenAfter(startToken); } else { firstTokenOfElseBlock = startToken; } /* * If the if block does not have curly braces and does not end in a semicolon * and the else block starts with (, [, /, +, ` or -, then it is not * safe to remove the else keyword, because ASI will not add a semicolon * after the if block */ const ifBlockMaybeUnsafe = elseNode.parent.consequent.type !== "BlockStatement" && lastIfToken.value !== ";"; const elseBlockUnsafe = /^[([/+`-]/u.test( firstTokenOfElseBlock.value, ); if (ifBlockMaybeUnsafe && elseBlockUnsafe) { return null; } const endToken = sourceCode.getLastToken(elseNode); const lastTokenOfElseBlock = sourceCode.getTokenBefore(endToken); if (lastTokenOfElseBlock.value !== ";") { const nextToken = sourceCode.getTokenAfter(endToken); const nextTokenUnsafe = nextToken && /^[([/+`-]/u.test(nextToken.value); const nextTokenOnSameLine = nextToken && nextToken.loc.start.line === lastTokenOfElseBlock.loc.start.line; /* * If the else block contents does not end in a semicolon, * and the else block starts with (, [, /, +, ` or -, then it is not * safe to remove the else block, because ASI will not add a semicolon * after the remaining else block contents */ if ( nextTokenUnsafe || (nextTokenOnSameLine && nextToken.value !== "}") ) { return null; } } if ( startToken.type === "Punctuator" && startToken.value === "{" ) { fixedSource = source.slice(1, -1); } else { fixedSource = source; } /* * Extend the replacement range to include the entire * function to avoid conflicting with no-useless-return. * https://github.com/eslint/eslint/issues/8026 * * Also, to avoid name collisions between two else blocks. */ return new FixTracker(fixer, sourceCode) .retainEnclosingFunction(elseNode) .replaceTextRange( [elseToken.range[0], elseNode.range[1]], fixedSource, ); }, }); } /** * Check to see if the node is a ReturnStatement * @param {Node} node The node being evaluated * @returns {boolean} True if node is a return */ function checkForReturn(node) { return node.type === "ReturnStatement"; } /** * Naive return checking, does not iterate through the whole * BlockStatement because we make the assumption that the ReturnStatement * will be the last node in the body of the BlockStatement. * @param {Node} node The consequent/alternate node * @returns {boolean} True if it has a return */ function naiveHasReturn(node) { if (node.type === "BlockStatement") { const body = node.body, lastChildNode = body.at(-1); return lastChildNode && checkForReturn(lastChildNode); } return checkForReturn(node); } /** * Check to see if the node is valid for evaluation, * meaning it has an else. * @param {Node} node The node being evaluated * @returns {boolean} True if the node is valid */ function hasElse(node) { return node.alternate && node.consequent; } /** * If the consequent is an IfStatement, check to see if it has an else * and both its consequent and alternate path return, meaning this is * a nested case of rule violation. If-Else not considered currently. * @param {Node} node The consequent node * @returns {boolean} True if this is a nested rule violation */ function checkForIf(node) { return ( node.type === "IfStatement" && hasElse(node) && naiveHasReturn(node.alternate) && naiveHasReturn(node.consequent) ); } /** * Check the consequent/body node to make sure it is not * a ReturnStatement or an IfStatement that returns on both * code paths. * @param {Node} node The consequent or body node * @returns {boolean} `true` if it is a Return/If node that always returns. */ function checkForReturnOrIf(node) { return checkForReturn(node) || checkForIf(node); } /** * Check whether a node returns in every codepath. * @param {Node} node The node to be checked * @returns {boolean} `true` if it returns on every codepath. */ function alwaysReturns(node) { if (node.type === "BlockStatement") { // If we have a BlockStatement, check each consequent body node. return node.body.some(checkForReturnOrIf); } /* * If not a block statement, make sure the consequent isn't a * ReturnStatement or an IfStatement with returns on both paths. */ return checkForReturnOrIf(node); } /** * Check the if statement, but don't catch else-if blocks. * @returns {void} * @param {Node} node The node for the if statement to check * @private */ function checkIfWithoutElse(node) { const parent = node.parent; /* * Fixing this would require splitting one statement into two, so no error should * be reported if this node is in a position where only one statement is allowed. */ if (!astUtils.STATEMENT_LIST_PARENTS.has(parent.type)) { return; } const consequents = []; let alternate; for ( let currentNode = node; currentNode.type === "IfStatement"; currentNode = currentNode.alternate ) { if (!currentNode.alternate) { return; } consequents.push(currentNode.consequent); alternate = currentNode.alternate; } if (consequents.every(alwaysReturns)) { displayReport(alternate); } } /** * Check the if statement * @returns {void} * @param {Node} node The node for the if statement to check * @private */ function checkIfWithElse(node) { const parent = node.parent; /* * Fixing this would require splitting one statement into two, so no error should * be reported if this node is in a position where only one statement is allowed. */ if (!astUtils.STATEMENT_LIST_PARENTS.has(parent.type)) { return; } const alternate = node.alternate; if (alternate && alwaysReturns(node.consequent)) { displayReport(alternate); } } //-------------------------------------------------------------------------- // Public API //-------------------------------------------------------------------------- return { "IfStatement:exit": allowElseIf ? checkIfWithoutElse : checkIfWithElse, }; }, };
Save
cmd:
run