- 🇦🇹Austria drupalfan2
Patch #89 does not work for 8.x-1.0-rc14 with Drupal 9.5.3 (it can be applied but it does not solve the problem).
I did not find any other patch or any other solution.
Does anyone have an idea how can I solve this?
Thx. - 🇦🇹Austria drupalfan2
#93 solved:
"Allow user to make asymmetric translation" has to be activated on Manage Form Display of the conent type. - 🇩🇪Germany hchonov 🇪🇺🇩🇪🇧🇬
While testing this patch out I noticed that if the language of the content is changed this does not work quite well with referenced entities and sometimes their language gets changed as well or sometimes the entity remains in its original language and then differs from the one of the content after save ... We should not be assuming that we can simply change the language of the referenced entities since suddenly they might not get displayed in other places if they are reused. Instead, we should be translating them as this is the safe option. Worked on 🐛 Main entity language change does not propagate to nested paragraphs in preview mode Needs review for adding support for propagating the language change and adding support in the patch here as well. I am not sure if this is the right issue though or if another is needed for this.
The last submitted patch, 95: 2822764-95.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- 🇨🇴Colombia waspper
There is a failing test about the same langcode for child entities. IMHO, it's not "valid", because this patch is in the way to allow having asymmetric content. I've removed some lines. Feel free to improve/blame :)
The last submitted patch, 97: 2822764-97.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.The last submitted patch, 99: 2822764-99.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs work
11 months ago 8:09am 19 July 2023 - @quadrexdev opened merge request.
- 🇺🇦Ukraine quadrexdev Lutsk
I've prepared a merge request to the 2.0.x-dev branch with an updated patch. For me it works well, appreciate it if anyone else could test it.
Also, attaching a file version of a patch in order to run some tests (cuz it seems for the 2.0.x-dev branch we have disabled tests running).
- last update
11 months ago 8 pass, 8 fail - last update
11 months ago Composer require failure - Status changed to Needs review
11 months ago 9:53am 19 July 2023 - last update
11 months ago Patch Failed to Apply - 🇺🇦Ukraine Goodmood
I'm still using rerolled patch from #90 (due to the same reason which mentioned there) over rc15 and D10.
With Drupal upgrade we have an exception when entity removed from ief widget and parent entity saved:
"Entity queries must explicitly set whether the query should be access checked or not"
This is because of absent accessCheck method on query in 'delete' method.
Added this check to patch from #90Also this should be added to patch #99 because otherwise it'll throw an exception on D10 as well.
- last update
10 months ago 8 pass, 8 fail I create a new patch for drupal 10.1.x, thanks for @quadrexdev, the #103 is useful for me. At the same time, I merged the patch in this issue Parent form entity builders run on IEF resulting in fatal errors 🐛 Parent form entity builders run on IEF resulting in fatal errors Active into the new patch.
And I added the code for checking entity langcode and form langcode to get the corresponding entity.- 🇺🇸United States drupalevangelist Detroit, MI
I could not apply the #106 patch on Drupal 9.5.9 with PHP 8.1 on the 8.x-1.0-rc15 version. Was anyone able to use this patch cleanly?
- 🇫🇷France JulienVey
Indeed the #106 patch is already based on the one you linked @drupalevangelist.
Here's the one that does not need any pre-patch :)
- 🇺🇸United States drupalevangelist Detroit, MI
Thank you! I was able to apply cleanly #111 patch on 8.x-1.0-rc15. But now I'm not encountering the following error when I try to create nodes using certain content types that has the Inline Entity Form widget configured:
Error: Call to a member function bundle() on null in Drupal\views\Plugin\EntityReferenceSelection\ViewsSelection->stripAdminAndAnchorTagsFromResults() (line 289 of core/modules/views/src/Plugin/EntityReferenceSelection/ViewsSelection.php).
- last update
9 months ago 2 pass, 14 fail - 🇧🇷Brazil adinancenci
111 did not worked on 1.0-rc15 so I made some slight alterations for anyone interested.
- last update
6 months ago 28 pass, 2 fail - last update
5 months ago PHPLint Failed - last update
5 months ago Patch Failed to Apply - last update
5 months ago 12 pass, 3 fail - last update
5 months ago Patch Failed to Apply - last update
5 months ago Patch Failed to Apply - 🇱🇻Latvia maijs
Adding refactored patch for the 3.x branch (c753ec1).
- The following code is reinstanted back to
InlineEntityFormComplex.php
file as removal of these caused namespace reference loss for existing code:use Drupal\Component\Utility\Tags; use Drupal\Core\Entity\Element\EntityAutocomplete;
- 🇱🇻Latvia maijs
Here's the follow-up update to the patch in #117.
- The following code is reinstanted back to
InlineEntityFormComplex.php
file as removal of these caused namespace reference loss for existing code:use Drupal\Component\Utility\Tags; use Drupal\Core\Entity\Element\EntityAutocomplete;
- 🇳🇱Netherlands JoshaHubbers
This is the patch is the same as #118, but rerolled for 3.x-rc19.
- last update
3 months ago 28 pass, 2 fail - 🇮🇳India sarvjeetsingh
The patch is same as #113, re--rolled it for 8.x-1.0-rc17
The last submitted patch, 120: 2822764-inline_entity_form-120.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.