Clean up problematic 2.x branch

Created on 14 November 2023, about 1 year ago
Updated 11 December 2023, about 1 year ago

Problem/Motivation

The 2.x branch and its commits violated semver and d.o rules, adding many unreviewed commits and regressions, causing mess and wasted resources.

This issue is about cleaning up the 2.x branch to bring it back inline with the work in the 1.x branch.

This issue is not about discussing accountability or governance impacts, please open other issues if needed.

Proposed resolution

- [X] Mark 2.x unsupported
- [X] Announce the cleanup on the project page
- [X] Create a 3.0.x branch to resume the 1.x work (so it is clear to people that they have to upgrade)
- [ ] Collect all 2.x commit issues
- [ ] Bulk assign all 2.x issues to 3.x
- [ ] Reopen issues committed to 2.x
- [ ] Finish the stable 1.x / now 3.0.x release

List of issues for more thorough review

- πŸ› Complex Widget selects wrong bundle when multiple bundles exist but user only has access to create one. Postponed: needs info
- ✨ New API hooks to react on inline entity save/delete Needs review
- πŸ› Translated entity should be used for the table presentation in the widget Needs review
- πŸ› Can't add more than two items with simple inline entity form widget under some circumstances Needs review
- ✨ Allow inline entity form submit handler to return the saved entity Needs review
- πŸ› Entity form simple weight duplicates Needs review
- πŸ› Wrong bundle form is displayed if user has permissions to create only one of the available bundles Fixed
- [#]
- [#]

List of issues to commit soon-ish

Approved (geek-merlin) in the process of cleanup.
- πŸ’¬ Quick Edit not working for Inline Entity Form - Complex Fixed
- πŸ› Using 'migrate' without 'migrate_plus' leads to fatal PHP error Fixed
- πŸ› Change type hints in inline_entity_form.api.php to align with Drupal core Fixed
- πŸ“Œ Drupal 10 Fixed
- πŸ› TypeError: krsort(): Argument #1 ($array) must be of type array, null given in krsort() Fixed
- πŸ“Œ Add label to weight select field for accessibility Fixed
- πŸ› Using 'migrate' without 'migrate_plus' leads to fatal PHP error Fixed
- πŸ› Memory allocation error Fixed
- ✨ Allow themes to alter inline entity forms Fixed
- πŸ“Œ Speed up FunctionalJavascriptTests tests Fixed

List of 1.x commite with no tag

# (tag: 2.0.0-rc8, tag: 2.0.0-rc6, origin/HEAD, origin/8.x-1.x, 8.x-1.x) Patch #9: let a user specify if her blog entries should be votable β†’ applied, Issue πŸ’¬ Quick Edit not working for Inline Entity Form - Complex Fixed

List of closed 2.x issues not mentioned in commit message, and not obvious regression fix

- πŸ› Using 'migrate' without 'migrate_plus' leads to fatal PHP error Fixed
- ✨ Pass context to canAddNew() and getCreateBundles() Needs review
- πŸ› Memory allocation error Fixed
- πŸ› Wrong bundle form is displayed if user has permissions to create only one of the available bundles Fixed
- πŸ› Wrong bundle form is displayed if user has permissions to create only one of the available bundles Fixed
- ✨ Allow themes to alter inline entity forms Fixed
- πŸ“Œ Speed up FunctionalJavascriptTests tests Fixed

List of 2.x commits

# Added the option to not open the default creation form for the required field
# fix: no need for core while we have info file
# docs: simple formatting
# Issue πŸ› Using 'migrate' without 'migrate_plus' leads to fatal PHP error Fixed by dsnopek, geek-merlin: Using 'migrate' without 'migrate_plus' leads to fatal PHP error
# Issue πŸ› Error: Call to a member function id() on boolean Postponed: needs info by mattjones86, Chris Matthews: Error: Call to a member function id() on boolean
# Issue πŸ› Complex Widget selects wrong bundle when multiple bundles exist but user only has access to create one. Postponed: needs info by rlmumford, Dinesh18: Complex Widget selects wrong bundle when multiple bundles exist but user only has access to create one
# Issue ✨ New API hooks to react on inline entity save/delete Needs review by JordiK, Odai Atieh: New API hooks to react on inline entity save/delete
# Issue πŸ› Change type hints in inline_entity_form.api.php to align with Drupal core Fixed by stefan.korn: Change type hints in inline_entity_form.api.php to align with Drupal core
# Reworked ComplexWidgetRevisionsTest
# Only visual changes
# Allow themes to implement the hook_inline_entity_form_entity_form_alter() and hook_inline_entity_form_reference_form_alter() hooks
# Issue πŸ› Fix integration with content moderation in multi-lingual scenarios Needs work by andreyjan, rp7, nicrodgers, texas-bronius: Fix integration with content moderation in multi-lingual scenarios
# Issue πŸ“Œ Ability to reference an entity and auto open its edit form Needs work by Charlie ChX Negyesi: Ability to reference an entity and auto open its edit form
# Issue πŸ“Œ Drupal 10 Fixed by chr.fritsch,podarok: Drupal 10 - rector and CodeSniffer
# Update src/Form/NodeInlineForm.php
# Issue πŸ› TypeError: krsort(): Argument #1 ($array) must be of type array, null given in krsort() Fixed by gauravjeet: TypeError: krsort(): Argument [#1] ($array) must be of type array, null given in krsort()
# Issue ✨ Allow hiding fieldset and title Needs work by Pancho, nevergone, ershov.andrey: Allow hiding fieldset and title
# (tag: 2.0.0-beta2, tag: 2.0.0-beta1) Issue ✨ Support for configuration entities Needs work by Spokje, Dylan Donkersgoed, attisan, DieterHolvoet, jienckebd, Suresh Prabhu Parkala, KapilV, dwkitchen: Support for configuration entities
# (tag: 2.0.0-beta3) Issue[#3257479]: Rerolled patch againist 2.0.x
# Added the option to not open the default creation form for the required field
# Resolved merge conflicts during rebase.
# Updated the changes from the previous commit.
# Updated the patch changes for 2.0.x branch
# Issue πŸ› Multiple duplicated messages while validating the inline entity form Needs work fix multiple errors while validating entity
# When we don't know what bundle to use when adding a new entity, instead of selecting the first one from the configured target bundles, limit it to only the first one of the user's ALLOWED bundles.
# (tag: 2.0.0-beta5) Issue πŸ› Translated entity should be used for the table presentation in the widget Needs review : Translated entity should be used for the table presentation in the widget
# Avoid memory limit error when checking for referenceable entities
# (tag: 2.0.0-beta6) Issue ✨ Auto-open inline entity forms (especially for cardinality: 1) Needs work by filsterjisah, bharath-kondeti, Anybody: Auto-open inline entity forms (especially for cardinality: 1)
# Issue πŸ› Can't add more than two items with simple inline entity form widget under some circumstances Needs review by AndyF, sanfair, Dylan Donkersgoed, andrei.vesterli, mlahde: Can't add more than two items with simple inline entity form widget under some circumstances
# Issue πŸ“Œ Add label to weight select field for accessibility Fixed by edmund.dunn, geek-merlin: Add label to weight select field for accessibility
# Issue ✨ Allow inline entity form submit handler to return the saved entity Needs review by jamiehollern, petar.gnjidic: Allow inline entity form submit handler to return the saved entity
# Issue πŸ› Entity form simple weight duplicates Needs review by tetranz, MorinLuc0, luizsgpetri: Entity form simple weight duplicates
# (tag: 2.0.0-beta7) Issue πŸ› D8: Multiple forms submit/cancel closes all child forms Needs work by paper boy, mrinalini9: D8: Multiple forms submit/cancel closes all child forms
# Patch #8 applied
# (tag: 2.0.0-beta8) fix: non needed suggestion
# Patch #2 Applied
# Patch #6 applied, Issue πŸ› Create/Update Labels buttons naming improvement Needs work
# Tried to fix test fail issue
# (tag: 2.0.0-rc2) Updated info.yml files according to modified deprecated code
# Removed deprecated code
# Patch #9 applied, Issue πŸ’¬ Quick Edit not working for Inline Entity Form - Complex Fixed
# Patch #4 applied, Issue πŸ› Notice: Undefined index: label in modules/contrib/inline_entity_form/inline_entity_form.module on line 345 Needs review
# (tag: 2.0.0-rc3) Patch #17 applied, Issue πŸ› Changes are not saved for some submit buttons Needs work
# Issue πŸ› IEF does not invoke hook_entity_prepare_form Fixed by ziomizar, eugene.brit, Musa.thomas: IEF does not invoke hook_entity_prepare_form
# Issue πŸ“Œ Fix the issues reported by phpcs Needs work by urvashi_vora, apaderno: Fix the issues reported by phpcs
# (tag: 2.0.0-rc4) Issue πŸ› IEF complex widget: Re-ordering / weight sometimes not updated Needs work by mmaldonado, R_H-L, StryKaizer, Rajab Natshah, geek-merlin, artemboiko: IEF complex widget: Re-ordering / weight sometimes not updated
# Issue ✨ Allow select widget for "Add existing items" Needs work by geek-merlin, arunkumark, Rithesh BK, almunnings, Goodmood, jennakoo, shivam_tiwari, the_glitch: Allow select widget for "Add existing items"
# Issue πŸ“Œ Add support for the menu entity. Needs work : add submodule "Inline Entity Menu Form"
# (tag: 2.0.0-rc5) lost dependency
# Fix: default settings
# (tag: 2.0.0-rc7) ✨ Add a FieldFormatter to display rendered menu Postponed : Add a new Field Formatter to display rendered menu
# (tag: 2.0.0-rc9) Issue πŸ“Œ Inline entity form menu not working properly Postponed : fix ajax inline_entity_menu_form
# Issue πŸ“Œ [Inline Entity Menu Form] Save is broken Downport : fix form alter
# Fixing existing label not using label overrides.
# Reuse labels instead.
# (tag: 2.0.0-rc10, origin/2.0.x) Issue ✨ Simple Widget - allow_new setting Fixed : Simple Widget - allow_new setting

πŸ“Œ Task
Status

Active

Version

1.0

Component

Miscellaneous

Created by

πŸ‡©πŸ‡ͺGermany geek-merlin Freiburg, Germany

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024