Banking Addons

Code review comment for lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-account_easy_reconcile-analytic-account

  1. bank-statement-reconcile-7.0-account_easy_reconcile-analytic-account
  2. Merge into bank-statement-reconcile-70
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

> Please put False default to avoid extrange values (browse_null or similar) on
> dictionary on l.32:
>
> 'analytic_account_id': (rec_method.analytic_account_id and
> rec_method.analytic_account_id.id or False),
>
> Regards.

The code is similar to the other lines around. Also, what would be the problem to have a browse_null here?

« Back to merge proposal