- Issue created by @kevinfunk
- last update
almost 2 years ago 223 pass - @kevinfunk opened merge request.
- 🇮🇳India vishalkhode
We are also seeing the same issue and can confirm the MR !93 fixes the issue.
- Status changed to RTBC
almost 2 years ago 4:56am 11 July 2023 - 🇬🇧United Kingdom jonathan1055
Thanks @kevinfunk for reporting this. I presume you are using the newly released 2.0.0
This function was added in 🐛 Don't run hook_modules_installed during config install Fixed - the last commit to be made before releasing. - last update
almost 2 years ago 223 pass - 🇬🇧United Kingdom jonathan1055
Looking for
hook_config_import_steps_alter
in Core D9, the only example I could find was in field.module where $context is defined as an array. The core.api.php example also defines it as a plain array.So maybe Drush is doing something a bit different? What Drush version are you using?
- 🇺🇸United States kevinfunk
@jonathan1055 I am using Drush 11.6.0.
The example for function hook_config_import_steps_alter does not show a type hint for$context
. I think it has to do with it being a batch. I also found an issue for Queue UI where they had an issue with the type hint #3313894 → . - 🇬🇧United Kingdom jonathan1055
You are right that the core.api.php does not have the array type hint, but in the comments it says: "The callable should accept a $context array by reference." Maybe that is where the confusion started. Thanks for linking the other issue. That one also had #3306333: Support calling the QueueUiBatch service from Drush → for background.
- last update
almost 2 years ago 223 pass - last update
almost 2 years ago 223 pass -
jonathan1055 →
committed fe4cee5b on 2.x authored by
kevinfunk →
Issue #3373725 by kevinfunk, vishalkhode: TypeError:...
-
jonathan1055 →
committed fe4cee5b on 2.x authored by
kevinfunk →
- Status changed to Fixed
almost 2 years ago 2:42pm 12 July 2023 - 🇬🇧United Kingdom jonathan1055
Fixed. Thank you again for reporting this and for providing the merge request.
- 🇮🇳India chandu7929 Pune
Hi @ jonathan1055 when can we expect this to be released? This is breaking our site installation.
- 🇬🇧United Kingdom jonathan1055
When 🐛 entityRevert breaks config:import if there are entity types that do not have a bundle class Fixed is done, which should be by tomorrow.
I think you can use the patch from this mr in your installation? https://git.drupalcode.org/project/scheduler/-/merge_requests/93.diff - 🇬🇧United Kingdom jonathan1055
I have released Scheduler 2.0.1 → which contains this fix.
Automatically closed - issue fixed for 2 weeks with no activity.