credativ team mailing list archive

[Kinner Vachhani <kin.vachhani@xxxxxxxxx>]

Kinner Vachhani has proposed merging lp:~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code into lp:~credativ/openobject-addons/6.1.

Requested reviews:
  credativ (credativ)
Related bugs:
  Bug #1198134 in OpenERP Addons: "General Ledger report does not sort by account code"
  https://bugs.launchpad.net/openobject-addons/+bug/1198134

For more details, see:
https://code.launchpad.net/~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code/+merge/173208
-- 
https://code.launchpad.net/~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code/+merge/173208
Your team credativ is requested to review the proposed merge of lp:~credativ/openobject-addons/6.1-lp1198134-General-Ledger-report-does-not-sort-by-account-code into lp:~credativ/openobject-addons/6.1.

=== 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 @@
 from common_report_header import common_report_header
 
 from tools.translate import _
+from operator import attrgetter
 
 class general_ledger(report_sxw.rml_parse, common_report_header):
     _name = 'report.account.general.ledger'
@@ -131,6 +132,7 @@
                 res.append(child_account)
         if not res:
             return [account]
+        res.sort(key=attrgetter('code'))
         return res
 
     def lines(self, account):