author | Lars Hjemli <hjemli@gmail.com> | 2008-12-07 10:45:28 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-12-07 10:45:28 (UTC) |
commit | c6a6aa2186daf39814baa0e71378c2e9e1041002 (patch) (unidiff) | |
tree | efd65ff67657677745b3d98f3840a4f93ce0f44c | |
parent | f86a23ff537258d36bf8f1876fa7a4bede6673d8 (diff) | |
download | cgit-c6a6aa2186daf39814baa0e71378c2e9e1041002.zip cgit-c6a6aa2186daf39814baa0e71378c2e9e1041002.tar.gz cgit-c6a6aa2186daf39814baa0e71378c2e9e1041002.tar.bz2 |
ui-stats: enable path-filtered stats
When a path is specified on the querystring the commit statistics will
now be filtered by this path.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-stats.c | 18 |
1 files changed, 15 insertions, 3 deletions
@@ -174,68 +174,74 @@ static void add_commit(struct string_list *authors, struct commit *commit, | |||
174 | item->util++; | 174 | item->util++; |
175 | authorstat->total++; | 175 | authorstat->total++; |
176 | cgit_free_commitinfo(info); | 176 | cgit_free_commitinfo(info); |
177 | } | 177 | } |
178 | 178 | ||
179 | static int cmp_total_commits(const void *a1, const void *a2) | 179 | static int cmp_total_commits(const void *a1, const void *a2) |
180 | { | 180 | { |
181 | const struct string_list_item *i1 = a1; | 181 | const struct string_list_item *i1 = a1; |
182 | const struct string_list_item *i2 = a2; | 182 | const struct string_list_item *i2 = a2; |
183 | const struct authorstat *auth1 = i1->util; | 183 | const struct authorstat *auth1 = i1->util; |
184 | const struct authorstat *auth2 = i2->util; | 184 | const struct authorstat *auth2 = i2->util; |
185 | 185 | ||
186 | return auth2->total - auth1->total; | 186 | return auth2->total - auth1->total; |
187 | } | 187 | } |
188 | 188 | ||
189 | /* Walk the commit DAG and collect number of commits per author per | 189 | /* Walk the commit DAG and collect number of commits per author per |
190 | * timeperiod into a nested string_list collection. | 190 | * timeperiod into a nested string_list collection. |
191 | */ | 191 | */ |
192 | struct string_list collect_stats(struct cgit_context *ctx, | 192 | struct string_list collect_stats(struct cgit_context *ctx, |
193 | struct Period *period) | 193 | struct Period *period) |
194 | { | 194 | { |
195 | struct string_list authors; | 195 | struct string_list authors; |
196 | struct rev_info rev; | 196 | struct rev_info rev; |
197 | struct commit *commit; | 197 | struct commit *commit; |
198 | const char *argv[] = {NULL, ctx->qry.head, NULL, NULL}; | 198 | const char *argv[] = {NULL, ctx->qry.head, NULL, NULL, NULL, NULL}; |
199 | int argc = 3; | ||
199 | time_t now; | 200 | time_t now; |
200 | long i; | 201 | long i; |
201 | struct tm *tm; | 202 | struct tm *tm; |
202 | char tmp[11]; | 203 | char tmp[11]; |
203 | 204 | ||
204 | time(&now); | 205 | time(&now); |
205 | tm = gmtime(&now); | 206 | tm = gmtime(&now); |
206 | period->trunc(tm); | 207 | period->trunc(tm); |
207 | for (i = 1; i < period->count; i++) | 208 | for (i = 1; i < period->count; i++) |
208 | period->dec(tm); | 209 | period->dec(tm); |
209 | strftime(tmp, sizeof(tmp), "%Y-%m-%d", tm); | 210 | strftime(tmp, sizeof(tmp), "%Y-%m-%d", tm); |
210 | argv[2] = xstrdup(fmt("--since=%s", tmp)); | 211 | argv[2] = xstrdup(fmt("--since=%s", tmp)); |
212 | if (ctx->qry.path) { | ||
213 | argv[3] = "--"; | ||
214 | argv[4] = ctx->qry.path; | ||
215 | argc += 2; | ||
216 | } | ||
211 | init_revisions(&rev, NULL); | 217 | init_revisions(&rev, NULL); |
212 | rev.abbrev = DEFAULT_ABBREV; | 218 | rev.abbrev = DEFAULT_ABBREV; |
213 | rev.commit_format = CMIT_FMT_DEFAULT; | 219 | rev.commit_format = CMIT_FMT_DEFAULT; |
214 | rev.no_merges = 1; | 220 | rev.no_merges = 1; |
215 | rev.verbose_header = 1; | 221 | rev.verbose_header = 1; |
216 | rev.show_root_diff = 0; | 222 | rev.show_root_diff = 0; |
217 | setup_revisions(3, argv, &rev, NULL); | 223 | setup_revisions(argc, argv, &rev, NULL); |
218 | prepare_revision_walk(&rev); | 224 | prepare_revision_walk(&rev); |
219 | memset(&authors, 0, sizeof(authors)); | 225 | memset(&authors, 0, sizeof(authors)); |
220 | while ((commit = get_revision(&rev)) != NULL) { | 226 | while ((commit = get_revision(&rev)) != NULL) { |
221 | add_commit(&authors, commit, period); | 227 | add_commit(&authors, commit, period); |
222 | free(commit->buffer); | 228 | free(commit->buffer); |
223 | free_commit_list(commit->parents); | 229 | free_commit_list(commit->parents); |
224 | } | 230 | } |
225 | return authors; | 231 | return authors; |
226 | } | 232 | } |
227 | 233 | ||
228 | void print_combined_authorrow(struct string_list *authors, int from, int to, | 234 | void print_combined_authorrow(struct string_list *authors, int from, int to, |
229 | const char *name, const char *leftclass, const char *centerclass, | 235 | const char *name, const char *leftclass, const char *centerclass, |
230 | const char *rightclass, struct Period *period) | 236 | const char *rightclass, struct Period *period) |
231 | { | 237 | { |
232 | struct string_list_item *author; | 238 | struct string_list_item *author; |
233 | struct authorstat *authorstat; | 239 | struct authorstat *authorstat; |
234 | struct string_list *items; | 240 | struct string_list *items; |
235 | struct string_list_item *date; | 241 | struct string_list_item *date; |
236 | time_t now; | 242 | time_t now; |
237 | long i, j, total, subtotal; | 243 | long i, j, total, subtotal; |
238 | struct tm *tm; | 244 | struct tm *tm; |
239 | char *tmp; | 245 | char *tmp; |
240 | 246 | ||
241 | time(&now); | 247 | time(&now); |
@@ -330,49 +336,55 @@ void print_authors(struct string_list *authors, int top, struct Period *period) | |||
330 | * for each author is another string_list which is used to calculate the | 336 | * for each author is another string_list which is used to calculate the |
331 | * number of commits per time-interval. | 337 | * number of commits per time-interval. |
332 | */ | 338 | */ |
333 | void cgit_show_stats(struct cgit_context *ctx) | 339 | void cgit_show_stats(struct cgit_context *ctx) |
334 | { | 340 | { |
335 | struct string_list authors; | 341 | struct string_list authors; |
336 | struct Period *period; | 342 | struct Period *period; |
337 | int top, i; | 343 | int top, i; |
338 | 344 | ||
339 | period = &periods[0]; | 345 | period = &periods[0]; |
340 | if (ctx->qry.period) { | 346 | if (ctx->qry.period) { |
341 | for (i = 0; i < sizeof(periods) / sizeof(periods[0]); i++) | 347 | for (i = 0; i < sizeof(periods) / sizeof(periods[0]); i++) |
342 | if (periods[i].code == ctx->qry.period[0]) { | 348 | if (periods[i].code == ctx->qry.period[0]) { |
343 | period = &periods[i]; | 349 | period = &periods[i]; |
344 | break; | 350 | break; |
345 | } | 351 | } |
346 | } | 352 | } |
347 | authors = collect_stats(ctx, period); | 353 | authors = collect_stats(ctx, period); |
348 | qsort(authors.items, authors.nr, sizeof(struct string_list_item), | 354 | qsort(authors.items, authors.nr, sizeof(struct string_list_item), |
349 | cmp_total_commits); | 355 | cmp_total_commits); |
350 | 356 | ||
351 | top = ctx->qry.ofs; | 357 | top = ctx->qry.ofs; |
352 | if (!top) | 358 | if (!top) |
353 | top = 10; | 359 | top = 10; |
354 | htmlf("<h2>Commits per author per %s</h2>", period->name); | 360 | htmlf("<h2>Commits per author per %s", period->name); |
361 | if (ctx->qry.path) { | ||
362 | html(" (path '"); | ||
363 | html_txt(ctx->qry.path); | ||
364 | html("')"); | ||
365 | } | ||
366 | html("</h2>"); | ||
355 | 367 | ||
356 | html("<form method='get' action='.' style='float: right; text-align: right;'>"); | 368 | html("<form method='get' action='.' style='float: right; text-align: right;'>"); |
357 | if (strcmp(ctx->qry.head, ctx->repo->defbranch)) | 369 | if (strcmp(ctx->qry.head, ctx->repo->defbranch)) |
358 | htmlf("<input type='hidden' name='h' value='%s'/>", ctx->qry.head); | 370 | htmlf("<input type='hidden' name='h' value='%s'/>", ctx->qry.head); |
359 | html("Period: "); | 371 | html("Period: "); |
360 | html("<select name='period' onchange='this.form.submit();'>"); | 372 | html("<select name='period' onchange='this.form.submit();'>"); |
361 | for (i = 0; i < sizeof(periods) / sizeof(periods[0]); i++) | 373 | for (i = 0; i < sizeof(periods) / sizeof(periods[0]); i++) |
362 | htmlf("<option value='%c'%s>%s</option>", | 374 | htmlf("<option value='%c'%s>%s</option>", |
363 | periods[i].code, | 375 | periods[i].code, |
364 | period == &periods[i] ? " selected" : "", | 376 | period == &periods[i] ? " selected" : "", |
365 | periods[i].name); | 377 | periods[i].name); |
366 | html("</select><br/><br/>"); | 378 | html("</select><br/><br/>"); |
367 | html("Authors: "); | 379 | html("Authors: "); |
368 | html(""); | 380 | html(""); |
369 | html("<select name='ofs' onchange='this.form.submit();'>"); | 381 | html("<select name='ofs' onchange='this.form.submit();'>"); |
370 | htmlf("<option value='10'%s>10</option>", top == 10 ? " selected" : ""); | 382 | htmlf("<option value='10'%s>10</option>", top == 10 ? " selected" : ""); |
371 | htmlf("<option value='25'%s>25</option>", top == 25 ? " selected" : ""); | 383 | htmlf("<option value='25'%s>25</option>", top == 25 ? " selected" : ""); |
372 | htmlf("<option value='50'%s>50</option>", top == 50 ? " selected" : ""); | 384 | htmlf("<option value='50'%s>50</option>", top == 50 ? " selected" : ""); |
373 | htmlf("<option value='100'%s>100</option>", top == 100 ? " selected" : ""); | 385 | htmlf("<option value='100'%s>100</option>", top == 100 ? " selected" : ""); |
374 | htmlf("<option value='-1'%s>All</option>", top == -1 ? " selected" : ""); | 386 | htmlf("<option value='-1'%s>All</option>", top == -1 ? " selected" : ""); |
375 | html("</select>"); | 387 | html("</select>"); |
376 | html("<noscript> <input type='submit' value='Reload'/></noscript>"); | 388 | html("<noscript> <input type='submit' value='Reload'/></noscript>"); |
377 | html("</form>"); | 389 | html("</form>"); |
378 | print_authors(&authors, top, period); | 390 | print_authors(&authors, top, period); |