I just committed Features integration to the Drupal 6 version: http://drupal.org/cvs?commit=501020

We need to port it to Drupal 7. Patch is attached.

Comments

voxpelli’s picture

Version:7.x-1.x-dev» 6.x-1.x-dev
Status:Patch (to be ported)» Needs review
StatusFileSize
new1.03 KB

The implementation wasn't a 100% correct - here's a follow up patch to the above - both need to be applied.

voxpelli’s picture

StatusFileSize
new1.04 KB

There was a typo in the previous patch - fixed in this one.

langworthy’s picture

StatusFileSize
new1.04 KB

Patch from #2 rerolled using --no-prefix flag with git diff

chromix’s picture

I'm getting the following error:

WD php: Parameter 3 to nodeformcols_features_pipe_node_alter() expected to be a reference, value given [error]
in drupal/acquia-drupal-1.2.36/includes/common.inc on line 2892.
WD php: file_put_contents(sites/all/modules/features/my_feature/my_feature.views_default.inc): [error]
failed to open stream: Permission denied in drush/includes/drush.inc on line 1164.
Created module: my_feature in sites/all/modules/features/my_feature [ok]
Parameter 3 to nodeformcols_features_pipe_node_alter() expected to be a reference, value given in [warning]
includes/common.inc on line 2892.
file_put_contents(sites/all/modules/features/my_feature/my_feature.views_default.inc): failed to [warning]
open stream: Permission denied in drush/includes/drush.inc on line 1164.

scottrigby’s picture

Why change $data to $export['features']['node']?

Also this patch drops $module_name (re #792472: Implement node type -> strongarm pipe for Features).

Maybe i'm missing why that needs to be there though?

voxpelli’s picture

@scottrigby: See #1078478: Multiple features and strongarm conflicts regarding $data - can you check your change against that patch as well?