Edit Course Offering

- fixed an issue that was causing a cross-listed offering to loose all coordinators when a not controlling course was edited
realkjw · Jul 12, 2016 · f593599 · f593599
1 parent 4a3119d
commit f593599
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/JavaSource/org/unitime/timetable/action/CourseOfferingEditAction.java b/JavaSource/org/unitime/timetable/action/CourseOfferingEditAction.java
@@ -260,7 +260,7 @@ private void doUpdate(HttpServletRequest request, CourseOfferingEditForm frm) th
 	        if (!limitedEdit || updateNote)
 	        	co.setScheduleBookNote(note);
 
-	        if (!limitedEdit || updateCoordinators || co.isIsControl().booleanValue()) {
+	        if ((!limitedEdit || updateCoordinators) && co.isIsControl().booleanValue()) {
 		        if (io.getCoordinators() == null) io.setCoordinators(new HashSet<DepartmentalInstructor>());
 		        for (Iterator<DepartmentalInstructor> i = io.getCoordinators().iterator(); i.hasNext(); ) {
 		            DepartmentalInstructor instructor = i.next();

diff --git a/WebContent/help/Release-Notes.xml b/WebContent/help/Release-Notes.xml
@@ -42,6 +42,12 @@
 				<line>Avoid NullPointerException when there is no date pattern (i.e., there is no default date pattern defined for the term).</line>
 			</description>
 		</item>
+		<item>
+			<name>Edit Course Offering</name>
+			<description>
+				<line>Fixed an issue that was causing a cross-listed offering to loose all coordinators when a not controlling course was edited.</line>
+			</description>
+		</item>
 	</category>
 	<category>
 		<title>Student Scheduling</title>