Merge branch 'master' of https://github.com/mgechev/codelyzer

* 'master' of https://github.com/mgechev/codelyzer:
  fix document
  fix document
  fix: order problem with multiple prefixes for selector

README.md

@@ -51,7 +51,7 @@ You can easily use codelyzer with your custom setup:
 #### Installation
 
 ```shell
-npm i codelyzer@2.0.0-beta.4 tslint@4.0.0 typescript@2.0.9 @angular/core@2.4.0 @angular/compiler@2.4.0 rxjs@5.0.1 zone.js@0.7.2
+npm i codelyzer@~2.0.0 tslint@4.0.0 typescript@2.0.9 @angular/core@2.4.0 @angular/compiler@2.4.0 rxjs@5.0.1 zone.js@0.7.2
 ```
 
 A. Using codelyzer package in PATH
@@ -79,7 +79,6 @@ Create the following `tslint.json` file like:
     "templates-use-public": true,
     "no-access-missing-member": true,
     "invoke-injectable": true,
-    "invoke-injectable": true,
     "template-to-ng-template": true
   }
 }

src/selectorNameBase.ts

@@ -85,6 +85,11 @@ export abstract class SelectorRule extends Lint.Rules.AbstractRule {
   }
 
   private setPrefixValidator(prefix: any, name: string) {
+    if (!this.isMultiPrefix) {
+      prefix = (prefix||[]).sort((a: string, b: string) => {
+        return a.length < b.length ? 1 : -1;
+      });
+    }
     let prefixExpression: string = this.isMultiPrefix?prefix:(prefix||[]).join('|');
     this.prefixValidator = SelectorValidator.prefix(prefixExpression, name);
   }

test/componentSelectorRule.spec.ts

@@ -85,6 +85,15 @@ describe('component-selector-prefix', () => {
             assertSuccess('component-selector', source, ['element', ['sg','ng','mg'], 'kebab-case']);
         });
 
+        it('should succeed when set valid selector in @Component using multiple prefixes and some prefixes are substring of others', () => {
+            let source = `
+      @Component({
+        selector: 'abc-bar-foo'
+      })
+      class Test {}`;
+            assertSuccess('component-selector', source, ['element', ['ab','abc','mg'], 'kebab-case']);
+        });
+
         it('should succeed without prefix', () => {
             let source = `
       @Component({