I might be missing something, but I believe these 2 functions are never called, since hook_features_enable and hook_features_disable were renamed at some point to hook_features_enable_feature and hook_features_disable_feature. I will provide a patch to fix this...

CommentFileSizeAuthor
#1 hook_features_enable-1425644-1.patch1.53 KBguillaumev

Comments

guillaumev’s picture

guillaumev commented
Status: Active » Needs review
StatusFileSize
new hook_features_enable-1425644-1.patch1.53 KB

Here is the patch...

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

See also #1330928: Minor Fix for Disabling/Enabling Features with Content Types which does the opposite by reverting the hook names, not sure which is correct (the linked issue points to the commit where this changed)

guillaumev’s picture

guillaumev commented

I'd like to raise this issue again because it is not fixed in the latest dev. I think one way or the other (changing node_features_enable or changing the hook name), this should be fixed...

hefox’s picture

hefox commented
Status: Needs review » Closed (duplicate)

Alternately... could just fix the core bug #1441950: Node types removed from hook_node_info with base = 'node_content' cannot be deleted and remove it. Anyhow duplicate of reopened #1055460: Disabling a feature does not disown content type