Creating a content type on _install() will break if aggregator is included in an installer profile because at the time the module is installed, no other module is available.

-> Move creation of content type to hook_enable()

This requires a test whether the content type already exists, because enable is called everytime a module is selected on the admin/build/modules page, not only when it is selected the first time (=installed)

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/new_aggrega...

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Assigned: Unassigned » aron novak
Status: Active » Fixed
mustafau’s picture

mustafau commented

There is nothing wrong with creating content type in _install() hook. Book module does it. Creating content type requires node module and node module is available to install profiles.

mustafau’s picture

mustafau commented

See http://api.drupal.org/api/function/book_install

alex_b’s picture

alex_b commented

@mustafau: thanks for the heads up. I wasn't aware of this.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.