Changeset 14519

Timestamp:

8 Feb 2017, 09:22:23 (8 years ago)

Author:

Henrik Bettermann

Message:

Sort arguments in table of ReportsContainerPage.

Location:

main/waeup.kofa/trunk

Files:

• CHANGES.txt (modified) (1 diff)
• src/waeup/kofa/browser/reports.py (modified) (1 diff)

main/waeup.kofa/trunk/CHANGES.txt

@@ -4 +4 @@
 1.4.2.dev0 (unreleased)
 =======================
+
+* Sort arguments in table of `ReportsContainerPage`.
 
 * Add `entry_session` and `p_session` filter to Student Payment Statistics

main/waeup.kofa/trunk/src/waeup/kofa/browser/reports.py

@@ -89 +89 @@
             generator = queryUtility(IReportGenerator, name=gen_name)
             gen_title = translate(getattr(generator, 'title', _('Unknown')))
-            args = ', '.join([str(x) for x in job.kwargs['kw'].values()])
-            descr = '%s (%s)' % (gen_title, args)
+            # Sort arguments
+            sorted_items = sorted(
+                job.kwargs['kw'].items(), key=lambda value:value[0])
+            sorted_values = [i[1] for i in sorted_items]
+            arguments = ', '.join([str(x) for x in sorted_values])
+            descr = '%s (%s)' % (gen_title, arguments)
             status = job.finished and 'ready' or 'running'
             status = job.failed and 'FAILED' or status