changeset 2107:15eafde0d14d

use constants for ticket meta keys
author Cheer Xiao <xiaqqaix@gmail.com>
date Thu, 02 May 2013 15:09:42 +0800
parents 02791a565ff0
children 55694eb202a3
files MoinMoin/constants/keys.py MoinMoin/storage/middleware/indexing.py
diffstat 2 files changed, 19 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/MoinMoin/constants/keys.py	Thu May 02 15:07:05 2013 +0800
+++ b/MoinMoin/constants/keys.py	Thu May 02 15:09:42 2013 +0800
@@ -104,6 +104,16 @@
 # keys for blog entries
 PTIME = "ptime"
 
+# keys for tickets
+EFFORT = "effort"
+DIFFICULTY = "difficulty"
+SEVERITY = "severity"
+PRIORITY = "priority"
+ASSIGNED_TO = "assigned_to"
+SUPERSEDED_BY = "superseded_by"
+DEPENDS_ON = "depends_on"
+CLOSED = "closed"
+
 # index names
 LATEST_REVS = 'latest_revs'
 ALL_REVS = 'all_revs'
--- a/MoinMoin/storage/middleware/indexing.py	Thu May 02 15:07:05 2013 +0800
+++ b/MoinMoin/storage/middleware/indexing.py	Thu May 02 15:09:42 2013 +0800
@@ -327,14 +327,14 @@
 
         # XXX This is a highly adhoc way to support indexing of ticket items.
         ticket_fields = {
-            'effort': NUMERIC(stored=True),
-            'difficulty': NUMERIC(stored=True),
-            'severity': NUMERIC(stored=True),
-            'priority': NUMERIC(stored=True),
-            'assigned_to': ID(stored=True),
-            'superseded_by': ID(stored=True),
-            'depends_on': ID(stored=True),
-            'closed': BOOLEAN(stored=True),
+            EFFORT: NUMERIC(stored=True),
+            DIFFICULTY: NUMERIC(stored=True),
+            SEVERITY: NUMERIC(stored=True),
+            PRIORITY: NUMERIC(stored=True),
+            ASSIGNED_TO: ID(stored=True),
+            SUPERSEDED_BY: ID(stored=True),
+            DEPENDS_ON: ID(stored=True),
+            CLOSED: BOOLEAN(stored=True),
         }
         latest_revs_fields.update(**ticket_fields)
 
@@ -702,7 +702,7 @@
             # username:JoeDoe searches for revisions modified by JoeDoe
             username=userid_pseudo_field_factory(USERID),
             # assigned:JoeDoe searches for tickets assigned to JoeDoe
-            assigned=userid_pseudo_field_factory('assigned_to'),  # XXX should be ASSIGNED_TO
+            assigned=userid_pseudo_field_factory(ASSIGNED_TO),
         )))
         return qp