Merge lp:~therp-nl/ocb-addons/6.1-lp1070418 into lp:ocb-addons/6.1

Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 6799
Proposed branch: lp:~therp-nl/ocb-addons/6.1-lp1070418
Merge into: lp:ocb-addons/6.1
Diff against target: 14 lines (+2/-0)
1 file modified
account_budget/account_budget.py (+2/-0)
To merge this branch: bzr merge lp:~therp-nl/ocb-addons/6.1-lp1070418
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza Approve
Stefan Rijnhart (Opener) Approve
Holger Brunn (Therp) test, code review Approve
Review via email: mp+190116@code.launchpad.net

Description of the change

support view and consolidation children in budget positions

To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :
review: Approve (test, code review)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Tnx. This has been merged in ocb-addons/7.0 a long time ago, so this should be good.

review: Approve
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM. Works fine on 7.0.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_budget/account_budget.py'
2--- account_budget/account_budget.py 2011-12-31 07:57:20 +0000
3+++ account_budget/account_budget.py 2013-10-09 12:35:12 +0000
4@@ -113,8 +113,10 @@
5 result = 0.0
6 if context is None:
7 context = {}
8+ account_obj = self.pool.get('account.account')
9 for line in self.browse(cr, uid, ids, context=context):
10 acc_ids = [x.id for x in line.general_budget_id.account_ids]
11+ acc_ids = account_obj._get_children_and_consol(cr, uid, acc_ids, context=context)
12 if not acc_ids:
13 raise osv.except_osv(_('Error!'),_("The Budget '%s' has no accounts!") % str(line.general_budget_id.name))
14 date_to = line.date_to

Subscribers

People subscribed via source and target branches