Today I discovered that the Features support is broken again. :-( :-(
I know there were some changes to the Features module a few days ago, and I think this is part of the problem: With latest version Feature module installed, ECK entities/bundles don't show up for export in the Features UI.

You can still enable ECK entities/bundles that were exported with earlier versions of Features (though they will be marked as 'overridden').

Marking this as major, since Features support is listed as a blocker on the project page.

CommentFileSizeAuthor
#6 1490936-6-updating_feature_integration.patch605 bytesitangalo
#4 1490936.patch362 bytesmojzis

Comments

itangalo’s picture

itangalo commented

(I don't think this is related to #1370762: Re-Enable Features Support, but add a link here anyway.)

mojzis’s picture

mojzis commented

could the use of eck__entity_type__load in eck.features.inc be a part of the problem ?
(sorry, copy and paste error, originaly wrong function pasted here)

mojzis’s picture

mojzis commented

'file' => drupal_get_path('module', 'eck') .'/eck.features.inc', is missing in eck_features_api

mojzis’s picture

mojzis commented
StatusFileSize
new 1490936.patch362 bytes

from http://drupalcode.org/project/features.git/blob_plain/refs/heads/7.x-1.x...

mojzis’s picture

mojzis commented
Status: Active » Needs review
itangalo’s picture

itangalo commented
StatusFileSize
new 1490936-6-updating_feature_integration.patch605 bytes

Concept in patch works fine! Updated it to match latest dev of ECK.

mojzis’s picture

mojzis commented
Status: Needs review » Reviewed & tested by the community

btw i have a feeling it is a problem in features, i've seen a similar problem in rules. but since this works, i would leave it here :)

itangalo’s picture

itangalo commented

#7: I too think this "bug" is caused by Features. I suspect that the last release had changes (or more strict requirements) in the API.

No blame to ECK – and no blame to the people fighting to push Features forward either. But cred to the people trying to make things work!

forestmars’s picture

forestmars commented

Thanks @Itangalo! RTBC+1

This is exactly what we needed. fmizzell is this going in the next dev version?

fmizzell’s picture

fmizzell commented

yes it is!! sorry.. I have been a little busy.. but I will commit it this afternoon

fmizzell’s picture

fmizzell commented
Status: Reviewed & tested by the community » Fixed

the patch has been commited :) thanks @itangalo and @mojsiz

Status: Fixed » Closed (fixed)

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