From: gfawcett <gfawcett@6d9bc8c9-1ec2-4278-b937-99fde70a366f>
Date: Wed, 29 Dec 2010 15:42:54 +0000 (+0000)
Subject: sort item titles case-insensitively.
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=be8a067f89e5a99c6a898d13a4edf14eaf5db9fd;p=syrup%2Fmasslnc.git

sort item titles case-insensitively.

I also moved the STOPWORDS definition out of the function body, to save a few
CPU cycles.

git-svn-id: svn://svn.open-ils.org/ILS-Contrib/servres/trunk@1145 6d9bc8c9-1ec2-4278-b937-99fde70a366f
---

diff --git a/conifer/syrup/models.py b/conifer/syrup/models.py
index 43eeb17..182541b 100644
--- a/conifer/syrup/models.py
+++ b/conifer/syrup/models.py
@@ -304,19 +304,12 @@ class Site(BaseModel):
         subtree element, or None if there is no match.
         """
 
+        # TODO: internationalize the stopwords list.
+        STOPWORDS = set(['a', 'an', 'that', 'there', 'the', 'this'])
+
         def sort_title(item):
             """First cut of a stop words routine."""
-            # TODO: this needs to either be in its own file or in settings
-            stopwords = '''
-			a
-			an
-                        that
-			there
-                        the
-                        this
-                '''.split()
-
-            normal_text = [t for t in item.split() if t.lower() not in stopwords]
+            normal_text = [t for t in item.lower().split() if t not in STOPWORDS]
             return  " ".join(normal_text)
 
         items = self.items()