| -rwxr-xr-x | tests/setup.sh | 13 | ||||
| -rwxr-xr-x | tests/t0101-index.sh | 2 | ||||
| -rwxr-xr-x | tests/t0104-tree.sh | 12 | ||||
| -rw-r--r-- | ui-shared.c | 20 |
4 files changed, 11 insertions, 36 deletions
diff --git a/tests/setup.sh b/tests/setup.sh index 1457dd5..e37306e 100755 --- a/tests/setup.sh +++ b/tests/setup.sh @@ -31,13 +31,6 @@ mkrepo() { git add file-$n git commit -m "commit $n" done - if test "$3" = "testplus" - then - echo "hello" >a+b - git add a+b - git commit -m "add a+b" - git branch "1+2" - fi cd $dir } @@ -47,7 +40,6 @@ setup_repos() mkdir -p trash/cache mkrepo trash/repos/foo 5 >/dev/null mkrepo trash/repos/bar 50 >/dev/null - mkrepo trash/repos/foo+bar 10 testplus >/dev/null cat >trash/cgitrc <<EOF virtual-root=/ cache-root=$PWD/trash/cache @@ -69,10 +61,6 @@ repo.path=$PWD/trash/repos/foo/.git repo.url=bar repo.path=$PWD/trash/repos/bar/.git repo.desc=the bar repo - -repo.url=foo+bar -repo.path=$PWD/trash/repos/foo+bar/.git -repo.desc=the foo+bar repo EOF } @@ -125,3 +113,4 @@ cgit_url() { CGIT_CONFIG="$PWD/trash/cgitrc" QUERY_STRING="url=$1" "$PWD/../cgit" } + diff --git a/tests/t0101-index.sh b/tests/t0101-index.sh index 07e39f9..445af6a 100755 --- a/tests/t0101-index.sh +++ b/tests/t0101-index.sh @@ -9,8 +9,6 @@ run_test 'find foo repo' 'grep -e "foo" trash/tmp' run_test 'find foo description' 'grep -e "\[no description\]" trash/tmp' run_test 'find bar repo' 'grep -e "bar" trash/tmp' run_test 'find bar description' 'grep -e "the bar repo" trash/tmp' -run_test 'find foo+bar repo' 'grep -e ">foo+bar<" trash/tmp' -run_test 'verify foo+bar link' 'grep -e "/foo+bar/" trash/tmp' run_test 'no tree-link' '! grep -e "foo/tree" trash/tmp' run_test 'no log-link' '! grep -e "foo/log" trash/tmp' diff --git a/tests/t0104-tree.sh b/tests/t0104-tree.sh index 0d62cc8..2516c72 100755 --- a/tests/t0104-tree.sh +++ b/tests/t0104-tree.sh @@ -18,16 +18,4 @@ run_test 'no line 2' ' grep -e "<a id=.n2. name=.n2. href=.#n2.>2</a>" trash/tmp ' -run_test 'generate foo+bar/tree' 'cgit_url "foo%2bbar/tree" >trash/tmp' - -run_test 'verify a+b link' ' - grep -e "/foo+bar/tree/a+b" trash/tmp -' - -run_test 'generate foo+bar/tree?h=1+2' 'cgit_url "foo%2bbar/tree&h=1%2b2" >trash/tmp' - -run_test 'verify a+b?h=1+2 link' ' - grep -e "/foo+bar/tree/a+b?h=1%2b2" trash/tmp -' - tests_done diff --git a/ui-shared.c b/ui-shared.c index a2f636c..c23bc75 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -221,21 +221,21 @@ static char *repolink(char *title, char *class, char *page, char *head, } else { html(ctx.cfg.script_name); html("?url="); - html_url_arg(ctx.repo->url); + html_attr(ctx.repo->url); if (ctx.repo->url[strlen(ctx.repo->url) - 1] != '/') html("/"); if (page) { - html_url_arg(page); + html(page); html("/"); if (path) - html_url_arg(path); + html_attr(path); } delim = "&"; } if (head && strcmp(head, ctx.repo->defbranch)) { html(delim); html("h="); - html_url_arg(head); + html_attr(head); delim = "&"; } return fmt("%s", delim); @@ -250,7 +250,7 @@ static void reporevlink(char *page, char *name, char *title, char *class, if (rev && strcmp(rev, ctx.qry.head)) { html(delim); html("id="); - html_url_arg(rev); + html_attr(rev); } html("'>"); html_txt(name); @@ -278,17 +278,17 @@ void cgit_log_link(char *name, char *title, char *class, char *head, if (rev && strcmp(rev, ctx.qry.head)) { html(delim); html("id="); - html_url_arg(rev); + html_attr(rev); delim = "&"; } if (grep && pattern) { html(delim); html("qt="); - html_url_arg(grep); + html_attr(grep); delim = "&"; html(delim); html("q="); - html_url_arg(pattern); + html_attr(pattern); } if (ofs > 0) { html(delim); @@ -333,13 +333,13 @@ void cgit_diff_link(char *name, char *title, char *class, char *head, if (new_rev && strcmp(new_rev, ctx.qry.head)) { html(delim); html("id="); - html_url_arg(new_rev); + html_attr(new_rev); delim = "&"; } if (old_rev) { html(delim); html("id2="); - html_url_arg(old_rev); + html_attr(old_rev); } html("'>"); html_txt(name); |