Changeset 350 for trunk/grails-app/services

Timestamp:

Feb 10, 2010, 5:09:22 AM (15 years ago)

Author:

gav

Message:

Move Asset delete to a service method and configure cascade deletion checks.

Location:

trunk/grails-app/services

Files:

• AssetService.groovy (modified) (3 diffs)
• AssetSubItemService.groovy (modified) (1 diff)

trunk/grails-app/services/AssetService.groovy

@@ -2 +2 @@
 
     boolean transactional = false
+
+    def assetSubItemService
 
     /**
@@ -13 +15 @@
         }
     }
+
+    def delete(params) {
+        Asset.withTransaction { status ->
+            def result = [:]
+
+            def fail = { Map m ->
+                status.setRollbackOnly()
+                if(result.assetInstance && m.field)
+                    result.assetInstance.errors.rejectValue(m.field, m.code)
+                result.error = [ code: m.code, args: ["Asset", params.id] ]
+                return result
+            }
+
+            result.assetInstance = Asset.get(params.id)
+
+            if(!result.assetInstance)
+                return fail(code:"default.not.found")
+
+            if(result.assetInstance.maintenanceActions)
+                return fail(code:"maintenanceActions.still.associated")
+
+            // Remove orphan assetSubItems.
+            def assetSubItems = new ArrayList(result.assetInstance.assetSubItems) // avoid ConcurrentModificationException.
+            def r
+            for(assetSubItem in assetSubItems) {
+                result.assetInstance.removeFromAssetSubItems(assetSubItem)
+                if(!assetSubItem.assets && !assetSubItem.parentItem) {
+                    r = assetSubItemService.delete(id: assetSubItem.id)
+                    if(r.error) {
+                        fail(code:"asset.subItems.delete.failure")
+                        break
+                    }
+                }
+            }
+
+            if(result.error)
+                return result
+
+            try {
+                result.assetInstance.delete(flush:true)
+                return result //Success.
+            }
+            catch(org.springframework.dao.DataIntegrityViolationException e) {
+                return fail(code:"default.delete.failure")
+            }
+
+        } // end withTransaction
+    } // end delete()
 
     def create(params) {
@@ -164 +214 @@
             return result
 
-        } //end withTransaction
+        } // end withTransaction
     } // end saveCopySrvce
 

trunk/grails-app/services/AssetSubItemService.groovy

@@ -19 +19 @@
 
         if(result.assetSubItemInstance.maintenanceActions)
-            return fail(code:"assetSubItem.maintenanceActions.associated")
+            return fail(code:"maintenanceActions.still.associated")
 
         try {