On 29/04/2009, at 1:40 PM, Henrique Recidive wrote:
2009/4/29 Chris Johnson <cxjohnson@gmail.com>:
Perhaps warning the user that they have N nodes of the content type supported by the module which is about to be uninstalled would be one idea?
That's exactly what core does when deleting a custom content type:
"Warning:</strong> there are currently @count %type posts on your site. They may not be able to be displayed or edited correctly, once you have removed this content type."
So I guess modules doing the same thing on uninstall is a good practice.
Could also be helpful to have a link in that message to admin/content/ node with the type=<content_type_to_be_deleted> filter set so that they can easily review the impact/scope of their change. Regards, Nik