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
7.0 set date_validity to proper value on copy() #248
Conversation
Convert class names to camelcase
''' | ||
By default, date_order is set to today on copy() | ||
cf addons/sale/sale.py. So we set date_validity to | ||
it's default value on copy() |
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.
it's -> its or this but not it's
@@ -59,6 +59,19 @@ def _default_date_validity(self, cr, uid, context=None): | |||
'date_validity': _default_date_validity, | |||
} | |||
|
|||
def copy(self, cr, uid, id, default=None, context=None): | |||
''' |
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.
Use double-quotes for docstring
Minor remark, but 👍 |
Could someone with the super-powers merge this please ? |
@alexis-via, we need 2 or more Up vote to merge. You could ask for label change to "needs review"(as OCA members follow through needs review label to check the PR status/review) code reviewed: 👍 |
Code review 👍 |
Syncing from upstream OCA/sale-workflow (12.0)
No description provided.