On behalf of a client, I need to backport the features integration to Drupal 6 to keep the project deployable.

Patch coming on comment 1.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Deciphered’s picture

Status: Active » Needs review
FileSize
16.06 KB

I have no doubts that this patch is not complete as I have only partially tested it, but this is definitely and I will continue to work on the patch until my itch is scratched.

Cheers,
Deciphered.

agentrickard’s picture

While I'd hate to disrupt the d6 version with what is, in fact, a huge change, this makes sense and should go in when it's tested.

Deciphered’s picture

Updated patch that actually works - Accidentally missed some files in the last one.

agentrickard’s picture

Does Features in D6 have the same bug related to failing to load custom feature files?

That is, this line doesn't actually work: 'default_file' => FEATURES_DEFAULTS_CUSTOM,.

See domain_features_load(), which is a workaround for that.

Deciphered’s picture

Status: Needs review » Needs work

Given that I did a straight backport of the D7 version, there's a good chance I backported that as well (I even backported the issue about reverting as I'd prefer them to be as similar as possible), thanks for the heads up though, I check that out today.

Deciphered’s picture

Updated patch as a couple of functions where missing.

Deciphered’s picture

Updated patch to removed destructive rebuild behaviour.

Deciphered’s picture

Updated patch again, forgot to 'git add' the additional files.