| -rw-r--r-- | cgit.c | 2 | ||||
| -rw-r--r-- | cgit.h | 2 | ||||
| -rw-r--r-- | ui-commit.c | 3 | ||||
| -rw-r--r-- | ui-log.c | 7 | ||||
| -rw-r--r-- | ui-shared.c | 16 | ||||
| -rw-r--r-- | ui-summary.c | 18 | ||||
| -rw-r--r-- | ui-tree.c | 1 |
7 files changed, 18 insertions, 31 deletions
@@ -100,7 +100,7 @@ static void cgit_print_repo_page(struct cacheitem *item) cgit_print_tree(cgit_query_sha1, cgit_query_path); break; case CMD_COMMIT: - cgit_print_commit(cgit_query_sha1); + cgit_print_commit(cgit_query_head); break; case CMD_DIFF: cgit_print_diff(cgit_query_head, cgit_query_sha1, cgit_query_sha2, @@ -205,8 +205,6 @@ extern void cgit_tree_link(char *name, char *title, char *class, char *head, char *rev, char *path); extern void cgit_log_link(char *name, char *title, char *class, char *head, char *rev, char *path); -extern void cgit_commit_link(char *name, char *title, char *class, char *head, - char *rev); extern void cgit_print_error(char *msg); extern void cgit_print_date(time_t secs, char *format); diff --git a/ui-commit.c b/ui-commit.c index 8864fc6..885f870 100644 --- a/ui-commit.c +++ b/ui-commit.c @@ -156,9 +156,6 @@ void cgit_print_commit(const char *hex) char *tmp; int i; - if (!hex) - hex = cgit_query_head; - if (get_sha1(hex, sha1)) { cgit_print_error(fmt("Bad object id: %s", hex)); return; @@ -31,8 +31,11 @@ void print_commit(struct commit *commit) html("<tr><td>"); cgit_print_age(commit->date, TM_WEEK * 2, FMT_SHORTDATE); html("</td><td>"); - cgit_commit_link(info->subject, NULL, NULL, cgit_query_head, - sha1_to_hex(commit->object.sha1)); + char *qry = fmt("h=%s", sha1_to_hex(commit->object.sha1)); + char *url = cgit_pageurl(cgit_query_repo, "commit", qry); + html_link_open(url, NULL, NULL); + html_ntxt(cgit_max_msg_len, info->subject); + html_link_close(); if (cgit_repo->enable_log_filecount) { files = 0; lines = 0; diff --git a/ui-shared.c b/ui-shared.c index 71c899a..64ee79c 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -136,8 +136,8 @@ static char *repolink(char *title, char *class, char *page, char *head, return fmt("%s", delim); } -static void reporevlink(char *page, char *name, char *title, char *class, - char *head, char *rev, char *path) +static char *reporevlink(char *page, char *name, char *title, char *class, + char *head, char *rev, char *path) { char *delim; @@ -164,18 +164,6 @@ void cgit_log_link(char *name, char *title, char *class, char *head, reporevlink("log", name, title, class, head, rev, path); } -void cgit_commit_link(char *name, char *title, char *class, char *head, - char *rev) -{ - if (strlen(name) > cgit_max_msg_len && cgit_max_msg_len >= 15) { - name[cgit_max_msg_len] = '\0'; - name[cgit_max_msg_len - 1] = '.'; - name[cgit_max_msg_len - 2] = '.'; - name[cgit_max_msg_len - 3] = '.'; - } - reporevlink("commit", name, title, class, head, rev, NULL); -} - void cgit_print_date(time_t secs, char *format) { char buf[64]; diff --git a/ui-summary.c b/ui-summary.c index 03dd078..29b76e3 100644 --- a/ui-summary.c +++ b/ui-summary.c @@ -15,10 +15,8 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1, { struct commit *commit; struct commitinfo *info; - char buf[256]; - char *ref; + char buf[256], *url; - ref = xstrdup(refname); strncpy(buf, refname, sizeof(buf)); commit = lookup_commit(sha1); // object is not really parsed at this point, because of some fallout @@ -27,13 +25,17 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1, if (commit && !parse_commit(commit)){ info = cgit_parse_commit(commit); html("<tr><td>"); - cgit_log_link(ref, NULL, NULL, ref, NULL, NULL); + cgit_log_link(refname, NULL, NULL, refname, NULL, NULL); html("</td><td>"); cgit_print_age(commit->date, -1, NULL); html("</td><td>"); html_txt(info->author); html("</td><td>"); - cgit_commit_link(info->subject, NULL, NULL, ref, NULL); + url = cgit_pageurl(cgit_query_repo, "commit", + fmt("h=%s", sha1_to_hex(sha1))); + html_link_open(url, NULL, NULL); + html_ntxt(cgit_max_msg_len, info->subject); + html_link_close(); html("</td></tr>\n"); cgit_free_commitinfo(info); } else { @@ -43,7 +45,6 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1, htmlf("*** bad ref %s ***", sha1_to_hex(sha1)); html("</td></tr>\n"); } - free(ref); return 0; } @@ -53,9 +54,8 @@ static void cgit_print_object_ref(struct object *obj) char *page, *arg, *url; if (obj->type == OBJ_COMMIT) { - cgit_commit_link(fmt("commit %s", sha1_to_hex(obj->sha1)), NULL, NULL, - cgit_query_head, sha1_to_hex(obj->sha1)); - return; + page = "commit"; + arg = "h"; } else if (obj->type == OBJ_TREE) { page = "tree"; arg = "id"; @@ -58,6 +58,7 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen, char *fullpath; enum object_type type; unsigned long size = 0; + char *url, *qry; name = xstrdup(pathname); fullpath = fmt("%s%s%s", cgit_query_path ? cgit_query_path : "", |