From 9d49ba5f0aee5adebaafaee917039215016dfb54 Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 14:08:23 +0300
Subject: [PATCH 1/9] #72 - cleand up main menu & added ID to routes

---
 templates/js/app.js                        |  4 ++--
 templates/js/controllers/mainController.js | 15 +++++----------
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/templates/js/app.js b/templates/js/app.js
index 99a5e63..bb9af14 100644
--- a/templates/js/app.js
+++ b/templates/js/app.js
@@ -46,7 +46,7 @@ app.config(['$routeProvider', '$locationProvider',
 				templateUrl: 'templates/views/tasks.html',
 				controller: 'tasksController'
 			})
-			.when('/myClasses', {
+			.when('/myClasses/:campusId', {
 				templateUrl: 'templates/views/myClasses.html',
 				controller: 'myClassesController'
 			})
@@ -54,7 +54,7 @@ app.config(['$routeProvider', '$locationProvider',
 				templateUrl: 'templates/views/newTask.html',
 				controller: 'newTasksController'
 			})
-			.when('/projects', {
+			.when('/projects/:classId', {
 				templateUrl: 'templates/views/projects.html',
 				controller: 'projectsController'
 			})
diff --git a/templates/js/controllers/mainController.js b/templates/js/controllers/mainController.js
index 5cfa78f..b9ac448 100644
--- a/templates/js/controllers/mainController.js
+++ b/templates/js/controllers/mainController.js
@@ -22,21 +22,16 @@ angular.module('SeHub')
 					"icon": "fa fa-tachometer",
 					"style": "selected",
 					"route": "/home"
-				}, {
-					"title": "My Campuses",
-					"icon": "fa fa-university",
+				},  {
+					"title": "Courses",
+					"icon": "fa fa-graduation-cap",
 					"style": "",
 					"route": "/campuses"
 				}, {
-					"title": "My Classes",
-					"icon": "fa fa-graduation-cap",
-					"style": "",
-					"route": "/myClasses"
-				}, {
-					"title": "My Projects",
+					"title": "Projects",
 					"icon": "fa fa-cube",
 					"style": "",
-					"route": "/projects"
+					"route": "/myClasses"
 				}, {
 					"title": "Tasks",
 					"icon": "fa fa-clipboard",

From 42210f73d577d9c443e3c72bb4b51e118b5c8016 Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 14:52:33 +0300
Subject: [PATCH 2/9] API Docs - index UI

---
 templates/API_Doc/api_doc_index.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/templates/API_Doc/api_doc_index.html b/templates/API_Doc/api_doc_index.html
index a1077de..e1b4ae1 100644
--- a/templates/API_Doc/api_doc_index.html
+++ b/templates/API_Doc/api_doc_index.html
@@ -54,8 +54,8 @@
             SE-Hub Docs Index
         </h1>
 
-
-    <div>
+<center>  
+    <div style="width: 40%">
       <ul class="collection">
         <li class="collection-item dismissable"><div>Users Related API<a href="/api/users/help" class="secondary-content"><i class="mdi-content-send"></i></a></div></li>
         <li class="collection-item dismissable"><div>Campuses Related API<a href="/api/campuses/help" class="secondary-content"><i class="mdi-content-send"></i></a></div></li>
@@ -66,7 +66,7 @@
         <li class="collection-item dismissable"><div>Miscellaneous Methods<a href="/api/help/misc" class="secondary-content"><i class="mdi-content-send"></i></a></div></li>
       </ul>
     </div>
-
+</center>
      <footer class="page-footer">
           <div class="container">
             <div class="row">

From 9ddc035443d8422cb66fd9b9b3046d91030f1e8e Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 14:53:40 +0300
Subject: [PATCH 3/9] apiServices - Added token Param in to getCamusesByUser

---
 templates/js/services/apiService.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/js/services/apiService.js b/templates/js/services/apiService.js
index e619186..7948658 100644
--- a/templates/js/services/apiService.js
+++ b/templates/js/services/apiService.js
@@ -99,8 +99,8 @@ service.factory('apiService', ['$http', function($http) {
 			};
 			return $http(req);
 		},
-		getCampusesByUser: function(){
-			var url =  (DEBUG ? "http://localhost:8080" : "http://se-hub.appspot.com") + "/api/courses/getCampusesByUser/" + token;
+		getCampusesByUser: function(token){
+			var url =  (DEBUG ? "http://localhost:8080" : "http://se-hub.appspot.com") + "/api/campuses/getCampusesByUser/" + token;
 			req = {
 				method : "GET",
 				url : url

From 0ccc8ba1dd9e809b9c622ccdeb4e8842ac42e84b Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 14:54:16 +0300
Subject: [PATCH 4/9] API Hot Fix. fixed syntax Error

---
 SE_API/CampusRoutes.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/SE_API/CampusRoutes.py b/SE_API/CampusRoutes.py
index c49fb3e..594975c 100644
--- a/SE_API/CampusRoutes.py
+++ b/SE_API/CampusRoutes.py
@@ -192,7 +192,7 @@ def getCampusesByUser(token):
         return bad_request("Bad user Token")
 
     arr = []
-    for i in user['campuses_id_list']:
+    for i in user.campuses_id_list:
         campus = Campus.get_by_id(int(i))
         arr.append(dict(json.loads(campus.to_JSON())))
 

From 0e37e51b4df6e07ee965c1ce767af358856c53cb Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 14:55:08 +0300
Subject: [PATCH 5/9] API: Added Get User By ID

---
 SE_API/UserRoutes.py       | 60 ++++++++++++++++++++++++++++++++++++++
 SE_API/Validation_Utils.py |  4 +++
 2 files changed, 64 insertions(+)

diff --git a/SE_API/UserRoutes.py b/SE_API/UserRoutes.py
index a448ae6..54f19a8 100644
--- a/SE_API/UserRoutes.py
+++ b/SE_API/UserRoutes.py
@@ -268,6 +268,66 @@ def getUserByToken(token):
                         status=200,
                         mimetype="application/json")  # Real response!
 
+    return no_content("No User Found")
+
+@user_routes.route('/api/users/getUserById/', defaults={'token': None, 'id': None})
+@user_routes.route('/api/users/getUserById/<string:token>/<string:id>', methods=["GET"])
+@auto.doc()
+def getUserById(token, id):
+    """
+    <span class="card-title">>This Call will return a user by a given UserId</span>
+    <br>
+    <b>Route Parameters</b><br>
+        - seToken: 'seToken'
+    <br>
+    <br>
+    <b>Payload</b><br>
+     - NONE
+    <br>
+    <br>
+    <b>Response</b>
+    <br>
+    200 - JSON Example:<br>
+    <code>
+        {<br>
+        'username': 'DarkLord',<br>
+        'name': 'Darth Vader',<br>
+        'email': 'darkLord@death.planet,<br>
+        'isLecturer': 'True',<br>
+        'seToken': 'xxxxxx-xxxxx-xxxxx-xxxxxx',<br>
+        'avatar_url': 'http://location.git.com/somthing'<br>
+        'isFirstLogin': False,<br>
+        'campuses_id_list': [{<br>
+                            'master_user_id': 111,<br>
+                            'id': 5629499534213120,<br>
+                            'email_ending': "@post.jce.ac.il",<br>
+                            'avatar_url': "https://yt3.ggpht.com/--ZkWxybWGOM/AAAAAAAAAAI/AAAAAAAAAAA/_nAICC_kzzI/s88-c-k-no/photo.jpg",<br>
+                            'title': "JCE"
+            }],<br>
+        'courses_id_list': ['a','b','c'],<br>
+        'id': 234253523<br>
+        }<br>
+    </code>
+    <br>
+    403 - No User Found
+    """
+    if token is None or id is None:
+        return no_content("No Token/ID, No User Found")
+
+    if get_user_by_token(token) is None:
+        return forbidden('Invalid Token')
+
+    u = get_user_by_id(int(id))
+    if u is None:
+        return no_content('No user Found')
+
+    for index, c in enumerate(u.campuses_id_list):
+        c = json.loads(Campus.get_by_id(int(c)).to_JSON())
+        u.campuses_id_list[index] = c
+
+        return Response(response=u.to_JSON(),
+                        status=200,
+                        mimetype="application/json")  # Real response!
 
     return no_content("No User Found")
 
diff --git a/SE_API/Validation_Utils.py b/SE_API/Validation_Utils.py
index 56da380..b6c0937 100644
--- a/SE_API/Validation_Utils.py
+++ b/SE_API/Validation_Utils.py
@@ -13,6 +13,10 @@ def get_user_by_token(token):
         return u
     return None
 
+def get_user_by_id(id):
+    u = User.get_by_id(id)
+    return u
+
 def get_campus_by_campusName(campusName):
     query = Campus.all()
     query.filter("title = ", campusName)

From 43de4f9b803a774b7ceec2103e9274c2287d870b Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 15:06:59 +0300
Subject: [PATCH 6/9] apiServices: Added GetUserById & get GetCampusByUserId

---
 templates/js/services/apiService.js | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/templates/js/services/apiService.js b/templates/js/services/apiService.js
index 7948658..9105c66 100644
--- a/templates/js/services/apiService.js
+++ b/templates/js/services/apiService.js
@@ -123,8 +123,26 @@ service.factory('apiService', ['$http', function($http) {
 				method : "GET",
 				url : url
 
+			};
+			return $http(req);
+		},
+		getUserById: function(token, id){
+			var url =  (DEBUG ? "http://localhost:8080" : "http://se-hub.appspot.com") + "/api/users/getUserById/" + token + "/" + id;
+			req = {
+				method : "GET",
+				url : url
+			};
+			return $http(req);
+		},
+		getCampusesByUserId: function(token, id){
+			var url =  (DEBUG ? "http://localhost:8080" : "http://se-hub.appspot.com") + "/api/campuses/getCampusesByUserID/" + token + "/" + id;
+			req = {
+				method : "GET",
+				url : url
+
 			};
 			return $http(req);
 		}
+		
 	};
 }]);
\ No newline at end of file

From 1c32ed9e56fc79eeb2f4da53ea265303ee58ac34 Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 15:07:26 +0300
Subject: [PATCH 7/9] API: Added /api/campuses/getCampusesByUserID/

---
 SE_API/CampusRoutes.py | 58 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 58 insertions(+)

diff --git a/SE_API/CampusRoutes.py b/SE_API/CampusRoutes.py
index 594975c..02d460e 100644
--- a/SE_API/CampusRoutes.py
+++ b/SE_API/CampusRoutes.py
@@ -205,6 +205,64 @@ def getCampusesByUser(token):
                         status=200,
                         mimetype="application/json")
 
+@campus_routes.route('/api/campuses/getCampusesByUserID/', defaults={'token': None, 'id': None})
+@campus_routes.route('/api/campuses/getCampusesByUserID/<string:token>/<string:id>', methods=['GET'])
+@auto.doc()
+def getCampusesByUserID(token, id):
+    """
+    <span class="card-title">This Call will return an array of all Campuses of a certain User By ID</span>
+    <br>
+    <b>Route Parameters</b><br>
+        - token: 'seToken' of requesting user
+        - The ID of <b>Wanted</b> User Campuses
+    <br>
+    <br>
+    <b>Payload</b><br>
+     - NONE <br>
+    <br>
+    <br>
+    <b>Response</b>
+    <br>
+    200 - JSON Array, Example:<br>
+    [<br>
+    {
+                'title': 'JCE',<br>
+                'email_ending': '@post.jce.ac.il',<br>
+                'master_user_id': 123453433341, (User that created the campus)<br>
+                'avatar_url': 'http://some.domain.com/imagefile.jpg',<br>
+                'id' : 1234567890<br>
+    },<br>
+    ....<br>
+    {<br>
+    ...<br>
+    }req<br>
+    ]<br>
+    <br>
+    403 - Invalid Token<br>
+    """
+
+    user = get_user_by_token(token)
+    if user is None:
+        return forbidden("Invalid Token")
+
+    user = get_user_by_id(int(id))
+    if user is None:
+        return no_content("No User")
+
+    arr = []
+    for i in user.campuses_id_list:
+        campus = Campus.get_by_id(int(i))
+        arr.append(dict(json.loads(campus.to_JSON())))
+
+    if len(arr) != 0:
+        return Response(response=json.dumps(arr),
+                        status=200,
+                        mimetype="application/json")
+    else:
+        return Response(response=[],
+                        status=200,
+                        mimetype="application/json")
+
 #----------------------------------------------------------
 #                     DELETE
 #----------------------------------------------------------

From cc3d65037df7b9707984f35e166151b14f046c66 Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 15:13:26 +0300
Subject: [PATCH 8/9] First Step Of Independed Profile. ID oriented. Needs
 Classes List

---
 templates/js/controllers/profileController.js | 52 ++++++++++---------
 templates/views/profile.html                  |  6 ++-
 2 files changed, 31 insertions(+), 27 deletions(-)

diff --git a/templates/js/controllers/profileController.js b/templates/js/controllers/profileController.js
index 40ce310..8c5f872 100644
--- a/templates/js/controllers/profileController.js
+++ b/templates/js/controllers/profileController.js
@@ -1,31 +1,33 @@
 angular.module('SeHub')
 	.controller('profileController', ['$scope', '$rootScope', 'dataService', 'apiService',
-										'$cookies', '$location', '$routeParams',
+		'$cookies', '$location', '$routeParams',
 		function($scope, $rootScope, dataService, apiService, $cookies, $location, $routeParams) {
 
 			var token = $cookies['com.sehub.www'];
-
+			var id = $routeParams.id;
 			$scope.loadingData = true;
 			$scope.isInRegisterMode = false;
 
-			$scope.title = "Profile " + $routeParams.id; 
+			$scope.title = "Profile";
 
-			apiService.getUserByToken(token).success(function(data) {
+			apiService.getUserById(token, id).success(function(data) {
 				if (data.message == 'No User Found') {
 					console.error("No User Found!");
 				}
-				$scope.loadingData = false;
 				$scope.user = data;
-		
-				console.log(data);
-				if ($scope.user.isFirstLogin) {
-					$scope.menuObj = {};
-					$scope.isInRegisterMode = true;
+				apiService.getCampusesByUserId(token, id).success(function(data) {
+					$scope.campuses = data;
+					console.log(data);
+
 					$scope.loadingData = false;
-					$location.path('/register')
-				}
+				}).error(function(err) {
+					console.error(err);
+				});
+
 			});
 
+
+
 			$scope.isEditMode = false;
 			$scope.profileMode = "Edit Profile";
 			$scope.profileModeIcon = "fa fa-pencil";
@@ -68,17 +70,17 @@ angular.module('SeHub')
 				"campus_avatar": "https://yt3.ggpht.com/--ZkWxybWGOM/AAAAAAAAAAI/AAAAAAAAAAA/_nAICC_kzzI/s88-c-k-no/photo.jpg"
 			}];
 
-			$scope.campuses = [{
-				'title': 'JCE',
-				'email_ending': '@post.jce.ac.il',
-				'master_user_id': 123453433341,
-				'avatar_url': 'https://yt3.ggpht.com/--ZkWxybWGOM/AAAAAAAAAAI/AAAAAAAAAAA/_nAICC_kzzI/s88-c-k-no/photo.jpg'
-			}, {
-				'title': 'Stanford',
-				'email_ending': '@post.jce.ac.il',
-				'master_user_id': 123453433341,
-				'avatar_url': 'https://encrypted-tbn1.gstatic.com/images?q=tbn:ANd9GcR9M4uQgaJP1zyiCGw-dK31hU8buWqeuOi9vTXBd4Y8hQcFTZqA'
-			}];
+			// $scope.campuses = [{
+			// 	'title': 'JCE',
+			// 	'email_ending': '@post.jce.ac.il',
+			// 	'master_user_id': 123453433341,
+			// 	'avatar_url': 'https://yt3.ggpht.com/--ZkWxybWGOM/AAAAAAAAAAI/AAAAAAAAAAA/_nAICC_kzzI/s88-c-k-no/photo.jpg'
+			// }, {
+			// 	'title': 'Stanford',
+			// 	'email_ending': '@post.jce.ac.il',
+			// 	'master_user_id': 123453433341,
+			// 	'avatar_url': 'https://encrypted-tbn1.gstatic.com/images?q=tbn:ANd9GcR9M4uQgaJP1zyiCGw-dK31hU8buWqeuOi9vTXBd4Y8hQcFTZqA'
+			// }];
 
 
 			$scope.labels = ['Commits', 'Issues Assigned', 'Messages', 'Open Tasks'];
@@ -89,8 +91,8 @@ angular.module('SeHub')
 				[28, 48, 40, 3]
 			];
 
-			$scope.isUser = function(){
-				return $scope.user.id.toString() === $routeParams.id;
+			$scope.isUser = function() {
+				return $scope.$parent.user.id.toString() /*The Actual User*/ === $routeParams.id /*The Profile User*/;
 			}
 
 		}
diff --git a/templates/views/profile.html b/templates/views/profile.html
index 2b2a26e..986f1d9 100644
--- a/templates/views/profile.html
+++ b/templates/views/profile.html
@@ -1,6 +1,8 @@
 <div>
-	<div class="loader" ng-if="loadingData">
+	<div class="loader" ng-if="loadingData" layout="row" layout-align="center">
+	
 		<md-progress-circular md-mode="indeterminate"></md-progress-circular>
+
 	</div>
 	<md-content layout-padding>
 		<md-card layout-padding>
@@ -96,7 +98,7 @@
                             		<img alt="{{ person.name }}" ng-src="{{ campus.avatar_url }}" class="md-avatar" />
 	                        	</div>
                             	<div class="port_spacer">
-					    			{{ campus.title }} - 
+					    			<strong>{{ campus.title }}</strong> {{campus.email_ending}} 
                             	</div>
 					    		<div class="spacer"></div>
 					    		<div>

From 1d5cbb18cd050ffa8318ee122343234f5c3fd3c4 Mon Sep 17 00:00:00 2001
From: Sagi Dayan <sagidayan@gmail.com>
Date: Fri, 26 Jun 2015 15:34:07 +0300
Subject: [PATCH 9/9] Profile - Edit And Delete Buttons Visible Only If
 Permitted

---
 templates/js/controllers/profileController.js | 4 +---
 templates/views/profile.html                  | 6 +++---
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/templates/js/controllers/profileController.js b/templates/js/controllers/profileController.js
index 8c5f872..80bfa6a 100644
--- a/templates/js/controllers/profileController.js
+++ b/templates/js/controllers/profileController.js
@@ -91,9 +91,7 @@ angular.module('SeHub')
 				[28, 48, 40, 3]
 			];
 
-			$scope.isUser = function() {
-				return $scope.$parent.user.id.toString() /*The Actual User*/ === $routeParams.id /*The Profile User*/;
-			}
+			$scope.isUser = $scope.$parent.user.id.toString() /*The Actual User*/ === $routeParams.id /*The Profile User*/;
 
 		}
 	]);
\ No newline at end of file
diff --git a/templates/views/profile.html b/templates/views/profile.html
index 986f1d9..d9083ac 100644
--- a/templates/views/profile.html
+++ b/templates/views/profile.html
@@ -17,7 +17,7 @@
 								<img ng-src="{{user.avatar_url}}" alt="" style="width:20%">
 							</div>
 							<div flex="70%">
-								<md-button class="md-raised" ng-click="changeProfileMode()" ng-if="isUser()">
+								<md-button class="md-raised" ng-click="changeProfileMode()" ng-if="isUser">
 									<i ng-class="profileModeIcon"></i> {{profileMode}}
 								</md-button>
 							</div>
@@ -80,7 +80,7 @@
                             	</div>
 					    		<div class="spacer"></div>
 					    		<div>
-						    		<md-button class="">
+						    		<md-button class="" ng-if="isUser">
 						    		X
 						    		</md-button>
 					    		</div>
@@ -102,7 +102,7 @@
                             	</div>
 					    		<div class="spacer"></div>
 					    		<div>
-						    		<md-button class="">
+						    		<md-button class="" ng-if="isUser">
 						    		X
 						    		</md-button>
 					    		</div>