-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix get rules from order with option value instead of option code #4269
Fix get rules from order with option value instead of option code #4269
Conversation
@@ -698,10 +698,10 @@ def description(self): | |||
value = attribute.value | |||
if isinstance(value, list): | |||
ops.append( | |||
"%s = '%s'" % (attribute.type, (", ".join([str(v) for v in value]))) | |||
"%s = '%s'" % (attribute.option.name, (", ".join([str(v) for v in value]))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be attribute.value because the option could be empty
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When you look at another abstract_models. For example at baskets module, it is also like i changed.
def description(self): |
@@ -55,7 +55,7 @@ def setUp(self): | |||
upc="1234", | |||
) | |||
product.attr.weight = 3 | |||
product.full_clean() | |||
# product.full_clean() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oeps
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #4269 +/- ##
==========================================
+ Coverage 87.84% 88.35% +0.51%
==========================================
Files 291 293 +2
Lines 16145 16083 -62
==========================================
+ Hits 14182 14210 +28
+ Misses 1963 1873 -90
|
This fixes showing order lines with the option value instead of the code.
#4263