Skip to content

Conversation

@devEcommercePL
Copy link

Accept capital-initial “Yes/No” and “Tak/Nie” in YesNo::cast()

• Added “Yes” and “Tak” to TRUE_LIKE.
• Added “No” and “Nie” to FALSE_LIKE.

This makes the boolean cast more forgiving when consuming data that capitalises only the first letter, improving robustness of Shopware integrations without changing existing behaviour.

devEcommercePL and others added 7 commits July 2, 2025 23:27
Accept capital-initial “Yes/No” and “Tak/Nie” in YesNo::cast()

• Added “Yes” and “Tak” to TRUE_LIKE.  
• Added “No” and “Nie” to FALSE_LIKE.  

This makes the boolean cast more forgiving when consuming data that capitalises only the first letter, improving robustness of Shopware integrations without changing existing behaviour.
…g configurator settings

- When no persisted parent exists, generate/set a parent product ID (using Uuid::randomHex()) on $swData and use it as parentId for all variants.
- Fallback to variantListingConfig property check (instead of displayParent) to keep parent listing behavior compatible with newer Shopware versions.
- For variants that don’t yet exist in SW, create configurator settings from their options with productId = resolved parent ID.
- Add filterProductsNotMatchingSkuPattern (filter not products and parts)
- Hide attributes with hide metadata value
- Disable default filterable attribute for new attributes
- Use only selected brands (temporary)
- Category: inject the main category as tree root
- Images: leave the original name as a prefix
- Change default: DEFAULT_STOCK_VALUE to "0", DEFAULT_GROSS_PRICE to "99999", minPurchaseMapping to "1"
Copy link
Collaborator

@daniel-memo-ict daniel-memo-ict left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @devEcommercePL,

Unfortunately I can't accept this PR as it currently is. This PR now contains a lot more changes than just a few additions to the YesNo class. Please revert it to your initial commit.

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