summaryrefslogtreecommitdiffabout
authorLars Hjemli <hjemli@gmail.com>2008-12-07 14:57:55 (UTC)
committer Lars Hjemli <hjemli@gmail.com>2008-12-07 14:57:55 (UTC)
commitb54ef9749c083afd86573112fad3b3ed8ee2d0e4 (patch) (side-by-side diff)
treec23d5f0b32b15e7b6d9d455671a8f5acdc198417
parentc3c925f4eca3e863bedaae4cfe237c3ad52092b6 (diff)
downloadcgit-b54ef9749c083afd86573112fad3b3ed8ee2d0e4.zip
cgit-b54ef9749c083afd86573112fad3b3ed8ee2d0e4.tar.gz
cgit-b54ef9749c083afd86573112fad3b3ed8ee2d0e4.tar.bz2
ui-stats.c: reuse cgit_add_hidden_formfields()
This makes sure that the currect branch and path is preserved when changing the stats period or author count. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--ui-stats.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/ui-stats.c b/ui-stats.c
index 1104485..9fc06d3 100644
--- a/ui-stats.c
+++ b/ui-stats.c
@@ -361,51 +361,50 @@ void cgit_show_stats(struct cgit_context *ctx)
if (!i) {
cgit_print_error(fmt("Unknown statistics type: %c", code));
return;
}
if (i > ctx->repo->max_stats) {
cgit_print_error(fmt("Statistics type disabled: %s",
period->name));
return;
}
authors = collect_stats(ctx, period);
qsort(authors.items, authors.nr, sizeof(struct string_list_item),
cmp_total_commits);
top = ctx->qry.ofs;
if (!top)
top = 10;
htmlf("<h2>Commits per author per %s", period->name);
if (ctx->qry.path) {
html(" (path '");
html_txt(ctx->qry.path);
html("')");
}
html("</h2>");
- html("<form method='get' action='.' style='float: right; text-align: right;'>");
- if (strcmp(ctx->qry.head, ctx->repo->defbranch))
- htmlf("<input type='hidden' name='h' value='%s'/>", ctx->qry.head);
+ html("<form method='get' action='' style='float: right; text-align: right;'>");
+ cgit_add_hidden_formfields(1, 0, "stats");
if (ctx->repo->max_stats > 1) {
html("Period: ");
html("<select name='period' onchange='this.form.submit();'>");
for (i = 0; i < ctx->repo->max_stats; i++)
htmlf("<option value='%c'%s>%s</option>",
periods[i].code,
period == &periods[i] ? " selected" : "",
periods[i].name);
html("</select><br/><br/>");
}
html("Authors: ");
html("");
html("<select name='ofs' onchange='this.form.submit();'>");
htmlf("<option value='10'%s>10</option>", top == 10 ? " selected" : "");
htmlf("<option value='25'%s>25</option>", top == 25 ? " selected" : "");
htmlf("<option value='50'%s>50</option>", top == 50 ? " selected" : "");
htmlf("<option value='100'%s>100</option>", top == 100 ? " selected" : "");
htmlf("<option value='-1'%s>All</option>", top == -1 ? " selected" : "");
html("</select>");
html("<noscript>&nbsp;&nbsp;<input type='submit' value='Reload'/></noscript>");
html("</form>");
print_authors(&authors, top, period);
}