Changeset 343 for trunk/grails-app/domain

Timestamp:

Feb 10, 2010, 2:49:13 AM (15 years ago)

Author:

gav

Message:

Improvements to CustomTagLib checkBoxList, sorting, dynamic displayFields and a small fix to the domain class set method.

Location:

trunk/grails-app/domain

Files:

• Asset.groovy (modified) (1 diff)
• Person.groovy (modified) (1 diff)

trunk/grails-app/domain/Asset.groovy

@@ -26 +26 @@
     }
 
-    //  This additional setter is used to convert the checkBoxList string
+    //  This additional setter is used to convert the checkBoxList string or string array
     //  of ids selected to the corresponding domain objects.
     public void setAssetSubItemsFromCheckBoxList(ids) {
         def idList = []
-        ids.each() {
-            if(it.isInteger())
-                idList << it.toInteger()
+        if(ids instanceof String) {
+                if(ids.isInteger())
+                    idList << ids.toInteger()
+        }
+        else {
+            ids.each() {
+                if(it.isInteger())
+                    idList << it.toInteger()
+            }
         }
         this.assetSubItems = idList.collect { AssetSubItem.get( it ) }

trunk/grails-app/domain/Person.groovy

@@ -55 +55 @@
     String toString() {"${this.firstName} ${this.lastName}"}
 
-    //  This additional setter is used to convert the checkBoxList string
+    //  This additional setter is used to convert the checkBoxList string or string array
     //  of ids selected to the corresponding domain objects.
     public void setPersonGroupsFromCheckBoxList(ids) {
         def idList = []
-        ids.each() {
-            if(it.isInteger())
-                idList << it.toInteger()
+        if(ids instanceof String) {
+                if(ids.isInteger())
+                    idList << ids.toInteger()
+        }
+        else {
+            ids.each() {
+                if(it.isInteger())
+                    idList << it.toInteger()
+            }
         }
         this.personGroups = idList.collect { PersonGroup.get( it ) }