- Issue created by @DieterHolvoet
- 🇧🇪Belgium DieterHolvoet Brussels
Maybe it could help to get rid of
PhpStorageFactory::get('trash')->deleteAll();
inTrashEntityTypeManager::clearCachedDefinitions()
? Is there any reason these files need to be regenerated on every cache rebuild? - 🇧🇪Belgium DieterHolvoet Brussels
You would think that regenerating the class suffix when deleting existing classes would prevent this issue. Maybe this only happens when someone visits the site between
PhpStorageFactory::get('trash')->deleteAll();
and\Drupal::state()->delete('trash.class_suffix');
? I'll switch these two lines and see if the issue still happens. - Merge request !25Delete the class prefix before deleting the classes → (Open) created by DieterHolvoet
- 🇷🇴Romania amateescu
FWIW, I added some `sleep()` calls in that code, executed
drush cr
anddrush updb -y
from the CLI, then browsed the site while the commands were still running, and I didn't bump into that error.However, if you find that the MR makes a difference in your case, I've no problem with merging it :)
- 🇧🇪Belgium DieterHolvoet Brussels
No it doesn't, the issue happened again since adding the patch.