-
-
Notifications
You must be signed in to change notification settings - Fork 665
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
Migration of account_invoice_blocking to 10.0 #197
Migration of account_invoice_blocking to 10.0 #197
Conversation
[ADD] account_invoice_blocking
from openerp.tests.common import TransactionCase | ||
from openerp import workflow | ||
from odoo.tests.common import TransactionCase | ||
from odoo import workflow |
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.
Workflow in v10?
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.
Some minor comments.
LGTM (code review).
_inherit = 'account.invoice' | ||
|
||
@api.model | ||
def _get_move_line(self, invoice): |
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.
Please convert this method from @api.model
to @api.multi
+ self.ensure_one()
without the self recordset argument, to follow standard changes since v10.
('invoice_id', '=', invoice.id)]) | ||
|
||
@api.model | ||
def _update_blocked(self, invoice, 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.
Please convert this method from @api.model
to @api.multi
without the self recordset argument, to follow standard changes since v10.
|
||
Usage | ||
===== | ||
|
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.
As better you describe how to use the module, as much more easier is to test it, so more people will be able to review it. It's just a recommendation. I rebuilding runbot for testing
e065573
to
6ad202d
Compare
@sylvain-garancher @rafaelbn Thanks for the review |
Travis error is unrelated #269 |
triggering a rebuild |
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.
Functional and code review 👍
No description provided.