- Issue created by @loze
- Merge request !8Issue #3410136 Fatal error after upgrading to drupal 10 โ (Merged) created by loze
- Open on Drupal.org โCore: 10.1.4 + Environment: PHP 8.1 & sqlite-3.27last update
about 1 year ago Waiting for branch to pass - Status changed to Needs review
about 1 year ago 12:42am 21 December 2023 - ๐บ๐ธUnited States loze Los Angeles
This fixes the error for me and I am able to use the site now.
- Status changed to Needs work
about 1 year ago 9:51pm 21 December 2023 - ๐บ๐ธUnited States loze Los Angeles
Actually after further testing, this doesn't solve the issue.
- Assigned to Anita verma
- Issue was unassigned.
- First commit to issue fork.
- Open on Drupal.org โCore: 10.1.4 + Environment: PHP 8.1 & sqlite-3.27last update
about 1 year ago Waiting for branch to pass - Open on Drupal.org โCore: 10.1.4 + Environment: PHP 8.1 & sqlite-3.27last update
about 1 year ago Waiting for branch to pass - Status changed to Needs review
about 1 year ago 1:57pm 24 December 2023 - ๐ง๐ชBelgium daften
I've updated the MR with a patch that works for me. I made the MenuLinkConfig plugin independent from menu_link_content, which should also fix the todo that was still there.
It's only tested briefly, so please thoroughly. I see e.g. in the menu editing for menu link config items that the machine name is not populated, not sure if that was there before.
- ๐ง๐ชBelgium daften
Updating the title, this seems to be new since Drupal 10.2 specifically
- ๐จ๐ฆCanada joelpittet Vancouver
@daften, The only change here is the interface change or did you both run into other changes needed?
This seems like the only change needed to fix this problem:
- public function getEntity() { + public function getEntity(): MenuLinkConfigInterface {
Alternatively you could change to the decorator instead of extending and avoid these kind of bugs. Would that be an OK direction for this issue?
- Open on Drupal.org โCore: 10.2.1 + Environment: PHP 8.1 & sqlite-3.27last update
about 1 year ago Waiting for branch to pass - ๐ฆ๐ฒArmenia armrus
There is a patch for module update after upgrading.
- ๐จ๐ฆCanada joelpittet Vancouver
I am just going to RTBC this as it's a fatal and up the priority
- Status changed to RTBC
8 months ago 12:06am 17 July 2024 - ๐จ๐ฆCanada joelpittet Vancouver
The patch in #11 seems to be just the MR in patch form
- ๐ง๐พBelarus beloglazov91
Fix the error regarding "Nullable types" as well in PHP 8.4.
- ๐จ๐ฆCanada joelpittet Vancouver
@beloglazov91 could you add that to the MR?
- ๐ฉ๐ชGermany geek-merlin Freiburg, Germany
Special thx to @joelpittet to notify the feature creel in รค10.
Technically, it's also addressing the related issue, but let's geit it in and call it a day.
Not adding #14 though. -
geek-merlin โ
committed a1b1f34c on 8.x-1.x authored by
loze โ
Issue #3410136 Fatal error after upgrading to drupal 10
-
geek-merlin โ
committed a1b1f34c on 8.x-1.x authored by
loze โ