- Issue created by @dieterholvoet
- Open on Drupal.org →Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8last update
almost 2 years ago Not currently mergeable. - @dieterholvoet opened merge request.
- last update
almost 2 years ago 11 pass - Status changed to Needs review
almost 2 years ago 8:36pm 23 July 2023 - last update
almost 2 years ago 10 pass, 2 fail - Status changed to Needs work
almost 2 years ago 10:09pm 23 July 2023 - 🇩🇪Germany FeyP
Thanks for filing the issue.
The assumption was that you made the update of the module before upgrading core, a direct upgrade of both at the same time has never been tested.
Looking at the test results, it looks like this is failing, if you do the update in Drupal 9. I guess because then the dependencies are not fulfilled, because there is no such user update hook, yet?
Looks like we need to work on this some more.
- 🇩🇪Germany FeyP
For now, I've added this as a known issue to the release notes. I'd also say this is a major issue.
- last update
almost 2 years ago 11 pass - 🇧🇪Belgium dieterholvoet Brussels
I added a workaround that should fix this issue. However, the test failure looks like a core bug, so I'll file an issue in that issue queue.
- Status changed to Needs review
almost 2 years ago 9:21am 24 July 2023 - last update
almost 2 years ago 11 pass - 🇩🇪Germany FeyP
Thanks! Test results look good now and the code looks good as well. I didn't realize that the fix would be that easy, I thought we might have to move everything to post update hooks, which might have had other problems, or work on the role config directly or something...
I'd like to give this some manual testing before I commit this, just in case, will do that later. Then I can hopefully also take a look at the MR for core, unless @smustgrave beats me to it ;).
- 🇩🇪Germany FeyP
Now had a chance to test the various upgrade paths manually and it seems to work as intended. RTBC.
53:20 52:48 Running- Status changed to RTBC
almost 2 years ago 2:01am 14 August 2023 -
FeyP →
committed 3ea733c9 on 4.x authored by
DieterHolvoet →
Issue #3376410 by DieterHolvoet, FeyP: The 9401 and 9402 update hooks...
-
FeyP →
committed 3ea733c9 on 4.x authored by
DieterHolvoet →
- Status changed to Fixed
almost 2 years ago 2:06am 14 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.