Fixes googlearchive#13 - angularfire:route does not add controller or…

… view Fixes googlearchive#14 - auth required option does not seem to work
jabas06 · Dec 19, 2014 · 6498ec4 · 6498ec4
1 parent a5307b5
commit 6498ec4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/route/index.js b/route/index.js
@@ -28,7 +28,7 @@ var Generator = module.exports = function Generator(name, skipFiles) {
     this.foundWhenForRoute = true;
   }
 
-  if( !skipFiles ) {
+  if( skipFiles !== true ) {
     this.hookFor('angularfire:controller');
     this.hookFor('angularfire:view');
   }
@@ -67,7 +67,9 @@ Generator.prototype.rewriteAppJs = function () {
     ]
   };
 
-  var whenMethod = this.env.options.authRequired? 'whenAuthenticated' : 'when';
+
+  var whenMethod = this.env.options.authRequired || this.options['auth-required']? 'whenAuthenticated' : 'when';
+  console.log(whenMethod, this.options);
 
   if (coffee) {
     config.splicable.unshift("." + whenMethod + " '/" + this.uri + "',");