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.
The issue #1057242: entity_uri() should not use $entity->uri (was $file->uri doesn't match the contract for uri callbacks) is closed as fixed.
On line #1294 of file_entity.module, there is the comment:
"@todo Use entity_uri once http://drupal.org/node/1057242 is fixed".
See: http://drupalcode.org/project/file_entity.git/blob/d3a647f96d14764087919...
Comment | File | Size | Author |
---|---|---|---|
#3 | use-entity_uri-1941884-3.patch | 1.23 KB | aaron |
#1 | use-entity_uri-1941884-1.patch | 839 bytes | aaron |
Comments
Comment #1
aaron CreditAttribution: aaron commentedThis was a piece of cake.
Comment #2
Devin Carlson CreditAttribution: Devin Carlson commented#1 looks good but from #1057242: entity_uri() should not use $entity->uri (was $file->uri doesn't match the contract for uri callbacks)'s change notice I'm not sure if this breaks anything on Drupal < 7.9? If so, we should add a dependency on Drupal core (system.module) >= 7.9 otherwise this should be good to go.
Comment #3
aaron CreditAttribution: aaron commentedHere you go. Thanks, Devin.
Comment #4
thekevinday CreditAttribution: thekevinday commentedI've used this for some time and have had no issues.
I've tried it with the latest dev version and have had no issues.
Comment #5
aaron CreditAttribution: aaron commentedCommitted to http://drupalcode.org/project/file_entity.git/commit/d6baac2