aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohn Keeping2014-12-29 22:27:55 +0000
committerJason A. Donenfeld2014-12-30 10:09:25 +0100
commitd6c40507711ff6cc19c8fd3f28412845a33faf50 (patch)
tree3676ebeac4941416d77a3ce589f0a4498ec5d4be
parent7552266aaccb9445e082fc04215afcb55ad543d8 (diff)
downloadcgit-d6c40507711ff6cc19c8fd3f28412845a33faf50.tar.gz
cgit-d6c40507711ff6cc19c8fd3f28412845a33faf50.tar.bz2
cgit-d6c40507711ff6cc19c8fd3f28412845a33faf50.zip
ui-diff: don't link to single file diff stat
Seeing the diff stat for a single file is pretty useless, so reset the diff type before generating the links to individual files in the diff stat so that the links will show a useful diff. Reported-by: Konstantin Ryabitsev <mricon@kernel.org> Signed-off-by: John Keeping <john@keeping.me.uk>
-rw-r--r--ui-diff.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/ui-diff.c b/ui-diff.c
index bf2ec57..5b6df1f 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -428,6 +428,16 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
428 if (show_ctrls) 428 if (show_ctrls)
429 cgit_print_diff_ctrls(); 429 cgit_print_diff_ctrls();
430 430
431 /*
432 * Clicking on a link to a file in the diff stat should show a diff
433 * of the file, showing the diff stat limited to a single file is
434 * pretty useless. All links from this point on will be to
435 * individual files, so we simply reset the difftype in the query
436 * here to avoid propagating DIFF_STATONLY to the individual files.
437 */
438 if (difftype == DIFF_STATONLY)
439 ctx.qry.difftype = ctx.cfg.difftype;
440
431 cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); 441 cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix);
432 442
433 if (difftype == DIFF_STATONLY) 443 if (difftype == DIFF_STATONLY)