Merge lp:~openerp-dev/openobject-addons/6.0-opw-50934-ado into lp:openobject-addons/6.0
Proposed by
Amit Dodiya (OpenERP)
Status: | Merged | ||||
---|---|---|---|---|---|
Merge reported by: | nel | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~openerp-dev/openobject-addons/6.0-opw-50934-ado | ||||
Merge into: | lp:openobject-addons/6.0 | ||||
Diff against target: |
75 lines (+8/-31) 1 file modified
account/account_move_line.py (+8/-31) |
||||
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-50934-ado | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Priyesh (OpenERP) (community) | Approve | ||
Amit Dodiya (OpenERP) (community) | Needs Resubmitting | ||
Review via email: mp+84565@code.launchpad.net |
Description of the change
Hello,
[FIX] : UnboundLocalError: local variable 'period_id' referenced before assignment while creating a journal items.
6.0 series with demo data and gtk client
1). Accounting > Configuration > Journals > Purchase Journal
2). Enable "check date not in period" (this is important to reproduce the bug)
3). Account > Journal Items > New
4). in the new line:
* select any account
* name = /
* journal = Purchase Journal
5). click Save and you get the stack trace below
Code is back-ported from trunk.
Thanks,
Amit
To post a comment you must log in.
Hello Amit,
Code is working as per expectation but take care while using var name.
You can use 'line' instead of using 'l'.
Thanks for the fix,
Priyesh