D7's pattern is to provide a drupal_alter(), so implementing modules can alter the definitions.
Patch will follow.

Members fund testing for the Drupal project. Drupal Association Learn more


amitaibu’s picture

Status: Active » Needs review
1.85 KB


NofarG’s picture

Status: Needs review » Closed (works as designed)

The alter hook is already provided by features. See features_get_default()

NofarG’s picture

Status: Closed (works as designed) » Needs review
666 bytes

New patch attached.

djdevin’s picture

Seems to be fine in D6 too. Same code.

DamienMcKenna’s picture

Status: Needs review » Reviewed & tested by the community

This is the correct approach.

hefox’s picture

ditto, patch looks good

peterpoe’s picture

Title: provide an alter hook for conditional_fields_default_fields() » Use features_get_default() in conditional_fields_features_rebuild() instead of custom hook
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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