- Issue created by @nginex
- π§πͺBelgium artycal
@nginex What cases is this issue referring to? I can see some ambiguity for 2 cases:
1. The entity is referenced in a Menu link, e.g. Main Navigation > node_to_export
2. The entity references a Menu link, e.g. node_to_export contains menu_linkI assume it's for the first case, but want to make sure
- πΊπ¦Ukraine nginex
It's follow up of π Add opportunity to export import content menu items with translations Fixed
The issue might have some performance impact and in many cases not needed to be enabled by default.
1. Create a node
2. On the right sidebar, click to add a menu link.
3. Export this content (menu link will be also exported)I would add this option to existing configuration form.
This should be developed for 1.3.x first and then migrated to 1.4.x (I can do it during review)
- @artycal opened merge request.
- Assigned to nginex
- Status changed to Needs review
over 1 year ago 8:33pm 7 August 2023 - First commit to issue fork.
- πΊπΈUnited States mortona2k Seattle
I needed this for 1.4.x so I did a reroll.
Why not focus efforts on 1.4.x instead of 1.3.x? Users could just update if they want the new features.
- Merge request !69Issue #3373378: Add a new setting to the configuration form to disable automatic menu link exporting during node export β (Open) created by mortona2k
- Issue was unassigned.
- πΊπΈUnited States mortona2k Seattle
I rebased 3373378-add-a-new-1.4.x onto 1.4.x to fix conflicts. Testrunner shows a phpunit failure so more adjustments are needed.
3373378-add-a-new-1.4.x-3363087-configure-export-settings contains MR69, plus work in progress on #3363087.
- First commit to issue fork.
- πΊπΈUnited States RoloDMonkey
I updated merge request 69 with the latest from 1.4.x and fixed the merge conflicts.
I am going to try using a patch from merge request 69 locally, but I will need other people to review this and see if it is ready.
- πΊπΈUnited States RoloDMonkey
I fixed a few minor issues left over from the merge request.
But, I did find some other problems.
First, the check for the new setting needs to be more careful, both where the default value is set in the form builder and in the
ContentSyncHelper->nodeAllowMenuLinkExportEnabled()
If the configuration is not set at all, then it should default to true. Even though it is supposed to be set to true in an update hook, there are cases where that update hook can immediately be overridden; for instance by importing the old config immediately after the update.Second, there are changes in
single_content_sync_preprocess_links__dropbutton__operations()
that look like they are from a different issue. Unless these are necessary for this issue to be resolved, they should not be included. - Status changed to Needs work
3 months ago 8:37pm 6 September 2024