I like it how the changelog conversion is implemented as action
plugin. One minor thing I'd add: the plugin should try to disable
itself after finishing the conversion. There is no need to
instanciate the whole plugin when it does nothing anymore. This of
course only works if the plugin directory is writable, so in
addition the ?do=check command should check if the import was
finished but the plugin is still enabled and issue an warning about
it.
Sounds good, but how do users re-enable it if they need to re-run the
import. I put 'importoldchangelog' in the $plugin_protected list of
the plugin manager. This disables all the plugin manager form
elements for the 'importoldchangelog' plugin. Should the plugin
manager be changed to allow protected plugins that are disabled to be
re-enabled?
