- 🇮🇳India Prudhvi32
Unable to decode output into JSON: Syntax error
Error: Call to a member function uuid() on null in taxonomy_convert_entity_ids_to_uuids().
on line 78 app\core\modules\taxonomy\taxonomy.post_update.phpFor this issue, we created a patch and patch is working fine.
- last update
9 months ago Patch Failed to Apply - last update
9 months ago Patch Failed to Apply - Status changed to Needs review
9 months ago 3:55pm 18 September 2023 - last update
9 months ago Custom Commands Failed - 🇷🇴Romania vasike Ramnicu Valcea
Here it is just a re-roll of previous patch. i hope i didn't miss a thing.
Let's see if it passes. Maybe we'll have also a MR that could help chasing and keep it updated till ... it's done
- last update
9 months ago Patch Failed to Apply - last update
9 months ago 29,479 pass - last update
9 months ago 30,164 pass - 🇷🇴Romania vasike Ramnicu Valcea
Small update, attempting to fix CS/phpstan errors.
+ A version for 10.1.x, as it seems they are not the same.
- last update
9 months ago 28,526 pass - Status changed to Needs work
9 months ago 2:03pm 19 September 2023 - 🇺🇸United States smustgrave
Taking a brief look and looks like this is very close.
Seems like changing to UUID should be captured in a change record though, just to let others know.
- last update
8 months ago Patch Failed to Apply - last update
8 months ago 28,526 pass - 🇳🇱Netherlands Lendude Amsterdam
There is still a lot of clean up happening here, which makes it really hard to review. While I like that logic is moved to its own methods instead of having one giant method, it makes reviewing this very hard, because its really hard to see which changes are needed here and which changes are just moving existing logic around. Please please please keep the patch/MR as small as possible
This feels very light on test coverage, but again, since it's hard to see what is actually new code here that should have test coverage, it might be sufficient.
Since we have an update hook, it should have an upgrade path test for that hook.