- π―π΄Jordan Rajab Natshah Jordan
Tested #12 with Inline Entity Form
2.0.0-beta3
- π―π΄Jordan Rajab Natshah Jordan
Tested #12 with Inline Entity Form
2.0.0-beta6
- Issue was unassigned.
-
podarok β
committed 3b0b8ddc on 2.0.x authored by
R_H-L β
Issue #3136514 by mmaldonado, R_H-L, StryKaizer, Rajab Natshah, geek-...
-
podarok β
committed 3b0b8ddc on 2.0.x authored by
R_H-L β
- Status changed to Fixed
over 1 year ago 3:23pm 25 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Needs review
about 1 year ago 10:20pm 9 December 2023 - Status changed to Needs work
about 1 year ago 10:40pm 9 December 2023 - πΊπΈUnited States dcam
This is a duplicate of π Cannot update the order weight using complex form widget Needs work . Both issues have patches with widely different solutions. We need to figure out which one to implement (if any). Both patches need tests. The less desirable solution's issue needs to be closed after granting credit to all contributors in the other issue.
- πΊπΈUnited States dcam
After evaluating the two available solutions, my preference is to close this one. This patch adds more code to sort the references in the event no entity was changed. But that function already sorts the references toward the end. So this introduces a huge code and functionality duplication problem. What happens if or when sorting functionality needs to be updated or fixed? That's just asking for more bug reports.
Furthermore, the π Cannot update the order weight using complex form widget Needs work patch removes the code that causes the bug. This patch just works around it. Maybe the other patch isn't a complete solution yet, but I would rather start from that point than just add more code bloat to this module.