- Issue created by @tte
- Status changed to Needs work
over 1 year ago 7:50am 12 May 2023 - 🇩🇪Germany tte
Sorry, I'm still investigating. The issue seems to be caused by https://git.drupalcode.org/project/inline_entity_form/-/blob/8.x-1.x/src...
The method countReferenceableEntities triggers the memory allocation error, as there's no limit involved when looking for referenceable entities. I've yet to find a workaround, after I thought I was on the right track. Versions of inline_entity_form prior to 8.x-1.0-rc15 are not affected.
- 🇩🇪Germany tte
Added the patch. As you only need to check whether there is more than one referenceable entity, you may use the getReferenceableEntities method with a limit of 2, which avoids loading all entities which are not needed in this case.
- @tte opened merge request.
- Status changed to Fixed
over 1 year ago 9:46am 13 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Needs review
about 1 year ago 7:13pm 10 December 2023 - Status changed to RTBC
about 1 year ago 7:14pm 10 December 2023 - First commit to issue fork.
- 🇺🇸United States dcam
I messed something up with that new MR 99. I was just trying to clean up the unrelated changes in MR 73, one of which fixed a code formatting problem and another that created a new code formatting problem. I figured they would cause problems for any phpcs issues.
I'm just going to hide those branches and start over.
Please don't credit me on this.
- Status changed to Needs review
12 months ago 11:26pm 23 December 2023 - 🇺🇸United States dcam
I think it's ok now. I'm sorry if I made things worse. Please let me know if I need to fix something and I'll do my best to correct it.
- Status changed to RTBC
12 months ago 12:40pm 2 January 2024 - 🇬🇧United Kingdom Sophie.SK
This patch has resolved the issue for me :)
The PR needs rebasing onto the dev branch but otherwise, marking as RTBC.
- Status changed to Needs work
12 months ago 9:33pm 7 January 2024 - 🇩🇪Germany geek-merlin Freiburg, Germany
Hmm, the Gitlab rebase does not work out.
Whoever does the rebase, feel free to self-rtbc. -
geek-merlin →
committed dbf05532 on 3.x
Issue #3359875 by dcam, tte, geek-merlin, Sophie.SK: Memory allocation...
-
geek-merlin →
committed dbf05532 on 3.x
- Status changed to Fixed
12 months ago 9:49pm 7 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- 🇮🇳India Supreetam09 Kolkata
Changes from this caused a new issue https://www.drupal.org/project/inline_entity_form/issues/3469864 🐛 Remove entity button not showing in Complex Inline Entity Form after upgrading to 3.x from 1.x Active