feat(cli): change generate --prefix option type from Boolean to string

use ``ng g c foo --prefix='bar'`` or ``ng g d foo --prefix='bar'`` in order to use
another prefix than the one defined by default in angular-cli.json apps[0].prefix
tslint rules "directive-selector" and "component-selector" can accept any array of prefix,
and is therefore compatible with this approach
this is a temporary solution pending angular#3452 closure

Author: nlm-pro

packages/angular-cli/blueprints/component/index.js

@@ -44,10 +44,8 @@ module.exports = {
       defaultPrefix = this.project.ngConfig.apps[0].prefix;
     }
 
-    console.log('prefix option : ' + JSON.stringify(this.options.prefix));
-    var prefix = this.options.prefix || this.options.prefix === '' ? '' : defaultPrefix;
+    var prefix = (this.options.prefix === 'false' || this.options.prefix === '') ? '' : (this.options.prefix || defaultPrefix);
     prefix = prefix && `${prefix}-`;
-    console.log('prefix after : ' + JSON.stringify(prefix));
 
     this.selector = stringUtils.dasherize(prefix + parsedPath.name);
 

packages/angular-cli/blueprints/directive/index.js

@@ -39,7 +39,7 @@ module.exports = {
       defaultPrefix = this.project.ngConfig.apps[0].prefix;
     }
 
-    var prefix = this.options.prefix || this.options.prefix === '' ? '' : defaultPrefix;
+    var prefix = (this.options.prefix === 'false' || this.options.prefix === '') ? '' : (this.options.prefix || defaultPrefix);
     prefix = prefix && `${prefix}-`;
 
 

tests/acceptance/generate-component.spec.js

@@ -140,8 +140,8 @@ describe('Acceptance: ng generate component', function () {
       });
   });
 
-  it('mycomp --prefix=\'\' will not prefix selector', () => {
-    return ng(['generate', 'component', 'mycomp', '--prefix=\'\''])
+  it('mycomp --no-prefix will not prefix selector', () => {
+    return ng(['generate', 'component', 'mycomp', '--no-prefix'])
       .then(() => {
         var testPath = path.join(root, 'tmp', 'foo', 'src', 'app', 'mycomp', 'mycomp.component.ts');
         expect(existsSync(testPath)).to.equal(true);
@@ -150,8 +150,18 @@ describe('Acceptance: ng generate component', function () {
       });
   });
 
-  it('mycomp --prefix=\'test\' will prefix selector with \'test-\'', () => {
-    return ng(['generate', 'component', 'mycomp', '--prefix=\'test\''])
+  it('mycomp --prefix= will not prefix selector', () => {
+    return ng(['generate', 'component', 'mycomp', '--prefix='])
+      .then(() => {
+        var testPath = path.join(root, 'tmp', 'foo', 'src', 'app', 'mycomp', 'mycomp.component.ts');
+        expect(existsSync(testPath)).to.equal(true);
+        var contents = fs.readFileSync(testPath, 'utf8');
+        expect(contents.indexOf('selector: \'mycomp\'') === -1).to.equal(false);
+      });
+  });
+
+  it('mycomp --prefix=test will prefix selector with \'test-\'', () => {
+    return ng(['generate', 'component', 'mycomp', '--prefix=test'])
       .then(() => {
         var testPath = path.join(root, 'tmp', 'foo', 'src', 'app', 'mycomp', 'mycomp.component.ts');
         expect(existsSync(testPath)).to.equal(true);