remove skipProvide, comment with todo for module option

Author: jschwarty

packages/schematics/angular/service/index.ts

@@ -21,38 +21,38 @@ import {
 } from '@angular-devkit/schematics';
 import * as stringUtils from '../strings';
 
-import * as ts from 'typescript';
+// import * as ts from 'typescript';
 import 'rxjs/add/operator/merge';
-import {findModule, buildRelativePath} from '../utility/find-module';
-import {addProviderToModule} from '../utility/ast-utils';
-import {InsertChange} from '../utility/change';
+// import {findModule, buildRelativePath} from '../utility/find-module';
+// import {addProviderToModule} from '../utility/ast-utils';
+// import {InsertChange} from '../utility/change';
 
 function addProviderToNgModule(options: any): Rule {
   return (host: Tree) => {
-    if (options.skipProvide) {
+    if (!options.module) {
       return host;
     }
 
-    const modulePath = findModule(host, options.sourceDir + '/' + options.path);
-
-    const sourceText = host.read(modulePath) !.toString('utf-8');
-    const source = ts.createSourceFile(modulePath, sourceText, ts.ScriptTarget.Latest, true);
-
-    const servicePath = `/${options.sourceDir}/${options.path}/`
-      + (options.flat ? '' : stringUtils.dasherize(options.name) + '/')
-      + stringUtils.dasherize(options.name)
-      + '.service';
-    const relativePath = buildRelativePath(modulePath, servicePath);
-    const changes = addProviderToModule(source, modulePath,
-      stringUtils.classify(`${options.name}Service`),
-      relativePath);
-    const recorder = host.beginUpdate(modulePath);
-    for (const change of changes) {
-      if (change instanceof InsertChange) {
-        recorder.insertLeft(change.pos, change.toAdd);
-      }
-    }
-    host.commitUpdate(recorder);
+    // TODO: Add option support for module path
+    // let modulePath = '';
+    // const sourceText = host.read(modulePath) !.toString('utf-8');
+    // const source = ts.createSourceFile(modulePath, sourceText, ts.ScriptTarget.Latest, true);
+    //
+    // const servicePath = `/${options.sourceDir}/${options.path}/`
+    //   + (options.flat ? '' : stringUtils.dasherize(options.name) + '/')
+    //   + stringUtils.dasherize(options.name)
+    //   + '.service';
+    // const relativePath = buildRelativePath(modulePath, servicePath);
+    // const changes = addProviderToModule(source, modulePath,
+    //   stringUtils.classify(`${options.name}Service`),
+    //   relativePath);
+    // const recorder = host.beginUpdate(modulePath);
+    // for (const change of changes) {
+    //   if (change instanceof InsertChange) {
+    //     recorder.insertLeft(change.pos, change.toAdd);
+    //   }
+    // }
+    // host.commitUpdate(recorder);
 
     return host;
   };

packages/schematics/angular/service/schema.json

@@ -25,11 +25,6 @@
       "default": true,
       "description": "Specifies if a spec file is generated."
     },
-    "skipProvide": {
-      "type": "boolean",
-      "default": false,
-      "description": "Allows for skipping the module provide."
-    },
     "module": {
       "type": "string",
       "default": "",