diff --git a/SE_API/API.py b/SE_API/API.py index 340b5f8..c98b2fe 100644 --- a/SE_API/API.py +++ b/SE_API/API.py @@ -208,7 +208,8 @@ def login(): debug = True # Change In Production if debug: - counter = 300 + import random + counter = random.randrange(300,400) @auto.doc() @app.route('/debug/login') diff --git a/SE_API/ProjectRoutes.py b/SE_API/ProjectRoutes.py index fc5bf90..a2d1383 100644 --- a/SE_API/ProjectRoutes.py +++ b/SE_API/ProjectRoutes.py @@ -145,7 +145,9 @@ def joinProject(token, projectId): return bad_request("User is already member in Project") project.membersId.append(str(user.key().id())) + user.projects_id_list.append(str(project.key().id())) + db.put(user) db.put(project) db.save @@ -370,12 +372,22 @@ def deleteProject(token,projectId): if p is None: return bad_request("no such Project") - if p.master_id == user.key().id(): - db.delete(p) - db.save - return accepted("Project deleted") + if p.master_id != user.key().id(): + return forbidden("user is not owner of Project") + + #remove all users related to project + for uId in p.membersId: + user = User.get_by_id(uId) + if user is None: + return bad_request("trying to remove a user from project failed") + user.projects_id_list.remove(p.key().id()) + db.put(user) + + + db.delete(p) + db.save + return accepted("Project deleted") - return forbidden("user is not owner of Project") diff --git a/SE_API/TaskRoutes.py b/SE_API/TaskRoutes.py index ee3bee6..c587901 100644 --- a/SE_API/TaskRoutes.py +++ b/SE_API/TaskRoutes.py @@ -218,9 +218,9 @@ def getAllTasksByCourse(token, courseId): return no_content() -@task_routes.route('/api/tasks/getAllFutureTasks//', methods=["GET"]) +@task_routes.route('/api/tasks/getAllFutureCampusTasks//', methods=["GET"]) @auto.doc() -def getAllFutureTasks(token, courseId): +def getAllFutureCampusTasks(token, courseId): """ >This Call will return an array of all Future Tasks in a course, ordered by date
@@ -284,6 +284,73 @@ def getAllFutureTasks(token, courseId): else: return no_content() +@task_routes.route('/api/tasks/getAllFutureTasks/', methods=["GET"]) +@auto.doc() +def getAllFutureTasks(token): + """ + >This Call will return an array of all Future Tasks ordered by date +
+ Route Parameters
+ - SeToken: token
+
+
+ Payload
+ - NONE +
+
+ Response +
+ 200 - JSON Example:
+ + {
+ 'title' : 'Task1',
+ 'courseName' : 'advance Math',
+ 'description' : 'prepare by sunday',
+ 'dueDate' : { + 'year' : 2015, + 'month' : 12, + 'day' : 23 + }
+ 'isPersonal' : true,
+ 'task_id' : 589689456894
+ }
+ +
+ """ + + user = get_user_by_token + if user is None: + return bad_request("Bad User Token") + + arr = [] + + for courseId in user.courses_id_list: + query = Task.all() + + try: + query.filter("courseId = ", int(courseId)) + except Exception as e: + return bad_request("Bad id format") + + for t in query.run(): + taskDic =dict(json.loads(t.to_JSON())) + #add a key 'forSortDate' for sorting dates + taskTime = datetime.datetime(taskDic['dueDate']['year'], taskDic['dueDate']['month'], taskDic['dueDate']['day']) + if taskTime >= datetime.date.today(): + taskDic['forSortDate'] = taskTime + arr.append(taskDic) + + #sort array by date, and remove added key + arr = sorted(arr, key=itemgetter('forSortDate'), reverse=False) + for i in arr: + del i['forSortDate'] + + if len(arr) != 0: + return Response(response=json.dumps(arr), + status=200, + mimetype="application/json") + else: + return no_content() @task_routes.route('/api/tasks/getTaskComponents//', methods=["GET"]) @@ -353,7 +420,7 @@ def getTaskComponents(token, taskId): @task_routes.route('/api/tasks/getAllUserTasks/', methods=["GET"]) @auto.doc() -def getAllTasksByUser(token): +def getAllUserTasks(token): """ >This Call will return an array of all of the User's Tasks
@@ -491,70 +558,81 @@ def getAllTasksByUser(token): -# @task_routes.route('/api/tasks/getUserFullTasksById//', methods=["GET"]) -# @auto.doc() -# def getFullTasksById(token, taskId): -# """ -# >This Call will return an array of all components for a given task -#
-# Route Parameters
-# - SeToken: token
-# - taskId: 1234567890 -#
-#
-# Payload
-# - NONE -#
-#
-# Response -#
-# 200 - JSON Example:
-# -# [ -# {
-# 'taskId' : 7589454894, -# 'userId' : -1, -# 'type' : 'kindOfType', -# 'label' : 'kindOfLabel', -# 'isMandatory' : true, -# 'order' : 2 -# }
-# {
-# 'taskId' : 7589454894, -# 'userId' : yossi, -# 'type' : 'otherKindOfType', -# 'label' : 'otherKindOfLabel', -# 'isMandatory' : false, -# 'order' : 4 -# }
-# ] -# -#
-# """ -# user = get_user_by_token(token) -# if user is None: -# return bad_request("Bad User Token") -# -# task = Task.get_by_id(int(taskId)) -# if task is None: -# return bad_request("Bad Task id") -# -# taskCompQuery = TaskComponent.all() -# taskCompQuery.filter("taskId = ", task.key().id()) -# -# if task.isPersonal: -# taskCompQuery.filter("userId = ", user.key().id()) -# else: -# taskCompQuery.filter("userId = ", user.key().id()) -# -# if taskCompQuery.count() == 0: -# #create componenets and Score for user -# -# -# for i in taskCompQuery.run(): -# print i.to_JSON() -# -# return no_content() +@task_routes.route('/api/tasks/getUserTaskById//', methods=["GET"]) +@auto.doc() +def getUserTaskById(token, taskId): + """ + >This Call will return an array of all components for a given task +
+ Route Parameters
+ - SeToken: token
+ - taskId: 1234567890 +
+
+ Payload
+ - NONE +
+
+ Response +
+ 200 - JSON Example:
+ + [ + {
+ 'taskId' : 7589454894, + 'userId' : -1, + 'type' : 'kindOfType', + 'label' : 'kindOfLabel', + 'isMandatory' : true, + 'order' : 2 + }
+ {
+ 'taskId' : 7589454894, + 'userId' : yossi, + 'type' : 'otherKindOfType', + 'label' : 'otherKindOfLabel', + 'isMandatory' : false, + 'order' : 4 + }
+ ] + +
+ """ + user = get_user_by_token(token) + if user is None: + return bad_request("Bad User Token") + + task = Task.get_by_id(int(taskId)) + if task is None: + return bad_request("Bad Task id") + + + taskCompQuery = TaskComponent.all() + taskCompQuery.filter("taskId = ", task.key().id()) + + if task.isPersonal: + taskCompQuery.filter("userId = ", user.key().id()) + else: + taskCompQuery.filter("userId = ", user.key().id())#TODO: fix to project + + #check if never created a personalized task and if so, create it + if taskCompQuery.count() == 0: + taskCompQuery = TaskComponent.all().filter("taskId = ", task.key().id()).filter("userId = ", -1) + for tc in taskCompQuery.run(): + tcNew = TaskComponent(taskId=tc.taskId, userId=user.key().id(), type=tc.type, label=tc.label, isMandatory=tc.isMandatory, order=tc.order) + db.put(tcNew) + + grade = TaskGrade(grade=0, taskId=task.key().id(), userId=user.key().id()) + db.put(grade) + + + + + + + + db.save + return no_content() diff --git a/SE_API/UserRoutes.py b/SE_API/UserRoutes.py index 09de2df..1e8440c 100644 --- a/SE_API/UserRoutes.py +++ b/SE_API/UserRoutes.py @@ -322,16 +322,15 @@ def getUserById(token, 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(), + return Response(response=u.to_JSON(), status=200, mimetype="application/json") # Real response! - return no_content("No User Found") - @user_routes.route('/api/users/getUsersByCampus//', methods=["GET"]) @@ -579,6 +578,7 @@ def removeUserFromCampus(token, userId, campusId): if requestingUser.key().id() != userToRemove.key().id(): return forbidden("No permission to delete user") + #remove campus from user campus list try: userToRemove.campuses_id_list.remove(campusId) except Exception as e: @@ -646,6 +646,13 @@ def removeUserFromCourse(token, userId, courseId): except Exception as e: return bad_request("user is not listed to this course") + #remove user from all projects in course + projects = Project.all().filter("courseId =", course.key().id()) + for p in projects: + if p.key().id() in userToRemove.projects_id_list: + userToRemove.projects_id_list.remove(p.key().id()) + p.membersId.remove(userToRemove.key().id()) + db.put(p) db.put(userToRemove) diff --git a/models/Message.py b/models/Message.py index 67906f9..1a64564 100644 --- a/models/Message.py +++ b/models/Message.py @@ -8,7 +8,7 @@ from models.Course import Course class Message(db.Model): groupId = db.IntegerProperty(required=True) - message = db.StringProperty(required=True) + message = db.StringProperty(required=True, multiline=True) msgDate = db.DateTimeProperty(required=True) master_id = db.IntegerProperty(required=True) isProject = db.BooleanProperty(default=False) diff --git a/models/Task.py b/models/Task.py index 5449fa2..1dd422a 100644 --- a/models/Task.py +++ b/models/Task.py @@ -15,7 +15,7 @@ class Task(db.Model): data = {'title' : self.title, 'courseId' : self.courseId, 'description' : self.description, - 'dueDate' : { + 'dueDate' : { 'year': self.dueDate.year, 'month': self.dueDate.month, 'day': self.dueDate.day