SAK-12241 Editing section name doesn't update section description

git-svn-id: https://source.sakaiproject.org/svn/sections/trunk@53354 66ffb92e-73f9-0310-93c1-f5514f145a0a
fuead · Oct 11, 2008 · 1966b95 · 1966b95
1 parent ece2da2
commit 1966b95
Showing 1 changed file with 23 additions and 16 deletions.
diff --git a/...impl/sakai/impl/src/java/org/sakaiproject/component/section/sakai/SectionManagerImpl.java b/...impl/sakai/impl/src/java/org/sakaiproject/component/section/sakai/SectionManagerImpl.java
@@ -1271,24 +1271,31 @@ public void updateSection(String sectionUuid, String title, Integer maxEnrollmen
 		// Disallow if we're in an externally managed site
 		ensureInternallyManaged(section.getCourse().getUuid());
 
-		// Set the decorator's fields
-		section.setTitle(title);
-		section.setMaxEnrollments(maxEnrollments);
-		section.setMeetings(filterMeetings(meetings));
-
 		// Decorate the framework section
 		Group group = findGroup(sectionUuid);
-		section.decorateGroup(group);
-
-		// Save the site with its new section
-		try {
-			siteService().save(group.getContainingSite());
-			clearSite(group.getContainingSite().getId());
-			postEvent("section.update", sectionUuid);
-		} catch (IdUnusedException ide) {
-			log.error("Error saving site... could not find site for section " + group, ide);
-		} catch (PermissionException pe) {
-			log.error("Error saving site... permission denied for section " + group, pe);
+		Site site = group.getContainingSite(); 
+
+		if (site != null && group != null) {
+			// Set the decorator's fields
+			section.setTitle(title);
+			section.setDescription(site.getTitle() + ", " + title);
+			section.setMaxEnrollments(maxEnrollments);
+			section.setMeetings(filterMeetings(meetings));
+
+			section.decorateGroup(group);
+
+			// Save the site with its new section
+			try {
+				siteService().save(site);
+				clearSite(site.getId());
+				postEvent("section.update", sectionUuid);
+			} catch (IdUnusedException ide) {
+				log.error("Error saving site... could not find site for section " + group, ide);
+			} catch (PermissionException pe) {
+				log.error("Error saving site... permission denied for section " + group, pe);
+			}
+		} else {
+			log.error("Error updating section: could not find site or group for section " + sectionUuid);
 		}
 	}