Next release
Assuming that there are no problems with the RTBCs, they should be committed and a new beta release cut.
Stable release blockers
#3050027: Inline Blocks on Layout Builder Fail to Clone Correctly
#3258535: Translatable entity reference fields are not cloned correctly
#3067006: Permissions do not work correctly
#3060223: Corrupted data on cloning nodes that contain paragraphs
#3010626: Sub-entity cloning has unnecessary recursion for form causing memory issues
#3181886: Type coupling a dependency in DynamicLocalTasks class
Nice to have
#2724939: Change cloned entity's created-date to the date it was cloned
#3103935: Provide aliases to cloned content.
#3002611: Clone translations
#3313155: Fix test for D10 readiness
#3312870: Coding standards issue fixes
#3287213: Automated Drupal 10 compatibility fixes
Original report
There have been 4 commits and are 9 RTBC issues since the last release, a new release would be great.
Comments
Comment #2
colanI repurposed this issue as a stable release roadmap, which will include multiple releases.
Comment #3
paulocsI also support it.
I can help you providing and reviewing patches.
Comment #4
colanComment #5
colanJust cleared out the RTBCs and cut beta5.
Comment #6
mparker17Note #2724939: Change cloned entity's created-date to the date it was cloned is ready for review.
Comment #7
rajeshreeputraComment #8
rajeshreeputraComment #9
rajeshreeputraRequesting review, as stable release blocker issue are fixed. Once we get approval on this will cut the 2.0.0 stable release.
Comment #10
justcaldwellJust my 2 cents, but anyone who manages a site using Layout Builder would likely consider #3050027: Inline Blocks on Layout Builder Fail to Clone Correctly a stable release blocker. The fix had to be reverted, so the issue is open again.
Comment #11
rajeshreeputraCertainly! Addressing the concern at hand, we can indeed consider making an exception for Layout Builder as part of our current development strategy. We understand that it holds a unique place in our ecosystem and requires special attention.
Additionally, your suggestion of creating a dedicated submodule to support entity cloning in conjunction with Layout Builder is a prudent measure. This approach allows us to safeguard against potential disruptions for other functionalities. By isolating this support into a separate module, we can ensure that if there are any specific compatibility issues or bugs, they can be managed without affecting the stability of the core system. This separation of concerns will facilitate smoother integration and provide a clearer path for maintenance and updates in the future.
Are there any other details or specific workflows you believe we should consider in this context?
Comment #12
rajeshreeputraRegarding the concerns about the Layout Builder's issue(#3050027: Inline Blocks on Layout Builder Fail to Clone Correctly) with the Entity Clone module, I believe we can address this issue independently rather than allowing it to delay the stable release. We will continue to monitor the issue closely while proceeding with the release.
Comment #13
rajeshreeputra@colan, @joevagyok, @vpeltot, @Upchuk, @NickDickinsonWilde I would appreciate hearing your opinion on this matter.
I have created the 2.0.0 tag, but it has not yet been released.
Comment #14
justcaldwellI appreciate your willingness to have the discussion @Rajeshreeputra, and I don't want to overstate or belabor the point. That said, I think a stable release implies that all known critical/major issues around Drupal core functionality have been resolved. If there were incompatibilities with other contrib projects, I'd get it. But Layout Builder and translated entity references are both core-provided functionality with outstanding issues (#3050027 and #3258535 respectively) that can and do lead to data loss.
If you do proceed with a "stable" release, please consider adding a warning to the release notes (and, ideally, the project page) that Entity Clone should not be used in those cases. Thank you!
Comment #15
colanMy thoughts: Let's make it a beta release until core support is resolved.
Comment #16
colanComment #17
rajeshreeputraSure, will cute the beta, along with #3363333: Cloneable entities functionality does not restrict content type entity and #3359414: Option to clone content nodes including all comments.
Comment #18
justcaldwellAdding blockers to the IS.
Comment #19
rajeshreeputraFor issues #3103935: Provide aliases to cloned content. and #3258535: Translatable entity reference fields are not cloned correctly, we have requested a review from the community. Upon receiving the 'Reviewed & Tested by the Community' (RTBC) status, we will proceed with the merge and release.
This will leave us with only issue #3050027: Inline Blocks on Layout Builder Fail to Clone Correctly to resolve before we can proceed with a stable release.
Comment #20
yesct commentedIs this roadmap still accurate?
https://www.drupal.org/project/entity_clone/issues/3050027
and
https://www.drupal.org/project/entity_clone/issues/3258535
are blockers?? Or can a D11 release be made with these bugs? And maybe the bugs can be mentioned in the release notes?