On Feb 26, 8:22 am, Ezra Parker <[EMAIL PROTECTED]> wrote: > > Btw what are you extending delete for? Is it something that might be > > an improvement to the default behaviour of an image object? > > Well, I was mainly trying to work around the error I mentioned, but I > the course of doing so I decided to also add in some cross-checking > for deletion of images linked to my custom types. > > I think that the most useful way the dmImage delete() method could be > enhanced would be to add an optional argument for a list of array > tables to query for references to the image object. For built-in > support of the system types, the value could default to: > > "dmNavigation_aObjectIDs,dmHTML_aObjectIDs" > > Then, if I wanted to add checking for image references from a custom > type, I could extend dmImage and call something like: > > super.delete(stObj.objectID, > "dmNavigation_aObjectIDs,dmHTML_aObjectIDs,myCustomType_aRelatedIDs") > > If you think this is a good idea I will go ahead and raise an > enhancement request for it. Thanks again,
Sounds like a great idea :) What about passing in just a list of content types, and then using the metadata cache to determine what array properties in the type are joining dmImage? With that in mind it may not be all that of a performance hit to set the default to automatically check for anything that joins dmImage in an array property. My only issue with all this is how we bubble up to the UI what happened during the delete. It's something we've been discussing for a little while internally. For example, it would be nice in the object admin screen to show messages indicating the success or failure of a requested task such as a delete. -- geoff http://www.daemon.com.au/ --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "farcry-dev" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/farcry-dev?hl=en -~----------~----~----~----~------~----~------~--~---
