]> granicus.if.org Git - cgit/commitdiff
ui-log.c: do not link from age column
authorLars Hjemli <hjemli@gmail.com>
Thu, 2 Jun 2011 10:30:26 +0000 (10:30 +0000)
committerLars Hjemli <hjemli@gmail.com>
Thu, 2 Jun 2011 10:30:26 +0000 (10:30 +0000)
The link url wasn't properly escaped, and since the link was identical
to the one used on the commit message it didn't serve any special purpose.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
tests/t0103-log.sh
ui-log.c

index def5c1812ac82017193a8d2a1be4aa22b8e39ca5..75dd761a0b51d6e9cde61b6fdebf019e43b81daa 100755 (executable)
@@ -18,7 +18,7 @@ run_test 'generate "with%20space/log?qt=grep&q=commit+1"' '
 run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
 run_test 'find link with %20 in path' 'grep -e "/with%20space/log/?qt=grep" trash/tmp'
 run_test 'find link with + in arg' 'grep -e "/log/?qt=grep&q=commit+1" trash/tmp'
-run_test BUG 'no links with space in path' '! grep -e "href=./with space/" trash/tmp'
+run_test 'no links with space in path' '! grep -e "href=./with space/" trash/tmp'
 run_test 'no links with space in arg' '! grep -e "q=commit 1" trash/tmp'
 run_test 'commit 2 is not visible' '! grep -e "commit 2" trash/tmp'
 
index 8add66aa0f41c16081a2ead29ba2c19a8f41ace5..15ed6a3f63dabea9cad60ae645b1450178dc274e 100644 (file)
--- a/ui-log.c
+++ b/ui-log.c
@@ -95,7 +95,6 @@ void show_commit_decorations(struct commit *commit)
 void print_commit(struct commit *commit, struct rev_info *revs)
 {
        struct commitinfo *info;
-       char *tmp;
        int cols = revs->graph ? 3 : 2;
        struct strbuf graphbuf = STRBUF_INIT;
        struct strbuf msgbuf = STRBUF_INIT;
@@ -130,11 +129,7 @@ void print_commit(struct commit *commit, struct rev_info *revs)
        }
        else {
                html("<td>");
-               tmp = fmt("id=%s", sha1_to_hex(commit->object.sha1));
-               tmp = cgit_fileurl(ctx.repo->url, "commit", ctx.qry.vpath, tmp);
-               html_link_open(tmp, NULL, NULL);
                cgit_print_age(commit->date, TM_WEEK * 2, FMT_SHORTDATE);
-               html_link_close();
                html("</td>");
        }
 
@@ -172,11 +167,7 @@ void print_commit(struct commit *commit, struct rev_info *revs)
 
        if (revs->graph) {
                html("</td><td>");
-               tmp = fmt("id=%s", sha1_to_hex(commit->object.sha1));
-               tmp = cgit_fileurl(ctx.repo->url, "commit", ctx.qry.vpath, tmp);
-               html_link_open(tmp, NULL, NULL);
                cgit_print_age(commit->date, TM_WEEK * 2, FMT_SHORTDATE);
-               html_link_close();
        }
 
        if (ctx.repo->enable_log_filecount) {