svn commit: r1585890 - /bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py

2014-04-08 Thread rjollos 
Author: rjollos
Date: Wed Apr  9 05:00:25 2014
New Revision: 1585890

URL: http://svn.apache.org/r1585890
Log:
0.8dev: Added Whoosh version to //SystemInfo// table. Refs #794.

Patch by devenderbindal.

Modified:
bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py

Modified: bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py?rev=1585890r1=1585889r2=1585890view=diff
==
--- bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py (original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/whoosh_backend.py Wed Apr  9 
05:00:25 2014
@@ -24,6 +24,7 @@ from datetime import datetime
 
 from trac.core import Component, implements, TracError
 from trac.config import Option, IntOption
+from trac.env import ISystemInfoProvider
 from trac.util.datefmt import utc
 
 import whoosh
@@ -44,7 +45,7 @@ class WhooshBackend(Component):
 
 Implements Whoosh SearchBackend interface
 
-implements(ISearchBackend)
+implements(ISearchBackend, ISystemInfoProvider)
 
 index_dir_setting = Option(
 BHSEARCH_CONFIG_SECTION,
@@ -124,7 +125,13 @@ class WhooshBackend(Component):
 else:
 self.index = None
 
-#ISearchBackend methods
+# ISystemInfoProvider methods
+
+def get_system_info(self):
+yield 'Whoosh', whoosh.versionstring()
+
+# ISearchBackend methods
+
 def start_operation(self):
 return self._create_writer()

Re: [Apache Bloodhound] #794: Whoosh version should be shown under System Information

2014-04-08 Thread Apache Bloodhound 
#794: Whoosh version should be shown under System Information
--+
  Reporter:  rjollos  |  Owner:  devenderbindal
  Type:  defect   | Status:  closed
  Priority:  major|  Milestone:  Release 8
 Component:  search   |Version:
Resolution:  fixed|   Keywords:  starter
--+
Changes (by rjollos):

 * status:  new = closed
 * resolution:   = fixed


Comment:

 A few small comments:
  * We don't need to wrap `import whoosh` in a `try`/`except` since the
 entire module should fail to load when whoosh is not present.
  * It appears you are not developing against the latest version of the
 trunk.

 
 (In [1585890])

 0.8dev: Added Whoosh version to //SystemInfo// table. Refs #794.

 Patch by devenderbindal.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/794#comment:4
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker

[Apache Bloodhound] New user registration: sunsongxp

2014-04-08 Thread Apache Bloodhound 
New user registration for user sunsongxp

--
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker