It seems I have made big mistake of how I added the invokeAllWIth. Here is proper updated patch that would call the parent::postLoad so that way it is compatible with D9 or D10 as the parent will know which one to invoke.
I have stumbuled on this issue as we have some code that does not properly populate the file_usage table which results in cache not being cleared. I am updating the patch for latest dev.
Thank you for working on this however it is not working.
In patch 40 there was added section to support "overrides_entity_type" to be array however in 43 and after that was removed. As a result of that removal the isOverwrittenForEntityType for file is always false because the check $plugin['settings']['overrides_entity_type'] === $entity_type_name will always fail. I will provide updated patch soon.
Can this be an issue? https://www.edwardradau.com/blog/permission-denied-issue-simplesamlphp-l...
There is issue with the patch from #86:
`.${$(this.progress.element).attr('class').replace(/\s/g, '.')},`,
this produces selector with extra comma at the end such as :
.ajax-progress.ajax-progress-throbber,
and this leads to
Uncaught Error: Syntax error, unrecognized expression: .ajax-progress.ajax-progress-throbber,
that breaks every ajax call with progress
Not sure when the condition for current_theme got changed but before it was array now it is single.
It cannot be put in separate update because once you have conditions with both node_type and current_theme which is a case we had it is impossible to run the update as its validation will fail either on node_type or on current theme.
@SocialNicheGuru as far as I am aware they are. getEntityClass exists as method and the use of `invokeAllWIth` is wrapped in method_exists check.
The updated works great for the node_type however there are more updates to be done as the current_theme is not single and not multiple also causes issues. Unfortunately it is not possible to make separate updates when there are configs with both node_type and current_theme conditions so I added that as well.
Adding new patch for D10 against the 2.x dev branch as some things are removed in the DomainStorage and needed changes:
1. Have to use $this->getEntityClass();
2. Have to use 'invokeAllWIth' for the modulehandles as 'getImplementations' is removed.
+1 for this being merged, the Context 5 rc1 for Drupal 10 has been released since 22 February 2023 so should be fine to go with it. Without this Drupal 10 upgrade is getting impossible.