[Zope-Checkins] CVS: Zope/lib/python/Products/ZCTextIndex - ZCTextIndex.py:1.45

[Andreas Jung]

Update of /cvs-repository/Zope/lib/python/Products/ZCTextIndex
In directory cvs.zope.org:/tmp/cvs-serv27745/lib/python/Products/ZCTextIndex

Modified Files:
	ZCTextIndex.py 
Log Message:
re-added multiple attribute support - all unit tests pass


=== Zope/lib/python/Products/ZCTextIndex/ZCTextIndex.py 1.44 => 1.45 ===
--- Zope/lib/python/Products/ZCTextIndex/ZCTextIndex.py:1.44	Sun Feb  2 13:37:25 2003
+++ Zope/lib/python/Products/ZCTextIndex/ZCTextIndex.py	Sun Feb  2 14:18:11 2003
@@ -70,6 +70,8 @@
         # Arguments can be passed directly to the constructor or
         # via the silly "extra" record.
         self._fieldname = field_name or getattr(extra, 'doc_attr', '') or id
+        self._indexed_attrs = self._fieldname.split(',')
+        self._indexed_attrs = [ attr.strip() for attr in  self._indexed_attrs if attr ]
 
         lexicon_id = lexicon_id or extra.lexicon_id
         lexicon = getattr(caller, lexicon_id, None)
@@ -148,7 +150,20 @@
 
     ## Pluggable Index APIs ##
 
-    def index_object(self, docid, obj, threshold=None):
+    def index_object(self, documentId, obj, threshold=None):
+        """ wrapper to handle indexing of multiple attributes """
+
+        # needed for backward compatibility
+        try: fields = self._indexed_attrs
+        except: fields  = [ self._fieldname ]
+
+        res = 0
+        for attr in fields:
+            res += self._index_object(documentId, obj, threshold, attr) 
+
+        return res
+
+    def _index_object(self, docid, obj, threshold=None, attr=None):
         # XXX We currently ignore subtransaction threshold
         text = getattr(obj, self._fieldname, None)
         if text is None:
@@ -181,7 +196,7 @@
             return None
         tree = QueryParser(self.getLexicon()).parseQuery(query_str)
         results = tree.executeQuery(self.index)
-        return  results, (self._fieldname,)
+        return  results, (self.id,)
 
     def getEntryForObject(self, documentId, default=None):
         """Return the list of words indexed for documentId"""
@@ -217,15 +232,14 @@
 
     def getIndexSourceNames(self):
         """Return sequence of names of indexed attributes"""
-        return [self._fieldname]
+        try:
+            return self._indexed_attrs 
+        except:
+            return [self._fieldname]
 
     def getIndexType(self):
         """Return index type string"""
         return getattr(self, '_index_type', self._index_factory.__name__)
-
-    def getFieldName(self):
-        """Return indexed attribute name"""
-        return self._fieldname
 
     def getLexiconURL(self):
         """Return the url of the lexicon used by the index"""