Refactoring.

Author: Zemke

app/Http/routes.php

@@ -35,15 +35,12 @@
 Route::resource('/api/todo', 'TodoController');
 Route::resource('/api/user', 'UserController');
 
-//Route::controllers([
-//    'api/auth' => 'Auth\AuthController',
-//    'api/password' => 'Auth\PasswordController',
-//]);
-
 // Catch all undefined routes. Always gotta stay at the bottom since order of routes matters.
 Route::any('{undefinedRoute}', function ($undefinedRoute) {
     return view('layout');
 })->where('undefinedRoute', '([A-z\d-\/_.]+)?');
 
-Blade::setContentTags('<%', '%>');        // for variables and all things Blade
-Blade::setEscapedContentTags('<%%', '%%>');   // for escaped data
+// Using different syntax for Blade to avoid conflicts with Jade.
+// You are well-advised to go without any Blade at all.
+Blade::setContentTags('<%', '%>'); // For variables and all things Blade.
+Blade::setEscapedContentTags('<%%', '%%>'); // For escaped data.

resources/js/app.js

@@ -8,8 +8,6 @@ angular.module('todoApp', [
   'TodoController',
   'UserController',
   'UserService',
-  'AuthService',
   'TodoService',
-  'SessionService'
 ]);
 

resources/js/appRoutes.js

@@ -20,6 +20,7 @@ angular.module('appRoutes', []).config(['$routeProvider', '$locationProvider', '
       .otherwise({
         redirectTo: '/'
       });
+    
     $locationProvider.html5Mode(true);
 
     $httpProvider.interceptors.push(['$rootScope', '$q', '$localStorage',
@@ -32,11 +33,11 @@ angular.module('appRoutes', []).config(['$routeProvider', '$locationProvider', '
             }
             return config;
           },
-          response: function (response) {
-            if (response.status === 401) {
-              // handle the case where the user is not authenticated
+          response: function (res) {
+            if (res.status === 401) {
+              // Handle unauthenticated user.
             }
-            return response || $q.when(response);
+            return res || $q.when(res);
           }
         };
       }

resources/js/controllers/MainController.js

@@ -10,6 +10,12 @@ angular.module('MainController', []).controller('MainController', ['$scope', '$l
       return route === $location.path();
     };
 
+    /**
+     * Query the authenticated user by the Authorization token from the header.
+     *
+     * @param user {object} If provided, it won't query from database, but take this one.
+     * @returns {null}
+     */
     $scope.getAuthenticatedUser = function (user) {
       if (user) {
         $scope.authenticatedUser = user;
@@ -21,11 +27,9 @@ angular.module('MainController', []).controller('MainController', ['$scope', '$l
       }
 
       new User().$getByToken(function (user) {
-        console.log(user);
         $scope.authenticatedUser = user;
       }, function (err) {
-        //console.log(err);
-        //$scope.authenticatedUser = err;
+        console.log(err);
       });
     };
 

resources/js/controllers/TodoController.js

@@ -11,9 +11,11 @@ angular.module('TodoController', []).controller('TodoController', ['$scope', '$l
         console.log(err);
       });
     };
+
     $scope.find = function () {
       $scope.todos = Todo.query();
     };
+
     $scope.remove = function (todo) {
       todo.$remove(function (res) {
         if (res) {
@@ -27,12 +29,14 @@ angular.module('TodoController', []).controller('TodoController', ['$scope', '$l
         console.log(err);
       })
     };
+
     $scope.update = function (todo) {
       todo.$update(function (res) {
       }, function (err) {
         console.log(err);
       });
     };
+
     $scope.findOne = function () {
       var splitPath = $location.path().split('/');
       var todoId = splitPath[splitPath.length - 1];