SAK-29616 - return the newly created assignment id in the addAssignme…

…nt method
jayrp11 · Jul 1, 2015 · ea95557 · ea95557
1 parent 4e79a73
commit ea95557
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...book-service/api/src/java/org/sakaiproject/service/gradebook/shared/GradebookService.java b/...book-service/api/src/java/org/sakaiproject/service/gradebook/shared/GradebookService.java
@@ -360,8 +360,9 @@ public boolean isAssignmentDefined(String gradebookUid, String assignmentTitle)
 	 * Create a new Gradebook-managed assignment.
 	 * 
 	 * @param assignmentDefinition
+	 * @return the id of the newly created assignment
 	 */
-	public void addAssignment(String gradebookUid, Assignment assignmentDefinition);
+	public Long addAssignment(String gradebookUid, Assignment assignmentDefinition);
 
 	/**
 	 * Modify the definition of an existing Gradebook-managed assignment.

diff --git a/...ice/impl/src/java/org/sakaiproject/component/gradebook/GradebookServiceHibernateImpl.java b/...ice/impl/src/java/org/sakaiproject/component/gradebook/GradebookServiceHibernateImpl.java
@@ -596,7 +596,7 @@ public Object doInHibernate(Session session) throws HibernateException {
     }
 
 	@Override
-	public void addAssignment(String gradebookUid, org.sakaiproject.service.gradebook.shared.Assignment assignmentDefinition) {
+	public Long addAssignment(String gradebookUid, org.sakaiproject.service.gradebook.shared.Assignment assignmentDefinition) {
 		if (!getAuthz().isUserAbleToEditAssessments(gradebookUid)) {
 			log.error("AUTHORIZATION FAILURE: User " + getUserUid() + " in gradebook " + gradebookUid + " attempted to add an assignment");
 			throw new SecurityException("You do not have permission to perform this operation");
@@ -609,7 +609,7 @@ public void addAssignment(String gradebookUid, org.sakaiproject.service.gradeboo
         }
 
 		Gradebook gradebook = getGradebook(gradebookUid);
-		createAssignment(gradebook.getId(), assignmentDefinition.getName(), points, assignmentDefinition.getDueDate(), !assignmentDefinition.isCounted(), assignmentDefinition.isReleased(), assignmentDefinition.isExtraCredit());
+		return createAssignment(gradebook.getId(), assignmentDefinition.getName(), points, assignmentDefinition.getDueDate(), !assignmentDefinition.isCounted(), assignmentDefinition.isReleased(), assignmentDefinition.isExtraCredit());
 	}
 
 	@SuppressWarnings({ "unchecked", "rawtypes" })