Merge lp:~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code into lp:~credativ/openobject-addons/6.1

Proposed by Kinner Vachhani
Status: Merged
Merged at revision: 7046
Proposed branch: lp:~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code
Merge into: lp:~credativ/openobject-addons/6.1
Diff against target: 19 lines (+2/-0)
1 file modified
account/report/account_general_ledger.py (+2/-0)
To merge this branch: bzr merge lp:~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code
Reviewer Review Type Date Requested Status
credativ Pending
Review via email: mp+173208@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account/report/account_general_ledger.py'
--- account/report/account_general_ledger.py 2012-10-17 11:17:02 +0000
+++ account/report/account_general_ledger.py 2013-07-05 13:30:40 +0000
@@ -32,6 +32,7 @@
32from common_report_header import common_report_header32from common_report_header import common_report_header
3333
34from tools.translate import _34from tools.translate import _
35from operator import attrgetter
3536
36class general_ledger(report_sxw.rml_parse, common_report_header):37class general_ledger(report_sxw.rml_parse, common_report_header):
37 _name = 'report.account.general.ledger'38 _name = 'report.account.general.ledger'
@@ -131,6 +132,7 @@
131 res.append(child_account)132 res.append(child_account)
132 if not res:133 if not res:
133 return [account]134 return [account]
135 res.sort(key=attrgetter('code'))
134 return res136 return res
135137
136 def lines(self, account):138 def lines(self, account):

Subscribers

People subscribed via source and target branches

to all changes: