To reproduce:
Grab latest head, and install standard profile.
Go straight to the module page and disable and then uninstall comment
Run cron to trigger field_purge_batch() (twice to be sure)
Go to dblog
Observe some exceptions that look like:
Drupal\field\FieldException: Attempt to create an instance of unknown field 1ee73fab-78da-499d-b306-6c78a15879b6 in Drupal\field\Plugin\Core\Entity\FieldInstance->__construct() (line 223 of /var/www/d8/core/modules/field/lib/Drupal/field/Plugin/Core/Entity/FieldInstance.php).
1ee73fab-78da-499d-b306-6c78a15879b6
is the Uuid of the instance
a debug bactrace here:
https://privatepaste.com/c1adf5434c
Comments
Comment #1
swentel CreditAttribution: swentel commentedUgh yeah, disabled modules are a pain, they are almost gone ... #1199946: Disabled modules are broken beyond repair so the "disable" functionality needs to be removed
Comment #2
ParisLiakos CreditAttribution: ParisLiakos commentedthis bug was introduced lately though, i stepped on to it in #15266: Replace aggregator category system with taxonomy
so it should be because of a commit after April 5 or so, probably Field API to CMI one
Comment #3
pcambraMaybe related: #1264756: Inline field purge within field_delete_field()
Comment #4
swentel CreditAttribution: swentel commentedBeen trying to reproduce this, but can't anymore .. should we defer to #2003966: Helper issue for #1199946 : Disabled modules are broken beyond repair ?
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedi just reproduced it with a clean install/ fresh HEAD.
The issue above will probably fix this, but lets keep this issue open till then
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedlets revisit after #1199946: Disabled modules are broken beyond repair so the "disable" functionality needs to be removed
Comment #7
David_Rothstein CreditAttribution: David_Rothstein commentedI couldn't reproduce this - is it still an issue? Also, this isn't related to disabled modules; it's an issue with uninstall.
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedyes, i can still reproduce this, but dunno, it might be just me
Comment #9
ParisLiakos CreditAttribution: ParisLiakos commentedseems #2081609: field purge should bail out on unknown entity types fixed this