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
1=== modified file 'account/report/account_general_ledger.py'
2--- account/report/account_general_ledger.py 2012-10-17 11:17:02 +0000
3+++ account/report/account_general_ledger.py 2013-07-05 13:30:40 +0000
4@@ -32,6 +32,7 @@
5 from common_report_header import common_report_header
6
7 from tools.translate import _
8+from operator import attrgetter
9
10 class general_ledger(report_sxw.rml_parse, common_report_header):
11 _name = 'report.account.general.ledger'
12@@ -131,6 +132,7 @@
13 res.append(child_account)
14 if not res:
15 return [account]
16+ res.sort(key=attrgetter('code'))
17 return res
18
19 def lines(self, account):

Subscribers

People subscribed via source and target branches

to all changes: