setup() returns without node_type_save() being executed on node types defined by enabled modules in their hook_node_info() (e.g 'blog'). This doesn't happen with modules installed 'in real life' through the UI, since admin/build/modules page callback includes a node_type_rebuild().

Marking as 'critical', since it raises false positive tests failures in #372743: Body and teaser as fields (which currently passes because it includes this fix).

(install profiles seem to have the same issue, BTW : if you add 'blog' to the list of modules in default_profile_modules() and install a fresh drupal, the 'blog' type only appears in the {node_type} table after the first visit to admin/build/modules, which triggers a node_types_rebuild(). This patch doesn't attempt to fix it.)

CommentFileSizeAuthor
setup_rebuild_node_types.patch667 bytesyched

Comments

catch’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed, but is there a reason why we don't just do a drupal_cache_clear_all() or whatever here and be done with it?

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

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