Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This module (among the others) incorrectly assumes that a fieldable entity must be a content entity also. This might be true for the core entities but custom entities can be fieldable without extending the Drupal\Core\Entity\ContentEntityInterface interface. (It is not easy but possible. :) )
Comment | File | Size | Author |
---|---|---|---|
#2 | file_entity-not-all-fieldable-entity-is-content-entity-2977747-2.patch | 2.42 KB | mxr576 |
Comments
Comment #2
mxr576Comment #3
mxr576Can somebody review this fix? It is in the queue waiting for review for a very long time...
Comment #5
joseph.olstad@mxr576
I have granted you full maintainer priviledges on the all branches. However if you could focus on the 8.x-2.x branch that'd be great.
please review all 8.x-2.x issues and commit /release whatever you see fit.
7.x-2.x/7.x-3.x is in a good spot right now, but 8.x-2.x needs tlc.
Thanks,
Comment #6
joseph.olstadThe tests for the 8.x-2.x branch seem to be complaining more than ever now, not sure if this patch made things worse or if it's other changes upstream.
Please investigate
Comment #7
mxr576Thanks for merging this patch and giving me maintainer rights. At this moment, I have enough projects to maintain but if I find some time I'll take a look around this one as well.
Comment #8
BerdirComment #9
BerdirThis is fine and doesn't cause the current issues.
Comment #10
mxr576