author | Lars Hjemli <hjemli@gmail.com> | 2008-12-07 14:57:55 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-12-07 14:57:55 (UTC) |
commit | b54ef9749c083afd86573112fad3b3ed8ee2d0e4 (patch) (side-by-side diff) | |
tree | c23d5f0b32b15e7b6d9d455671a8f5acdc198417 | |
parent | c3c925f4eca3e863bedaae4cfe237c3ad52092b6 (diff) | |
download | cgit-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>
-rw-r--r-- | ui-stats.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -369,35 +369,34 @@ void cgit_show_stats(struct cgit_context *ctx) } 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" : ""); |