aboutsummaryrefslogtreecommitdiff
path: root/ui-diff.c
diff refs
from: back
to: back
| flip
diff options
context:
space:
mode:
Diffstat (limited to 'ui-diff.c')
-rw-r--r--ui-diff.c163
1 files changed, 7 insertions, 156 deletions
diff --git a/ui-diff.c b/ui-diff.c
index c77ca2f..6f42dd4 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -1,17 +1,11 @@
/* ui-diff.c: show diff between two blobs
*
* Copyright (C) 2006-2014 cgit Development Team <cgit@lists.zx2c4.com>
- * Copyright (C) 2026 Saya Andy <saya.andy@posteo.com>
- *
- * Modified 2026 by Saya Andy:
- * Select variable refs from web ui for diff
*
* Licensed under GNU General Public License v2
* (see COPYING for full license text)
*/
-#define USE_THE_REPOSITORY_VARIABLE
-
#include "cgit.h"
#include "ui-diff.h"
#include "html.h"
@@ -340,145 +334,16 @@ static void filepair_cb(struct diff_filepair *pair)
cgit_ssdiff_footer();
}
-static int print_ref_option(const struct reference *ref, void *cb_data)
-{
- const char **sel = (const char **)cb_data;
-
- html_option(ref->name, ref->name, sel[0]);
- return 0;
-}
-
-static void cgit_print_ref_select(const char *name, const char *selected)
-{
- /*
- * Offer every branch and tag so that any two refs can be compared.
- */
- html("<select name='");
- html_attr(name);
- html("' onchange='this.form.submit();'>");
- refs_for_each_branch_ref(get_main_ref_store(the_repository),
- print_ref_option, (void *)&selected);
- if (ctx.repo->enable_remote_branches)
- refs_for_each_remote_ref(get_main_ref_store(the_repository),
- print_ref_option, (void *)&selected);
- refs_for_each_tag_ref(get_main_ref_store(the_repository),
- print_ref_option, (void *)&selected);
- html("</select>");
-}
-
-/*
- * The number of commits to step back from the selected ref, so that any
- * ancestor of a branch or tag can be picked without typing a rev into
- * the url by hand.
- */
-static void cgit_print_ref_offset(const char *name, int value)
-{
- html(" <input type='text' size='2' name='");
- html_attr(name);
- html("' title='N commits back from the selected ref'");
- if (value)
- htmlf(" value='%d'", value);
- html("> <span class='hint'>back</span>");
-}
-
void cgit_print_diff_ctrls(void)
{
int i, curr;
- html("<form class='diff-ctrls' method='get'>");
- /*
- * Emit the same hidden fields as cgit_add_hidden_formfields(), except
- * id/id2, which are carried by the ref-picker selects below. One
- * form wraps both panels so any submission carries all controls.
- */
- if (!ctx.cfg.virtual_root) {
- struct strbuf url = STRBUF_INIT;
- strbuf_addf(&url, "%s/%s", ctx.qry.repo, ctx.qry.page);
- if (ctx.qry.vpath)
- strbuf_addf(&url, "/%s", ctx.qry.vpath);
- html_hidden("url", url.buf);
- strbuf_release(&url);
- }
- if (ctx.qry.head && ctx.repo->defbranch &&
- strcmp(ctx.qry.head, ctx.repo->defbranch))
- html_hidden("h", ctx.qry.head);
- if (ctx.qry.showmsg)
- html_hidden("showmsg", "1");
-
- html("<div class='cgit-panel'>");
- html("<b>diff refs</b>");
- html("<table>");
- html("<tr><td colspan='2'/></tr>");
-
- /* ref pickers: compare any two refs (branches/tags) */
- {
- const char *new_ref = ctx.qry.oid && *ctx.qry.oid ?
- ctx.qry.oid : ctx.qry.head;
- const char *old_ref = ctx.qry.oid2;
-
- html("<tr>");
- html("<td class='label'>from:</td>");
- html("<td class='ctrl'>");
- cgit_print_ref_select("id2", old_ref);
- cgit_print_ref_offset("id2off", ctx.qry.id2off);
- html("</td></tr><tr>");
- html("<td class='label'>to:</td>");
- html("<td class='ctrl'>");
- cgit_print_ref_select("id", new_ref);
- cgit_print_ref_offset("idoff", ctx.qry.idoff);
- html("</td></tr>");
-
- html("<tr><td/><td class='ctrl'>");
- html("<input type='submit' value='apply'>");
- html(" | ");
- {
- const char *flip_new_ref;
- int flip_idoff;
- int idoff = ctx.qry.idoff;
- int id2off = ctx.qry.id2off;
-
- if (old_ref && *old_ref) {
- /* explicit from ref */
- flip_new_ref = old_ref;
- flip_idoff = id2off;
- } else if (!is_null_oid(old_rev_oid)) {
- /*
- * The from rev is the parent of the (stepped)
- * to rev, i.e. new_ref~(idoff+1); keep the
- * ref name in the url instead of a materialized
- * parent sha.
- */
- flip_new_ref = new_ref;
- flip_idoff = idoff + 1;
- } else {
- /* root: no from rev to flip to */
- flip_new_ref = NULL;
- flip_idoff = 0;
- }
-
- /*
- * cgit_diff_link() emits the offsets from ctx.qry;
- * swap them around the call so they follow the refs.
- */
- ctx.qry.idoff = flip_idoff;
- ctx.qry.id2off = idoff;
- cgit_diff_link("flip", "swap from and to", "button",
- ctx.qry.head,
- flip_new_ref, new_ref,
- ctx.qry.path);
- ctx.qry.idoff = idoff;
- ctx.qry.id2off = id2off;
- }
- html("</td></tr>");
- }
- html("</table>");
- html("</div>");
-
html("<div class='cgit-panel'>");
html("<b>diff options</b>");
+ html("<form method='get'>");
+ cgit_add_hidden_formfields(1, 0, ctx.qry.page);
html("<table>");
html("<tr><td colspan='2'/></tr>");
-
html("<tr>");
html("<td class='label'>context:</td>");
html("<td class='ctrl'>");
@@ -512,8 +377,8 @@ void cgit_print_diff_ctrls(void)
html("<tr><td/><td class='ctrl'>");
html("<noscript><input type='submit' value='reload'/></noscript>");
html("</td></tr></table>");
- html("</div>");
html("</form>");
+ html("</div>");
}
void cgit_print_diff(const char *new_rev, const char *old_rev,
@@ -521,7 +386,6 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
{
struct commit *commit, *commit2;
const struct object_id *old_tree_oid, *new_tree_oid;
- struct strbuf new_refbuf = STRBUF_INIT, old_refbuf = STRBUF_INIT;
diff_type difftype;
/*
@@ -529,23 +393,15 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
* entire commit to detect renames so we must limit the paths in our
* own callbacks and not pass the prefix to the diff machinery.
*/
- if (ctx.qry.follow && ctx.repo->enable_follow_links) {
+ if (ctx.qry.follow && ctx.cfg.enable_follow_links) {
current_prefix = prefix;
prefix = "";
} else {
current_prefix = NULL;
}
- /*
- * An empty rev is the same as no rev; a zero offset means the ref
- * itself.
- */
- if (!new_rev || !*new_rev)
+ if (!new_rev)
new_rev = ctx.qry.head;
- if (ctx.qry.idoff) {
- strbuf_addf(&new_refbuf, "%s~%d", new_rev, ctx.qry.idoff);
- new_rev = new_refbuf.buf;
- }
if (repo_get_oid(the_repository, new_rev, new_rev_oid)) {
cgit_print_error_page(404, "Not found",
"Bad object name: %s", new_rev);
@@ -559,12 +415,7 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
}
new_tree_oid = get_commit_tree_oid(commit);
- if (old_rev && *old_rev) {
- if (ctx.qry.id2off) {
- strbuf_addf(&old_refbuf, "%s~%d", old_rev,
- ctx.qry.id2off);
- old_rev = old_refbuf.buf;
- }
+ if (old_rev) {
if (repo_get_oid(the_repository, old_rev, old_rev_oid)) {
cgit_print_error_page(404, "Not found",
"Bad object name: %s", old_rev);
@@ -573,7 +424,7 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
} else if (commit->parents && commit->parents->item) {
oidcpy(old_rev_oid, &commit->parents->item->object.oid);
} else {
- oidclr(old_rev_oid, the_repository->hash_algo);
+ oidclr(old_rev_oid);
}
if (!is_null_oid(old_rev_oid)) {