User filter for timeline

[yotamrubin@…]

Hello there,

I've just added the capability to filter the timeline by username. The change should have been trivial, but apparently the Timeline is rather difficult to get into. The process_request function is quite large, and has a very complex namespace. I did some dumb refactoring - moved out logical blocks in process_request to their own functions, but the logic remains the same. Additionally, the way I added the user filter is rather ad-hoc, since the current filter mechanism is a boolean one, either it's enabled, or not. Maybe next weekend I'll work on improving the filter mechanism.

[yotamrubin@…]

The userfilter patch

[cboos]

Ouch, sorry, but this is never going to make it into Trac, as development of the 0.10-stable branch is for bug fixes only.

The trunk (Trac 0.11dev) is in heavy development since months, and the timeline module has changed a lot, though your points would still be valid with the new version of the code, see source:trunk/trac/timeline/web_ui.py. Look also at the new approach we started in e.g. r4970 for handling requests.

So it would be very much appreciated if you reworked your patch for the trunk. Also, be aware that the web templates are quite different now: see TracDev/PortingFromClearSilverToGenshi.

[cboos]

As #1198 was the original feature request, with already many people CC'ed there, I'm closing this one as duplicate.

[anonymous]

Ok then. I've looked at the development version of trac. Seems like basic issues mentioned in my ticket appear in that version as well. I'll adapt my patch to the development version, and maybe resolve the filter issue as well.

Replying to cboos:

Ouch, sorry, but this is never going to make it into Trac, as development of the 0.10-stable branch is for bug fixes only. The trunk (Trac 0.11dev) is in heavy development since months, and the timeline module has changed a lot, though your points would still be valid with the new version of the code, see source:trunk/trac/timeline/web_ui.py. Look also at the new approach we started in e.g. r4970 for handling requests. So it would be very much appreciated if you reworked your patch for the trunk. Also, be aware that the web templates are quite different now: see TracDev/PortingFromClearSilverToGenshi.

[francois_at_granade.com]

The first patch attached to this bug was slightly buggy (it would break the other types of filters). This new patch doesn't break anything.