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.
which i find weird tbh.
i think though we should add it as dependency.
Comment | File | Size | Author |
---|---|---|---|
#6 | file_entity-token-2114793-6-do-not-test.patch | 6.5 KB | ParisLiakos |
#6 | interdiff.txt | 4.34 KB | ParisLiakos |
#3 | file_entity-token-2114793-2.patch | 2.16 KB | ParisLiakos |
#3 | interdiff.txt | 1.39 KB | ParisLiakos |
#1 | file_entity-token-2114793-FAIL.patch | 915 bytes | ParisLiakos |
Comments
Comment #1
ParisLiakos CreditAttribution: ParisLiakos commenteda test that demonstrates the failure
Comment #3
ParisLiakos CreditAttribution: ParisLiakos commentedso lets add token as dependency
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedyes, of course it failed because bot does not checkout the token module
locally the test passes
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedlets get rid of the module_exists calls too
Comment #7
Chris Matthews CreditAttribution: Chris Matthews commentedThe 5 year old patch in #6 does not apply to the latest file_entity 7.x-2.x-dev and if still relevant needs a reroll.