aboutsummaryrefslogtreecommitdiffstats
path: root/ui-tag.c
diff options
context:
space:
mode:
authorJohn Keeping <john@keeping.me.uk>2015-08-14 12:47:19 +0100
committerJason A. Donenfeld <Jason@zx2c4.com>2015-08-14 15:46:51 +0200
commit696a33b66f8638d5c3e0464d66760c6d60ff24a5 (patch)
tree8da0938fb4cc6ac6dc487dfc9a117cc1f910de3e /ui-tag.c
parentsummary: move layout into page function (diff)
downloadcgit-696a33b66f8638d5c3e0464d66760c6d60ff24a5.tar.xz
cgit-696a33b66f8638d5c3e0464d66760c6d60ff24a5.zip
tag: move layout into page function
This also allows us to return proper HTTP error codes when something goes wrong. Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'ui-tag.c')
-rw-r--r--ui-tag.c13
1 files changed, 10 insertions, 3 deletions
diff --git a/ui-tag.c b/ui-tag.c
index c1d1738..0afc663 100644
--- a/ui-tag.c
+++ b/ui-tag.c
@@ -52,20 +52,24 @@ void cgit_print_tag(char *revname)
strbuf_addf(&fullref, "refs/tags/%s", revname);
if (get_sha1(fullref.buf, sha1)) {
- cgit_print_error("Bad tag reference: %s", revname);
+ cgit_print_error_page(404, "Not found",
+ "Bad tag reference: %s", revname);
goto cleanup;
}
obj = parse_object(sha1);
if (!obj) {
- cgit_print_error("Bad object id: %s", sha1_to_hex(sha1));
+ cgit_print_error_page(500, "Internal server error",
+ "Bad object id: %s", sha1_to_hex(sha1));
goto cleanup;
}
if (obj->type == OBJ_TAG) {
tag = lookup_tag(sha1);
if (!tag || parse_tag(tag) || !(info = cgit_parse_tag(tag))) {
- cgit_print_error("Bad tag object: %s", revname);
+ cgit_print_error_page(500, "Internal server error",
+ "Bad tag object: %s", revname);
goto cleanup;
}
+ cgit_print_layout_start();
html("<table class='commit-info'>\n");
htmlf("<tr><td>tag name</td><td>");
html_txt(revname);
@@ -93,7 +97,9 @@ void cgit_print_tag(char *revname)
print_download_links(revname);
html("</table>\n");
print_tag_content(info->msg);
+ cgit_print_layout_end();
} else {
+ cgit_print_layout_start();
html("<table class='commit-info'>\n");
htmlf("<tr><td>tag name</td><td>");
html_txt(revname);
@@ -104,6 +110,7 @@ void cgit_print_tag(char *revname)
if (ctx.repo->snapshots)
print_download_links(revname);
html("</table>\n");
+ cgit_print_layout_end();
}
cleanup: