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.
See #1199946-133: Disabled modules are broken beyond repair so the "disable" functionality needs to be removed.
Field purging needs the entity storage controller, and shouldn't try anything if the entity type is unknown.
Comment | File | Size | Author |
---|---|---|---|
#3 | field_purge-unknown_entity_type-2081609-3.patch | 1.37 KB | yched |
#3 | interdiff.txt | 812 bytes | yched |
#1 | field_purge-unknown_entity_type-2081609-1.patch | 1.25 KB | yched |
Comments
Comment #1
yched CreditAttribution: yched commentedPatch.
Comment #2
swentel CreditAttribution: swentel commentedWill this still matter after disabled modules goes in ? Other than that, looks good to me.
Comment #3
yched CreditAttribution: yched commentedIt's probably going to be #2080823: Create API to discover config entities' soft dependencies and use this to present a confirm form on module uninstall rather than directly #1199946: Disabled modules are broken beyond repair so the "disable" functionality needs to be removed, but yes, that's possible.
Added a "@todo Revisit"
Comment #4
swentel CreditAttribution: swentel commentedOk, let's get this in for now then.
Comment #5
alexpottCommitted 304da11 and pushed to 8.x. Thanks!
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedthis fixed #2004316: Exception on field_purge_batch() after deleting a field
i marked it as duplicate, maybe @yched or @swentel could confirm this is the right status
thanks!