Delete content types and field instances upon uninstallation

PageFlip itself (beta1 at least) doesn't create any database tables. It just stores some entries in the variables table. If you imported content types from the included text files, you'll have to manually remove those content types from your Drupal installation.

The variables should probably be cleaned up on uninstall, so I'll leave this open.

I have disabled the modules, the modules weren't in the deinstall section.
I have 3 content types of page book, which didn't removed himself.
I don't have a delete button, so i can't remove them, i would like to clean up the content types, any idea's?

(all the fields that we're related to eachother are deleted)

You can run this query on your site's database to make the content types deletable:

UPDATE node_type SET custom = 1 WHERE type IN ('pageflip_book', 'pageflip_chapter', 'pageflip_page');

(If you don't have direct database access method, I'm pretty sure the devel module gives you a box into which you can type a query.)

thanks adamdicarlo! this helped me :)

The module should delete its content types when you choose to uninstall it, so I'm turning this issue into a bug report for that.

In file pageflip.module append this function

/*
 * Implementation of hook_disable().
 */
function pageflip_disable() {
  module_load_include('inc', 'pageflip', 'pageflip.import');
  _pageflip_remove_fields();
}

and in pageflip.import.inc append this one

/*
 * Deletes all the fields created by the PageFlip content types
 */
function _pageflip_remove_fields(){
  node_type_delete('pageflip_page');
  node_type_delete('pageflip_chapter');
  node_type_delete('pageflip_book');
  variable_del('pageflip_fields_installed');
}

After appending and saving, upon disabling the module, the node types will be deleted.

PS: I attach the pageflip module (version 6.x-1.0-beta4) with the changes described above.

This code should all be in hook_uninstall() in the pageflip.install file (not hook_disable). So just hook_uninstall() -- there's no need to have a separate function (_pageflip_remove_fields). Could you roll this as an actual patch off of 6.x-1.x and submit that?

Yes, it really has to be in the .install file. But since the developer used the _disable hook and a _pageflip_insert_fields function, I just followed his road. If I have enough time, I will make a patch but do not really count on me for this one.

Same issue in Drupal 7. Unable to uninstall the 3 content types mentioned above. Going to try the DB Update Query to see if that takes care of it.

DB Update did not take care of the problem. I still have these 3 rogue content types that I can't delete.

Any ideas on how to get rid of them?

yes , Same problem in D7.

Committed to 6.x-1.x: http://drupalcode.org/project/pageflip.git/commit/cc2ae1a

Now also fixed in 7.x-1.x: http://drupalcode.org/project/pageflip.git/commit/3270be1