Prepare for role-based access control of states

meanjs · Jul 24, 2015 · e21805f · e21805f
1 parent 9fc88e6
commit e21805f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 10 deletions.
diff --git a/modules/articles/client/config/articles.client.routes.js b/modules/articles/client/config/articles.client.routes.js
@@ -18,7 +18,7 @@ angular.module('articles').config(['$stateProvider',
 			url: '/create',
 			templateUrl: 'modules/articles/views/create-article.client.view.html',
 			data: {
-				requiresLogin: true
+				forbiddenRoles: ['guest']
 			}
 		}).
 		state('articles.view', {
@@ -29,7 +29,7 @@ angular.module('articles').config(['$stateProvider',
 			url: '/:articleId/edit',
 			templateUrl: 'modules/articles/views/edit-article.client.view.html',
 			data: {
-				requiresLogin: true
+				forbiddenRoles: ['guest']
 			}
 		});
 	}

diff --git a/modules/core/client/app/init.js b/modules/core/client/app/init.js
@@ -13,13 +13,16 @@ angular.module(ApplicationConfiguration.applicationModuleName).config(['$locatio
 angular.module(ApplicationConfiguration.applicationModuleName).run(function($rootScope, $state, Authentication) {
     // Check authentication before changing state
     $rootScope.$on('$stateChangeStart', function(event, toState, toParams, fromState, fromParams) {
-        if (toState.data && toState.data.requiresLogin && Authentication.user === '') {
-            event.preventDefault();
-            $state.go('authentication.signin', {}, {
-                notify: false
-            }).then(function() {
-                $rootScope.$broadcast('$stateChangeSuccess', 'authentication.signin', {}, toState, toParams);
-            });
+        if (toState.data && toState.data.forbiddenRoles) {
+            // If access of guest user is forbidden:
+            if (toState.data.forbiddenRoles.indexOf('guest') !== -1 && Authentication.user === '') {
+                event.preventDefault();
+                $state.go('authentication.signin', {}, {
+                    notify: false
+                }).then(function() {
+                    $rootScope.$broadcast('$stateChangeSuccess', 'authentication.signin', {}, toState, toParams);
+                });
+            }
         }
     });
 });

diff --git a/modules/users/client/config/users.client.routes.js b/modules/users/client/config/users.client.routes.js
@@ -10,7 +10,7 @@ angular.module('users').config(['$stateProvider',
 				url: '/settings',
 				templateUrl: 'modules/users/views/settings/settings.client.view.html',
 				data: {
-					requiresLogin: true
+					forbiddenRoles: ['guest']
 				}
 			}).
 			state('settings.profile', {