Changeset 352 for trunk/grails-app/controllers

Timestamp:

Feb 12, 2010, 12:14:23 PM (15 years ago)

Author:

gav

Message:

Added some checks to prevent recurrsion loops in Asset tree.

File:

• trunk/grails-app/controllers/AssetSubItemDetailedController.groovy (modified) (2 diffs)

trunk/grails-app/controllers/AssetSubItemDetailedController.groovy

@@ -116 +116 @@
             flash.message = "AssetSubItem not found with id ${params.id}"
             redirect(action: "search")
-        }
-        else {
-            return [ assetSubItemInstance : assetSubItemInstance ]
-        }
+            return
+        }
+
+        def possibleParentItems = assetSubItemService.possibleParentItems(assetSubItemInstance)
+
+        return [ assetSubItemInstance : assetSubItemInstance, possibleParentItems: possibleParentItems ]
     }
 
@@ -137 +139 @@
             assetSubItemInstance.properties = params
 
-//             if(!assetSubItemInstance.asset && !assetSubItemInstance.parentItem) {
-//                 flash.message = "Either an asset or parentItem is required."
-//                 render(view:'edit',model:[assetSubItemInstance:assetSubItemInstance])
-//                 return
-//             }
-// 
-//             if(assetSubItemInstance.asset && assetSubItemInstance.parentItem) {
-//             flash.message = "Both an asset and parentItem may not be specified at the same time."
-//                 render(view:'edit',model:[parentItem:parentItem])
-//                 return
-//             }
+            if(assetSubItemInstance.assets) {
+                assetSubItemInstance.parentItem = null
+            }
 
             if(!assetSubItemInstance.hasErrors() && assetSubItemInstance.save(flush: true)) {