Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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!