Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
We need to remove the discount inline conditions from the module. This module expose a new field type for Drupal, and should stay to this role. For the time being, we have included the discount inline conditions in the module, but those conditions have to move into the commerce_discount since only this module is using them.
Comment | File | Size | Author |
---|---|---|---|
#13 | Capture.PNG | 6.45 KB | zmove |
#2 | 2129997-remove-discount-inline-conditions-2.patch | 32.56 KB | mglaman |
Comments
Comment #1
jkuma CreditAttribution: jkuma commentedComment #2
mglamanHere is the patch that removes Commerce Discount related inline conditions.
Related patch to Commerce Discount will be created next and linked back to here.
Comment #3
mglamanAdding reference to #2244969: Add inline conditions from inline_conditions into module. in Commerce Discount queue.
Comment #4
Ralt CreditAttribution: Ralt commentedI've tested applying both #2244969 and #2129997 succesfully. Everything I've tested keeps working. Commits here (for inline_conditions) and here (for commerce_discount) if one wants to cherry-pick the commits.
Comment #5
Ralt CreditAttribution: Ralt commentedComment #6
mglamanThanks Ralt!
Only request to maintainers - if you cherry-pick off of the GitHub fork, there isn't commit authorship :/ Not an end of the world thing, but one of those little things that makes it feel worth it.
Comment #8
joelpittetThis function isn't being used, did it sneak in by accident?
Comment #10
jkuma CreditAttribution: jkuma commentedHello joel,
I've removed that old function, thanks for the catch!
Comment #11
joelpittetNo problem, @goldorak thanks for the commit.
Comment #13
zmove CreditAttribution: zmove commentedHave a problem when I try to update a rule with an operator :
And see the screenshot, the operator list is empty. Can be considered as major issue if, as it seems to be, it break all rules with operators.
Comment #14
joelpittet@zmove please open a new issue for that, and to get the most, provide steps to reproduce the issue.
Comment #15
zmove CreditAttribution: zmove commentedOk sorry, issue created : #2534972: Rules lost its operator with the latest update
Comment #16
joelpittetThank you zmove for cross posting the issue back here.