Do not generate the users service if it already exists (#334)

feathersjs · Aug 25, 2018 · bb92f0d · bb92f0d
1 parent 6ebe00c
commit bb92f0d
Show file tree

Hide file tree

Showing 2 changed files with 50 additions and 85 deletions.
diff --git a/packages/generator-feathers/generators/authentication/index.js b/packages/generator-feathers/generators/authentication/index.js
@@ -165,15 +165,17 @@ module.exports = class AuthGenerator extends Generator {
         dependencies.push(`@feathersjs/authentication-${strategy}`);
       }
     });
-
-    // Create the users service
-    this.composeWith(require.resolve('../service'), {
-      props: {
-        name: context.entity,
-        path: `/${context.kebabEntity}`,
-        authentication: context
-      }
-    });
+
+    if(!this.fs.exists(this.destinationPath(this.libDirectory, 'services', context.kebabEntity, `${context.kebabEntity}.service.js`))) {
+      // Create the users service
+      this.composeWith(require.resolve('../service'), {
+        props: {
+          name: context.entity,
+          path: `/${context.kebabEntity}`,
+          authentication: context
+        }
+      });
+    }
 
     // If the file doesn't exist yet, add it to the app.js
     if (!this.fs.exists(this.destinationPath(this.libDirectory, 'authentication.js'))) {

diff --git a/packages/generator-feathers/package-lock.json b/packages/generator-feathers/package-lock.json