Add knobs to enable/disable files/lines changed in log view
These columns can cause lots of IO on the server, so add settings to
explicitly enable them. Also, add per repo settings to optionally disable
the columns if sitewide enabled.
While at it, do not allow repo.snapshot to enable snapshots if the global
setting is disabled.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
diff --git a/ui-log.c b/ui-log.c
index 9d0ec02..4237921 100644
--- a/ui-log.c
+++ b/ui-log.c
@@ -19,7 +19,8 @@
void inspect_files(struct diff_filepair *pair)
{
files++;
- cgit_diff_files(pair->one->sha1, pair->two->sha1, count_lines);
+ if (cgit_repo->enable_log_linecount)
+ cgit_diff_files(pair->one->sha1, pair->two->sha1, count_lines);
}
void print_commit(struct commit *commit)
@@ -39,13 +40,17 @@
html_link_open(url, NULL, NULL);
html_ntxt(cgit_max_msg_len, info->subject);
html_link_close();
- files = 0;
- lines = 0;
- cgit_diff_commit(commit, inspect_files);
- html("</td><td class='right'>");
- htmlf("%d", files);
- html("</td><td class='right'>");
- htmlf("%d", lines);
+ if (cgit_repo->enable_log_filecount) {
+ files = 0;
+ lines = 0;
+ cgit_diff_commit(commit, inspect_files);
+ html("</td><td class='right'>");
+ htmlf("%d", files);
+ if (cgit_repo->enable_log_linecount) {
+ html("</td><td class='right'>");
+ htmlf("%d", lines);
+ }
+ }
html("</td><td>");
html_txt(info->author);
html("</td></tr>\n");
@@ -81,10 +86,14 @@
html("<table class='list nowrap'>");
html("<tr class='nohover'><th class='left'>Date</th>"
- "<th class='left'>Message</th>"
- "<th class='left'>Files</th>"
- "<th class='left'>Lines</th>"
- "<th class='left'>Author</th></tr>\n");
+ "<th class='left'>Message</th>");
+
+ if (cgit_repo->enable_log_filecount) {
+ html("<th class='left'>Files</th>");
+ if (cgit_repo->enable_log_linecount)
+ html("<th class='left'>Lines</th>");
+ }
+ html("<th class='left'>Author</th></tr>\n");
if (ofs<0)
ofs = 0;