Thread (5 messages) flat view 5 messages, 3 authors, 2016-06-15

Re: [PATCH 1/2] gitweb: Fix file links in "grep" search

From: Thomas Perl <hidden>
Date: 2016-06-15 22:52:47

Possibly related (same subject, not in this thread)

Hi,

2012/1/5 Jakub Narebski [off-list ref]:
There were two bugs in generating file links (links to "blob" view),
one hidden by the other.  The correct way of generating file link is

       href(action=>"blob", hash_base=>$co{'id'},
            file_name=>$file);

It was $co{'hash'} (this key does not exist, and therefore this is
undef), and 'hash' instead of 'hash_base'.
[...]
Thomas, could you check if this fixes your issue?
Sorry for taking a bit longer to respond on this one, but I just got
around to test all problematic cases that I described with the patch
applied - it fixes the problem for me (i.e. I can successfully grep in
non-master branches and then clicking the link brings me to the right
location).

As far as I'm concerned, the patch can be applied and fixes the bug.

Thanks for the quick fix! :)
Thomas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help