@angular-eslint/eslint-plugin-template 16.0.4-alpha.9 → 16.1.0

package/dist/rules/attributes-order.js

@@ -146,9 +146,13 @@ function byLocation(one, other) {
 }
 function byOrder(order, alphabetical) {
     return function (one, other) {
+        var _a, _b, _c, _d;
         const orderComparison = getOrderIndex(one, order) - getOrderIndex(other, order);
         if (alphabetical && orderComparison === 0) {
-            return one.name > other.name ? 1 : -1;
+            return ((_b = (_a = one.keySpan) === null || _a === void 0 ? void 0 : _a.details) !== null && _b !== void 0 ? _b : one.name) >
+                ((_d = (_c = other.keySpan) === null || _c === void 0 ? void 0 : _c.details) !== null && _d !== void 0 ? _d : other.name)
+                ? 1
+                : -1;
         }
         return orderComparison;
     };
@@ -175,9 +179,27 @@ function toTemplateReferenceVariableOrderType(reference) {
     return Object.assign(Object.assign({}, reference), { orderType: "TEMPLATE_REFERENCE" /* OrderType.TemplateReferenceVariable */ });
 }
 function extractTemplateAttrs(node) {
-    return isTmplAstTemplate(node.parent)
-        ? node.parent.templateAttrs.map(toStructuralDirectiveOrderType)
-        : [];
+    if (!isTmplAstTemplate(node.parent)) {
+        return [];
+    }
+    /*
+     * There may be multiple "attributes" for a structural directive even though
+     * there is only a single HTML attribute:
+     * e.g. `<ng-container *ngFor="let foo of bar"></ng-container>`
+     * will parsed as two attributes (`ngFor` and `ngForOf`)
+     */
+    const attrs = node.parent.templateAttrs.map(toStructuralDirectiveOrderType);
+    // Pick up on any subsequent `let` bindings, e.g. `index as i`
+    let sourceEnd = attrs[attrs.length - 1].sourceSpan.end;
+    node.parent.variables.forEach((v) => {
+        if (v.sourceSpan.start.offset <= sourceEnd.offset &&
+            sourceEnd.offset < v.sourceSpan.end.offset) {
+            sourceEnd = v.sourceSpan.end;
+        }
+    });
+    return [
+        Object.assign(Object.assign({}, attrs[0]), { sourceSpan: new bundled_angular_compiler_1.ParseSourceSpan(attrs[0].sourceSpan.start, sourceEnd) }),
+    ];
 }
 function normalizeInputsOutputs(inputs, outputs) {
     const extractedInputs = inputs
@@ -200,19 +222,21 @@ function isOnSameLocation(input, output) {
         input.sourceSpan.end === output.sourceSpan.end);
 }
 function getMessageName(expected) {
+    var _a, _b;
+    const fullName = (_b = (_a = expected.keySpan) === null || _a === void 0 ? void 0 : _a.details) !== null && _b !== void 0 ? _b : expected.name;
     switch (expected.orderType) {
         case "STRUCTURAL_DIRECTIVE" /* OrderType.StructuralDirective */:
-            return `*${expected.name}`;
+            return `*${fullName}`;
         case "TEMPLATE_REFERENCE" /* OrderType.TemplateReferenceVariable */:
-            return `#${expected.name}`;
+            return `#${fullName}`;
         case "INPUT_BINDING" /* OrderType.InputBinding */:
-            return `[${expected.name}]`;
+            return `[${fullName}]`;
         case "OUTPUT_BINDING" /* OrderType.OutputBinding */:
-            return `(${expected.name})`;
+            return `(${fullName})`;
         case "TWO_WAY_BINDING" /* OrderType.TwoWayBinding */:
-            return `[(${expected.name})]`;
+            return `[(${fullName})]`;
         default:
-            return expected.name;
+            return fullName;
     }
 }
 function getStartPos(expected) {

package/dist/rules/no-duplicate-attributes.js

@@ -7,6 +7,7 @@ const get_original_attribute_name_1 = require("../utils/get-original-attribute-n
 exports.RULE_NAME = 'no-duplicate-attributes';
 const DEFAULT_OPTIONS = {
     allowTwoWayDataBinding: true,
+    allowStylePrecedenceDuplicates: false,
     ignore: [],
 };
 exports.default = (0, create_eslint_rule_1.createESLintRule)({
@@ -27,6 +28,11 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
                         default: DEFAULT_OPTIONS.allowTwoWayDataBinding,
                         description: `Whether or not two-way data binding is allowed as an exception to the rule.`,
                     },
+                    allowStylePrecedenceDuplicates: {
+                        type: 'boolean',
+                        default: DEFAULT_OPTIONS.allowStylePrecedenceDuplicates,
+                        description: `Whether or not Angular style precedence is allowed as an exception to the rule. See https://angular.io/guide/style-precedence#style-precedence`,
+                    },
                     ignore: {
                         type: 'array',
                         items: { type: 'string' },
@@ -44,14 +50,37 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         },
     },
     defaultOptions: [DEFAULT_OPTIONS],
-    create(context, [{ allowTwoWayDataBinding, ignore }]) {
+    create(context, [{ allowTwoWayDataBinding, allowStylePrecedenceDuplicates, ignore }]) {
         const parserServices = (0, utils_1.getTemplateParserServices)(context);
         return {
             Element$1({ inputs, outputs, attributes }) {
-                const duplicateInputsAndAttributes = findDuplicates([
+                // According to the Angular documentation (https://angular.io/guide/style-precedence#style-precedence)
+                // Angular merges both attributes which means their combined use can be seen as valid
+                const angularStylePrecedenceDuplicatesAllowed = ['class', 'style'];
+                let duplicateInputsAndAttributes = findDuplicates([
                     ...inputs,
                     ...attributes,
                 ]);
+                if (allowStylePrecedenceDuplicates) {
+                    const inputsIgnored = inputs.filter((input) => angularStylePrecedenceDuplicatesAllowed.includes((0, get_original_attribute_name_1.getOriginalAttributeName)(input)));
+                    if ((inputsIgnored === null || inputsIgnored === void 0 ? void 0 : inputsIgnored.length) > 0) {
+                        const attributesIgnored = attributes.filter((attr) => angularStylePrecedenceDuplicatesAllowed.includes((0, get_original_attribute_name_1.getOriginalAttributeName)(attr)));
+                        const inputsNotIgnored = inputs.filter((input) => !inputsIgnored.includes(input));
+                        const attributesNotIgnored = attributes.filter((attr) => !attributesIgnored.includes(attr));
+                        const ignoreDuplicated = [
+                            ...findDuplicates(inputsIgnored),
+                            ...findDuplicates(attributesIgnored),
+                        ];
+                        const notIgnoredDuplicates = [
+                            ...findDuplicates(inputsNotIgnored),
+                            ...findDuplicates(attributesNotIgnored),
+                        ];
+                        duplicateInputsAndAttributes = [
+                            ...ignoreDuplicated,
+                            ...notIgnoredDuplicates,
+                        ];
+                    }
+                }
                 const filteredOutputs = allowTwoWayDataBinding
                     ? outputs.filter((output) => {
                         return !inputs.some((input) => input.sourceSpan.start === output.sourceSpan.start &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/eslint-plugin-template",
-  "version": "16.0.4-alpha.9+6d26c59",
+  "version": "16.1.0",
   "description": "ESLint plugin for Angular Templates",
   "license": "MIT",
   "main": "dist/index.js",
@@ -17,11 +17,11 @@
     "LICENSE"
   ],
   "dependencies": {
-    "@angular-eslint/bundled-angular-compiler": "16.0.4-alpha.9+6d26c59",
-    "@angular-eslint/utils": "16.0.4-alpha.9+6d26c59",
-    "@typescript-eslint/type-utils": "5.59.8",
-    "@typescript-eslint/utils": "5.59.8",
-    "aria-query": "5.1.3",
+    "@angular-eslint/bundled-angular-compiler": "16.1.0",
+    "@angular-eslint/utils": "16.1.0",
+    "@typescript-eslint/type-utils": "5.62.0",
+    "@typescript-eslint/utils": "5.62.0",
+    "aria-query": "5.3.0",
     "axobject-query": "3.1.1"
   },
   "devDependencies": {
@@ -31,5 +31,5 @@
     "eslint": "^7.20.0 || ^8.0.0",
     "typescript": "*"
   },
-  "gitHead": "6d26c590e4b15e0b28a6ff7467560537e2b9b92d"
+  "gitHead": "dce6381cafcbe2de109d0a53b4c9c3e3bf1ae569"
 }