Chris,

I think your best bet would be to add a button and event to your Settings.ascx file. (assuming you have one) And adding code that deletes the desired photos / db entries.

So, you have a 2 step process:

1.  Delete files through settings

2. Delete module as normal

Or if you already have an admin control to upload images then just add the ability to delete (or delete all)

Trying to tie into the module delete event just doesn't seem worth the effort.

 

Robert

 

Most modules create a foreign key constraint on the ModuleId column of their primary data table back to the DNN Modules table with cascade delete enabled so that when the module instance is finally deleted, data used by the module will be deleted from the database. Is this a best practice? Probabably not as the addition of the FK constraint effectively alters a core DNN database table which could cause problems should a framework upgrade need to drop or significantly alter the Modules table. Since the Modules table will most likely always include a ModuleId column of type int as its primary key, I still feel comfortable in following this approach - until the core exposes a module delete event or other API based approach, which I agree we developers need. Without such an event/api based approach there is no good way currently for the module to delete non-database stored files associated with the module instance from the physical file system.

Take particular note of the above phrase "when the module instance is finally deleted . . ." Remember that when a module is deleted it is placed in the module recycle bin (by setting the IsDeleted field in the Modules table) until the recycle bin is emptied of all or that particular module reference. At that point and not before, the deletion of the module instance from the Module (and other core tables) will occur, triggering the cascade delete and removal of data associated with that module instance from the module's custom tables.

---

Bill, WESNet Designs
Team Lead - DotNetNuke Gallery Module Project (Not Actively Being Developed)
Extensions Forge Projects . . .
Current: UserExport, ContentDeJour, ePrayer, DNN NewsTicker, By Invitation
Coming Soon: FRBO-For Rent By Owner