- Issue created by @Patryk Padus
- π΅π±Poland Patryk Padus
Screen showing how must be configured for this issue to be visible.
Path of screen where configuration was taken: /admin/commerce/config/product-variation-types - π΅π±Poland Patryk Padus
This is only one issue that block new stable version in my opinion when verified that issue is in dev branch from maintiner
- π΅π±Poland Patryk Padus
Possible temporary actions:
- Disabling module, adding variation and enabling won't bring error.
- Don't use multiple type (work fine on on type) - πͺπͺEstonia drugan
We need to add support for this Drupal Commerce feature:
- π΅π±Poland Patryk Padus
@drugan but you reference issue that is resolved. Is there some changes need?
- πͺπͺEstonia drugan
@Patryk Padus
The related issue is to look for the diff introduced so now we need to adjust Commerce Bulk for those changes.
Particularly, Drupal Commerce added some methods for a product type like
getVariationTypeIds()
.Before the current issue is fixed the Commerce Bulk should be disabled in case when multiple variation types are used on some of the product types. Otherwise Drupal Commerce will throw fatal error if it ,,sees,, an attempt to call
getVariationTypeId()
method on such kind of a product type. - Status changed to Needs work
about 1 year ago 5:02pm 20 August 2024 - π³πΏNew Zealand luke.stewart
luke.stewart β made their first commit to this issueβs fork.
- π³πΏNew Zealand luke.stewart
I've created an issue fork and pushed in some code. This is a WIP. We have it working on a site - but the site in question doesn't actually use this module at the moment, and when we attempted an uninstall we got different errors -> https://www.drupal.org/project/commerce_bulk/issues/3470872 π Unable to uninstall when using Layout Builder Active
In an attempt to get around this I just updated the calls to the method that were hitting the error and have picked the first id from the returned array as we now get a list instead of one id.
I'm sure there is probably a more appropriate way to solve this problem - however if like us you are just trying to get around this error combined with can't install due to layout builder, then the above fork should help.