You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by Apache Bloodhound <de...@bloodhound.apache.org> on 2014/03/28 06:17:36 UTC

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

#794: Whoosh version should be shown under System Information
---------------------+----------------------------
 Reporter:  rjollos  |      Owner:  devenderbindal
     Type:  defect   |     Status:  new
 Priority:  major    |  Milestone:  next 0.x
Component:  search   |    Version:
 Keywords:  starter  |
---------------------+----------------------------
 The Whoosh version should be shown under //System Information// on the
 `about.html` and `error.html` pages.

 [[Image(SystemInformation.png)]]

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/794>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

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

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#794: Whoosh version should be shown under System Information
----------------------+----------------------------
  Reporter:  rjollos  |      Owner:  devenderbindal
      Type:  defect   |     Status:  new
  Priority:  major    |  Milestone:  next 0.x
 Component:  search   |    Version:
Resolution:           |   Keywords:  starter
----------------------+----------------------------

Comment (by devenderbindal):

 Initial Patch Submitted only for /about page at
 [https://github.com/dev9226134/bloodhound/commit/cabf781f40315746da62fa31f46b158a521c2c46
 Github]

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

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

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#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

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

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#794: Whoosh version should be shown under System Information
----------------------+----------------------------
  Reporter:  rjollos  |      Owner:  devenderbindal
      Type:  defect   |     Status:  new
  Priority:  major    |  Milestone:  Release 8
 Component:  search   |    Version:
Resolution:           |   Keywords:  starter
----------------------+----------------------------
Changes (by rjollos):

 * milestone:  next 0.x => Release 8


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

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

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#794: Whoosh version should be shown under System Information
----------------------+----------------------------
  Reporter:  rjollos  |      Owner:  devenderbindal
      Type:  defect   |     Status:  new
  Priority:  major    |  Milestone:  next 0.x
 Component:  search   |    Version:
Resolution:           |   Keywords:  starter
----------------------+----------------------------

Comment (by rjollos):

 Some implementation hints:
  * It looks like the Whoosh version can be retrieved by:
  {{{#!python
 >>> import whoosh
 >>> whoosh.__version__
 (2, 6, 0)
 }}}
  * `ISystemInfoProvider` from `trac.env` needs to be implemented in
 `whoosh_backend.py`. There are numerous example in the codebase and also
 [http://www.edgewall.org/docs/branches-1.0-stable/html/api/trac_env.html#trac.env.ISystemInfoProvider
 API documentation] on the interface.
  {{{#!sh
 $ grep -r "trac.env.ISystemInfoProvider\|import.*ISystemInfoProvider"
 --exclude-dir=.svn
 bloodhound_multiproduct/multiproduct/env.py:
 implements(trac.env.ISystemInfoProvider, IPermissionRequestor)
 trac/tracopt/mimeview/silvercity.py:from trac.env import
 ISystemInfoProvider
 trac/tracopt/versioncontrol/svn/svn_fs.py:from trac.env import
 ISystemInfoProvider
 trac/doc/api/trac_env.rst:.. autoclass :: trac.env.ISystemInfoProvider
 trac/doc/api/trac_env.rst:   See also
 :extensionpoints:`trac.env.ISystemInfoProvider`
 trac/trac/mimeview/pygments.py:from trac.env import ISystemInfoProvider
 trac/trac/mimeview/rst.py:from trac.env import ISystemInfoProvider
 trac/trac/web/chrome.py:from trac.env import IEnvironmentSetupParticipant,
 ISystemInfoProvider
 }}}

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