Skip to content

Fix: add parent product to id#139

Merged
ahuininga-orisha merged 3 commits into
developfrom
fix-grouped-export-configurables
Apr 24, 2026
Merged

Fix: add parent product to id#139
ahuininga-orisha merged 3 commits into
developfrom
fix-grouped-export-configurables

Conversation

@ahuininga-orisha

Copy link
Copy Markdown
Collaborator

Description

This PR addresses an issue in the Grouped Export where simple products linked to multiple parent products (e.g., two or more configurables) caused the Tweakwise import to fail due to duplicate IDs.

Changes

To ensure unique identifiers, the logic for generating item IDs in the grouped export has been updated:

  • With Configurable Parent: The ID is now formatted as {simple_id}-{parent_id}.
  • Without Configurable Parent: The ID defaults to {simple_id}-{simple_id}.

Caution

Breaking Change
This modification breaks compatibility with events in the Magento 2 Tweakwise module. If you utilize the grouped export alongside these events, you must update the Magento 2 Tweakwise module accordingly.


How to Test

Verify the fix using a mix of Configurable, Bundle, Grouped, and Individually Visible Simple products.

1. Non-Grouped Export (Regression Testing)

Perform an export and publish. Verify that products are correctly displayed on the following pages:

  • Lister Page
  • Recommendations
  • Suggestions
  • Search
  • ALP

2. Grouped Export (Fix Verification)

  1. Assign a single Simple Product to two different Configurable Products.
  2. Run the export.
  3. Verify: Ensure the Tweakwise import completes successfully without duplicate ID errors.
  4. Publish and verify product visibility on:
    • Lister Page
    • Recommendations
    • Suggestions
    • Search
    • ALP

Comment thread Model/Helper.php Outdated
Comment thread Model/Helper.php Outdated
@ahuininga-orisha ahuininga-orisha merged commit 4739fb8 into develop Apr 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants