#695: Add support for the CommitTicketUpdater component
--------------------------+-----------------------
  Reporter:  bsmithers    |      Owner:  rjollos
      Type:  enhancement  |     Status:  review
  Priority:  major        |  Milestone:  Release 8
 Component:  trac core    |    Version:  0.7.0
Resolution:               |   Keywords:  vcs hooks
--------------------------+-----------------------

Comment (by olemis):

 Replying to [comment:18 rjollos]:
 >  * [attachment:t695_r1545950_twill_write_html.diff] will be included in
 [trac:milestone:1.0.2 Trac 1.0.2] (trac:#11440, trac:#10018).
 >  * Part of
 [attachment:t695_r1534486_commit_ticket_update.tests.func.diff] committed
 to Trac in [trac 12434] for inclusion in Trac 1.0.2 (trac:#11440).
 >
 > I think we should see if it is possible to subclass
 `CommitTicketUpdater` rather than modifying it
 ([attachment:t695_r1534486_commit_ticket_update_checklinks.diff]). At
 least, this approach has worked well for other situations: trac:#11349,
 trac:#10755.

 Well , the fact is that the notion of multi-product has been there since
 the beginning . I mean , all previous patches are built under the
 assumption that the component is operating in a multi-product environment
 . Subclassing may be an option , and I'm not against doing so , but ...
 why bother if the component is already aware of multi-product architecture
 ?

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

Reply via email to