the repository which powers this website
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 <[email protected]> Signed-off-by: John Keeping <[email protected]>
John Keeping 2014-12-30
parent 7552266 · commit d6c4050
-rw-r--r--ui-diff.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/ui-diff.c b/ui-diff.c
index bf2ec57d..5b6df1ff 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,
if (show_ctrls)
cgit_print_diff_ctrls();
+ /*
+ * Clicking on a link to a file in the diff stat should show a diff
+ * of the file, showing the diff stat limited to a single file is
+ * pretty useless. All links from this point on will be to
+ * individual files, so we simply reset the difftype in the query
+ * here to avoid propagating DIFF_STATONLY to the individual files.
+ */
+ if (difftype == DIFF_STATONLY)
+ ctx.qry.difftype = ctx.cfg.difftype;
+
cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix);
if (difftype == DIFF_STATONLY)