|
123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113 |
- /**
- * @fileoverview Rule to disallow whitespace before properties
- * @author Kai Cataldo
- */
- "use strict";
-
- //------------------------------------------------------------------------------
- // Requirements
- //------------------------------------------------------------------------------
-
- const astUtils = require("./utils/ast-utils");
-
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
-
- module.exports = {
- meta: {
- type: "layout",
-
- docs: {
- description: "disallow whitespace before properties",
- category: "Stylistic Issues",
- recommended: false,
- url: "https://eslint.org/docs/rules/no-whitespace-before-property"
- },
-
- fixable: "whitespace",
- schema: [],
-
- messages: {
- unexpectedWhitespace: "Unexpected whitespace before property {{propName}}."
- }
- },
-
- create(context) {
- const sourceCode = context.getSourceCode();
-
- //--------------------------------------------------------------------------
- // Helpers
- //--------------------------------------------------------------------------
-
- /**
- * Reports whitespace before property token
- * @param {ASTNode} node the node to report in the event of an error
- * @param {Token} leftToken the left token
- * @param {Token} rightToken the right token
- * @returns {void}
- * @private
- */
- function reportError(node, leftToken, rightToken) {
- context.report({
- node,
- messageId: "unexpectedWhitespace",
- data: {
- propName: sourceCode.getText(node.property)
- },
- fix(fixer) {
- let replacementText = "";
-
- if (!node.computed && !node.optional && astUtils.isDecimalInteger(node.object)) {
-
- /*
- * If the object is a number literal, fixing it to something like 5.toString() would cause a SyntaxError.
- * Don't fix this case.
- */
- return null;
- }
-
- // Don't fix if comments exist.
- if (sourceCode.commentsExistBetween(leftToken, rightToken)) {
- return null;
- }
-
- if (node.optional) {
- replacementText = "?.";
- } else if (!node.computed) {
- replacementText = ".";
- }
-
- return fixer.replaceTextRange([leftToken.range[1], rightToken.range[0]], replacementText);
- }
- });
- }
-
- //--------------------------------------------------------------------------
- // Public
- //--------------------------------------------------------------------------
-
- return {
- MemberExpression(node) {
- let rightToken;
- let leftToken;
-
- if (!astUtils.isTokenOnSameLine(node.object, node.property)) {
- return;
- }
-
- if (node.computed) {
- rightToken = sourceCode.getTokenBefore(node.property, astUtils.isOpeningBracketToken);
- leftToken = sourceCode.getTokenBefore(rightToken, node.optional ? 1 : 0);
- } else {
- rightToken = sourceCode.getFirstToken(node.property);
- leftToken = sourceCode.getTokenBefore(rightToken, 1);
- }
-
- if (sourceCode.isSpaceBetweenTokens(leftToken, rightToken)) {
- reportError(node, leftToken, rightToken);
- }
- }
- };
- }
- };
|