author | Lars Hjemli <hjemli@gmail.com> | 2007-06-17 11:57:51 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-06-17 11:57:51 (UTC) |
commit | 48c487d72daef7e71683a85f775db8d36ab20341 (patch) (unidiff) | |
tree | 8770334d7c77bd3e81f42ee12550e6db5febd4d2 /ui-tree.c | |
parent | 426032f767763b46f462de51d1ebded34f671d79 (diff) | |
download | cgit-48c487d72daef7e71683a85f775db8d36ab20341.zip cgit-48c487d72daef7e71683a85f775db8d36ab20341.tar.gz cgit-48c487d72daef7e71683a85f775db8d36ab20341.tar.bz2 |
Add git_log_link() and fix bug in generic repolink function
The generic repolink function compared head with cgit_query_head, which
almost always would be the same pointer. The test now compares with
repo.defbranch, which is the wanted behavour.
Bug discovered while adding cgit_log_link(), so this commit also contain
that change.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-tree.c | 12 |
1 files changed, 4 insertions, 8 deletions
@@ -87,24 +87,20 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen, | |||
87 | cgit_tree_link(name, NULL, "ls-dir", cgit_query_head, | 87 | cgit_tree_link(name, NULL, "ls-dir", cgit_query_head, |
88 | curr_rev, fullpath); | 88 | curr_rev, fullpath); |
89 | } else { | 89 | } else { |
90 | cgit_tree_link(name, NULL, "ls-blob", cgit_query_head, | 90 | cgit_tree_link(name, NULL, "ls-blob", cgit_query_head, |
91 | curr_rev, fullpath); | 91 | curr_rev, fullpath); |
92 | } | 92 | } |
93 | htmlf("</td><td class='ls-size'>%li</td>", size); | 93 | htmlf("</td><td class='ls-size'>%li</td>", size); |
94 | 94 | ||
95 | html("<td><a href='"); | 95 | html("<td>"); |
96 | qry = fmt("h=%s&path=%s%s%s", curr_rev, | 96 | cgit_log_link("L", "Log", "button", cgit_query_head, curr_rev, |
97 | cgit_query_path ? cgit_query_path : "", | 97 | fullpath); |
98 | cgit_query_path ? "/" : "", pathname); | 98 | html("</td></tr>\n"); |
99 | url = cgit_pageurl(cgit_query_repo, "log", qry); | ||
100 | html_attr(url); | ||
101 | html("' title='Log' class='button'>L</a></td>"); | ||
102 | html("</tr>\n"); | ||
103 | free(name); | 99 | free(name); |
104 | return 0; | 100 | return 0; |
105 | } | 101 | } |
106 | 102 | ||
107 | static void ls_head() | 103 | static void ls_head() |
108 | { | 104 | { |
109 | html("<table class='list'>\n"); | 105 | html("<table class='list'>\n"); |
110 | html("<tr class='nohover'>"); | 106 | html("<tr class='nohover'>"); |