Changeset 387 for trunk/grails-app/controllers

Timestamp:

Feb 18, 2010, 8:10:41 PM (15 years ago)

Author:

gav

Message:

Capitalise asset and sub items during creation and update.
Small correction to alt menu.

Location:

trunk/grails-app/controllers

Files:

• AssetDetailedController.groovy (modified) (4 diffs)
• AssetSubItemDetailedController.groovy (modified) (4 diffs)

trunk/grails-app/controllers/AssetDetailedController.groovy

@@ -1 +1 @@
 import org.codehaus.groovy.grails.plugins.springsecurity.Secured
 import org.codehaus.groovy.grails.commons.ConfigurationHolder
+import org.apache.commons.lang.WordUtils
 
 @Secured(['ROLE_AppAdmin', 'ROLE_Manager', 'ROLE_AssetManager'])
@@ -288 +289 @@
             assetInstance.properties = params
 
+            use(WordUtils) {
+                assetInstance.name = assetInstance.name.capitalize()
+                assetInstance.description = assetInstance.description.capitalize()
+            }
+
             assetInstance.setAssetSubItemsFromCheckBoxList(params.assetSubItems)
 
             if(!assetInstance.hasErrors() && assetInstance.save(flush: true)) {
-                flash.message = "Asset ${params.id} updated"
+                flash.message = "Asset '${assetInstance.name}' updated"
                 redirect(action:show,id:assetInstance.id)
             }
@@ -335 +341 @@
     def save = {
         def result = assetService.save(params)
+
+        if(!result.error) {
+            flash.message = g.message(code: "default.create.success", args: ["Asset",  "'${result.assetInstance.name}'"])
+            redirect(action:show, id: result.assetInstance.id)
+            return
+        }
+
+        render(view:'create', model:[assetInstance: result.assetInstance])
+    }
+
+    /**
+    * Copy save action.
+    */
+    def saveCopy = {
+        def result = assetService.saveCopy(params)
 
         if(!result.error) {
@@ -342 +363 @@
         }
 
-        render(view:'create', model:[assetInstance: result.assetInstance])
-    }
-
-    /**
-    * Copy save action.
-    */
-    def saveCopy = {
-        def result = assetService.saveCopy(params)
-
-        if(!result.error) {
-            flash.message = g.message(code: "default.create.success", args: ["Asset", result.assetInstance.id])
-            redirect(action:show, id: result.assetInstance.id)
-            return
-        }
-
         if(result.error.code == "default.not.found") {
             flash.message = g.message(code: result.error.code, args: ["Asset", params.assetToCopy?.id])

trunk/grails-app/controllers/AssetSubItemDetailedController.groovy

@@ -1 +1 @@
 import org.codehaus.groovy.grails.plugins.springsecurity.Secured
 import org.codehaus.groovy.grails.commons.ConfigurationHolder
+import org.apache.commons.lang.WordUtils
 
 @Secured(['ROLE_AppAdmin', 'ROLE_Manager', 'ROLE_AssetManager'])
@@ -165 +166 @@
             assetSubItemInstance.properties = params
 
+            use(WordUtils) {
+                assetSubItemInstance.name = assetSubItemInstance.name.capitalize()
+                assetSubItemInstance.description = assetSubItemInstance.description.capitalize()
+            }
+
             if(assetSubItemInstance.assets) {
                 assetSubItemInstance.parentItem = null
@@ -170 +176 @@
 
             if(!assetSubItemInstance.hasErrors() && assetSubItemInstance.save(flush: true)) {
-                flash.message = "AssetSubItem ${params.id} updated"
+                flash.message = "Sub Item '${assetSubItemInstance.name}' updated"
 
                 if(assetSubItemInstance.assets)
@@ -217 +223 @@
         if(!result.error) {
             // Success.
-            flash.message = g.message(code: "default.create.success", args: ["AssetSubItem", result.assetSubItemInstance.name])
+            flash.message = g.message(code: "default.create.success", args: ["Sub Item", "'${result.assetSubItemInstance.name}'"])
 
             /// @todo: This just grabs the first parent in an unsorted list and needs improving.