Culapov Andrei wrote: > posted patch to trac > http://trac.edgewall.org/ticket/7293 > >
Thanks, but I was already reading it ;-) Looks quite good to me. -- Christian > On Thu, May 29, 2008 at 4:33 PM, Christian Boos <[EMAIL PROTECTED]> wrote: > >> Culapov Andrei wrote: >> >>> Hi Richard, >>> we implemented something exactly like this. Take a look >>> https://projects.optaros.com/trac/oforge/report/9 >>> >>> >> 401 :-) >> >> Other than that, the code looks interesting and even more so the OForge >> project. >> However, for this particular feature, I think it would have made more >> sense to write a patch against Trac, instead of writing a request filter >> + stream filter, as (to me at least) this issue looks more like a defect >> of the report subsystem rather than an optional feature: if you don't >> set the PRIORITY dynamic var, Trac reports an error page with no way to >> specify that value, which is not terribly useful. A simple list of input >> boxes for the missing variables would do, and plugins could still be >> implemented if anything more fancy is needed. >> >> -- Christian >> >> > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Trac Development" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/trac-dev?hl=en -~----------~----~----~----~------~----~------~--~---
