incubator-bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <bloodhound-...@incubator.apache.org>
Subject Re: [Apache Bloodhound] #23: quick ticket should check for correct ticket module to use
Date Tue, 24 Apr 2012 07:08:02 GMT
#23: quick ticket should check for correct ticket module to use
------------------------+-------------------------------------
  Reporter:  gjm        |      Owner:  gjm
      Type:  defect     |     Status:  closed
  Priority:  major      |  Milestone:  RC1 for initial release
 Component:  dashboard  |    Version:
Resolution:  fixed      |   Keywords:
------------------------+-------------------------------------

Comment (by olemis):

 Sorry for the delay but it wasn't until now that I could get this done . I
 had to solve critical issues related to #39 first ''':-/''' .
 [attachment:t23_r1328689_flexible_qct.diff​ Attached patch] is built on
 top of [attachment:ticket:39:t39_r1328689_indexerror.diff other patch
 submitted for #39] , hence you'll need to apply that one in first place .
 Recall it's built on top of r1328689 . New features are :

   - Extension point interface `IQuickCreateTicketBackend` .
   - `DefaultQuickCreateTicketBackend` component implementing the former
     so as to provide a simple way to create tickets by using Trac
     built-in components. Used by default in case no other choice
     is configured in ''trac.ini''
   - `ExtensionOption` property used to select active backend.

 It's not meant to replace your previous implementation but to make the
 solution flexible and configurable by using Trac built-in mechanisms. The
 idea is :

   - Implement a similar backend in multi-product plugin
     (i.e. reuse your previous code ''';)''' .
   - Make the installer to set related config option in ''trac.ini''
     so as to use multi-product backend since the beginning.


 Nonetheless I'll review your implementation in order to determine whether
 further enhancements may be needed so as to make some room for features
 committed recently.

 '''TODO''': Maybe it'd be nice to research whether this can be done using
 ticket change listeners or any other similar existing mechanism.

 '''PS''' : feel free to reopen the ticket if appropriate . We could also
 open a separate ticket to discuss and work on these enhancements ''';)'''

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/23#comment:9>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound (incubating) issue tracker

Mime
View raw message