In #624018: Exportables and Features support for WYSIWYG 7.x we're adding optional features support without creating a dependency on Entity API. Such features require Entity API, but currently it's not added as a dependency.

We can safely add this dependency as, if it is not needed, Features will filter it out. E.g., when adding a Profile2 profile type, only Profile2 is detected as a dependency since it depends on Entity API.

CommentFileSizeAuthor
#3 1047704-3-entity-features-dependency.patch917 bytesnedjo
#1 1047704-1-entity-features-dependency.patch768 bytesnedjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Active » Needs review
FileSize
1047704-1-entity-features-dependency.patch768 bytes

Patch attached.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs review » Needs work

Hm, ok as features won't add unnecessary dependencies, I think we can do that. However we should document that in a comment.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Needs work » Needs review
FileSize
1047704-3-entity-features-dependency.patch917 bytes

More explanatory comment.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs review » Fixed

thanks, committed.

Status: Fixed » Closed (fixed)

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