A module that works based on all entities like Metatag and Pathauto (in the future) operate on an entity basis not on individual entity types. Making those modules support features by having to implement each component pipe alter will mean eventually some entity types that could be exported will not be supported. Let's use drupal_alter()'s multi-$base ability to actually run a general alter hook.

Comments

dave reid’s picture

Title: Provide an actual hook_features_pipe_component_alter() for general altering. » Provide an actual hook_features_pipe_alter() for general altering.
dave reid’s picture

Status: Active » Needs review
StatusFileSize
new4.33 KB

Patch for review including changes in documentation and supporting this hook in modulename.features.inc hook group files.

febbraro’s picture

Assigned: Unassigned » febbraro
febbraro’s picture

febbraro’s picture

Status: Needs review » Fixed
tim.plunkett’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Assigned: febbraro » Unassigned
Status: Fixed » Patch (to be ported)
hefox’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new3.19 KB

Quickly done backport; seems fine from quick glance.

Did features_pipe first, cause looking at d7, looks to be the order drupal_alter executes them in.

damienmckenna’s picture

Bump for testbot.

  • febbraro committed 476b24e on 8.x-3.x
    Issue #1300780 by Dave Reid: Added Provide an actual...