default_handler activates wrong Menuitem

[st.vogelsang@…]

1) to test add following line in configuration of trac trac.ini

[trac]
default_handler = TicketModule

2) open the default project page in browser, correct page will be shown but wrong menuitem is activated. (please have look at the screenshoot)

[st.vogelsang@…]

picture illustrates the error prone highlighting of the menu item bar

[Ryan Ollos <ryano@…>]

It looks to me that the underlying cause of this probably results in the same behavior we see when using the ​th:MenusPlugin: ​th:#4914.

[Ryan J Ollos]

Proposed fix in log:rjollos.git:t8791.

[Jun Omae]

I think that could use a compiled re object rather than calling re.match(...) each time.

-    ticket_path_re = r'/ticket/([0-9]+)$'
+    ticket_path_re = re.compile(r'/ticket/([0-9]+)$')

[Ryan J Ollos]

Replying to jomae:

I think that could use a compiled re object rather than calling re.match(...) each time.

Thanks, I will include the suggested change to ticket_path_re.

[Ryan J Ollos]

Committed to 1.0-stable in [13158:13159], merged to trunk in [13160].

And we are off and running with the 1.0.3 development :)