-
-
Notifications
You must be signed in to change notification settings - Fork 369
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
[ADD] add module 'account_easy_reconcile' : v8 migration #110
Conversation
def _get_date(reconcile): | ||
if reconcile.last_history.date: | ||
return datetime.strptime(reconcile.last_history.date, | ||
DEFAULT_SERVER_DATETIME_FORMAT) |
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 fields.Datetime.from_string(reconcile.last_history.date)
|
||
@api.one | ||
@api.depends('reconcile_ids', 'reconcile_partial_ids') | ||
def _reconcile_line_ids(self): |
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.
Could be renamed _compute_reconcile_line_ids
or _get_reconcile_line_ids
for clarity
Still some cleaning to do |
[IMP] add translations [FIX] add decorator multi
ccc4e52
to
f559a5f
Compare
dummy, where, params = ml_obj._where_calc( | ||
cr, uid, eval(rec.filter), context=context).get_sql() | ||
eval(self.filter)).get_sql() |
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 change this to use safe_eval.
A PR on 7.0 to fix this is welcome too.
@gurneyalex fixed. |
👍 |
@damdam-s Thanks for the fixes, can you address the last 2-3 comments I have made? If you disagree just say it so 😄 |
@yvaucher done. oups ;) I forgot some of your comments. |
Thanks for the fixes 👍 merging... |
8.0 improve selection of invoices to pay, solves OCA#93
No description provided.