TracBrowser 'o' key fails to follow .. link

[michelbriand@…]

The problem

In TracBrowser you can use 'j', 'k' and 'o' keys to navigate with the keyboard.

Unfortunately, on the '..' link to go up, the 'o' key makes the browser go to an undefined page.

It seems that the link is appended with the 'undefined' string.

Error on Trac site

This produce the following error on Trac (​http://trac.edgewall.org/) installed with unknown version (strangely removed from pages…):

Trac Error

<p class="message">No node /sandbox/undefined at revision 7803</p><p>You can <a href="/log/sandbox/undefined?rev=7803&amp;mode=path_history">search</a> in the repository history to see if that path existed but was later removed</p>

TracGuide — The Trac User and Administration Guide

Error on my setup (Trac 0.11.1)

This produce the following error my Trac installed with 0.11.1 version:

Trac Error

No node /trunk/doc/undefined at revision 430

You can search in the repository history to see if that path existed but was later removed

TracGuide — The Trac User and Administration Guide

[Christian Boos]

Yep, known issue. I'd be pleased if you could test the (last) patch on #7074, which fixes this problem among other things.

[Christian Boos]

Ah yes, about the "strange" error you saw here (Trac running 0.11stable-r7784): this is something which happened during 0.11-stable development, not in a released version. It is now fixed (r7794).

[anonymous]

If you think the patch fixes the problem, ok. But I cannot test a patch. As I'm in a production environment.

In which release can I except it to be available ?

[Christian Boos]

As the ticket says, 0.12.

The specific part which fixes the issue raised here can be backported to 0.11.3:

           } else {
-            window.location = selection.find('a.file').attr('href');
+            var href = selection.find('a.file').attr('href');
+            if (href) {
+              if (event.keyCode == 65)
+                href += '?annotate=blame';
+            } else {
+              href = selection.find('a.parent').attr('href');
+            }
+            if (href)
+              window.location = href;
           }

[Christian Boos]

(copy/pasted, also needs removal of the 'a' → blame part)

[Christian Boos]

Fixed in r7918.