When installing multiple features as part of an install profile I've found that later features can't depend on node types provided.

Specifically, importing nodes using node_export fails for one of my node types as node_type_get_name() returned false.

The attached patch adds a call to node_types_rebuild() when enabling a feature that provides new node types.

CommentFileSizeAuthor
#1 features-node_types_rebuild-2510352.patch1.06 KBjacktonkin

Comments

jacktonkin’s picture

jacktonkin commented
StatusFileSize
new features-node_types_rebuild-2510352.patch1.06 KB
hefox’s picture

hefox commented
Status: Needs review » Needs work

Haven't reviewed, but doesn't pass coding standards from a quick review -- else needs to be
}
else

Thanks for the patch

mpotter’s picture

mpotter commented
Status: Needs work » Fixed

Committed this along with the coder change from #2 to df1726a.

  • mpotter committed df1726a on 7.x-2.x authored by jacktonkin 
    Issue #2510352 by jacktonkin: Call node_types_rebuild() when providing...

Status: Fixed » Closed (fixed)

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

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented

Thanks! Fixes very annoying issue.

Could you take a look at #2143765: features_modules_enabled() doesn't work when features_load_feature cache doesn't include the feature. Similar issue, but with taxonomies.