# HG changeset patch # User Mads Kiilerich # Date 1592484808 -7200 # Node ID 5463f4b13fc3bb8507e25e8f1266a3f368b44668 # Parent 4bf9d7f182531cc8f51d1891ea729afd51ca2edb diff: name context size and whitespace ignore flag variable names consistently diff -r 4bf9d7f18253 -r 5463f4b13fc3 kallithea/controllers/changeset.py --- a/kallithea/controllers/changeset.py Thu Jun 18 14:25:23 2020 +0200 +++ b/kallithea/controllers/changeset.py Thu Jun 18 14:53:28 2020 +0200 @@ -351,11 +351,11 @@ cs2 = changeset.raw_id cs1 = changeset.parents[0].raw_id if changeset.parents else EmptyChangeset().raw_id - context_lcl = get_line_ctx('', request.GET) - ign_whitespace_lcl = get_ignore_ws('', request.GET) + diff_context_size = get_line_ctx('', request.GET) + ignore_whitespace_diff = get_ignore_ws('', request.GET) raw_diff = diffs.get_diff(c.db_repo_scm_instance, cs1, cs2, - ignore_whitespace=ign_whitespace_lcl, context=context_lcl) + ignore_whitespace=ignore_whitespace_diff, context=diff_context_size) diff_limit = None if c.fulldiff else self.cut_off_limit file_diff_data = [] if method == 'show': diff -r 4bf9d7f18253 -r 5463f4b13fc3 kallithea/controllers/compare.py --- a/kallithea/controllers/compare.py Thu Jun 18 14:25:23 2020 +0200 +++ b/kallithea/controllers/compare.py Thu Jun 18 14:53:28 2020 +0200 @@ -212,8 +212,8 @@ # set callbacks for generating markup for icons c.ignorews_url = _ignorews_url c.context_url = _context_url - ignore_whitespace = request.GET.get('ignorews') == '1' - line_context = safe_int(request.GET.get('context'), 3) + ignore_whitespace_diff = request.GET.get('ignorews') == '1' + diff_context_size = safe_int(request.GET.get('context'), 3) c.a_rev = self._get_ref_rev(c.a_repo, org_ref_type, org_ref_name, returnempty=True) @@ -275,8 +275,8 @@ log.debug('running diff between %s and %s in %s', rev1, c.cs_rev, org_repo.scm_instance.path) raw_diff = diffs.get_diff(org_repo.scm_instance, rev1=rev1, rev2=c.cs_rev, - ignore_whitespace=ignore_whitespace, - context=line_context) + ignore_whitespace=ignore_whitespace_diff, + context=diff_context_size) diff_processor = diffs.DiffProcessor(raw_diff, diff_limit=diff_limit) c.limited_diff = diff_processor.limited_diff diff -r 4bf9d7f18253 -r 5463f4b13fc3 kallithea/controllers/files.py --- a/kallithea/controllers/files.py Thu Jun 18 14:25:23 2020 +0200 +++ b/kallithea/controllers/files.py Thu Jun 18 14:53:28 2020 +0200 @@ -558,8 +558,8 @@ @LoginRequired(allow_default_user=True) @HasRepoPermissionLevelDecorator('read') def diff(self, repo_name, f_path): - ignore_whitespace = request.GET.get('ignorews') == '1' - line_context = safe_int(request.GET.get('context'), 3) + ignore_whitespace_diff = request.GET.get('ignorews') == '1' + diff_context_size = safe_int(request.GET.get('context'), 3) diff2 = request.GET.get('diff2', '') diff1 = request.GET.get('diff1', '') or diff2 c.action = request.GET.get('diff') @@ -623,8 +623,8 @@ if c.action == 'download': raw_diff = diffs.get_gitdiff(node1, node2, - ignore_whitespace=ignore_whitespace, - context=line_context) + ignore_whitespace=ignore_whitespace_diff, + context=diff_context_size) diff_name = '%s_vs_%s.diff' % (diff1, diff2) response.content_type = 'text/plain' response.content_disposition = ( @@ -634,22 +634,21 @@ elif c.action == 'raw': raw_diff = diffs.get_gitdiff(node1, node2, - ignore_whitespace=ignore_whitespace, - context=line_context) + ignore_whitespace=ignore_whitespace_diff, + context=diff_context_size) response.content_type = 'text/plain' return raw_diff else: fid = h.FID(diff2, node2.path) - line_context_lcl = get_line_ctx(fid, request.GET) - ign_whitespace_lcl = get_ignore_ws(fid, request.GET) - + diff_context_size = get_line_ctx(fid, request.GET) + ignore_whitespace_diff = get_ignore_ws(fid, request.GET) diff_limit = None if fulldiff else self.cut_off_limit c.a_rev, c.cs_rev, a_path, diff, st, op = diffs.wrapped_diff(filenode_old=node1, filenode_new=node2, diff_limit=diff_limit, - ignore_whitespace=ign_whitespace_lcl, - line_context=line_context_lcl, + ignore_whitespace=ignore_whitespace_diff, + line_context=diff_context_size, enable_comments=False) c.file_diff_data = [(fid, fid, op, a_path, node2.path, diff, st)] diff -r 4bf9d7f18253 -r 5463f4b13fc3 kallithea/controllers/pullrequests.py --- a/kallithea/controllers/pullrequests.py Thu Jun 18 14:25:23 2020 +0200 +++ b/kallithea/controllers/pullrequests.py Thu Jun 18 14:53:28 2020 +0200 @@ -569,8 +569,8 @@ c.cs_comments = c.cs_repo.get_comments(raw_ids) c.cs_statuses = c.cs_repo.statuses(raw_ids) - ignore_whitespace = request.GET.get('ignorews') == '1' - line_context = safe_int(request.GET.get('context'), 3) + ignore_whitespace_diff = request.GET.get('ignorews') == '1' + diff_context_size = safe_int(request.GET.get('context'), 3) c.ignorews_url = _ignorews_url c.context_url = _context_url fulldiff = request.GET.get('fulldiff') @@ -581,7 +581,7 @@ c.a_rev, c.cs_rev, org_scm_instance.path) try: raw_diff = diffs.get_diff(org_scm_instance, rev1=c.a_rev, rev2=c.cs_rev, - ignore_whitespace=ignore_whitespace, context=line_context) + ignore_whitespace=ignore_whitespace_diff, context=diff_context_size) except ChangesetDoesNotExistError: raw_diff = safe_bytes(_("The diff can't be shown - the PR revisions could not be found.")) diff_processor = diffs.DiffProcessor(raw_diff, diff_limit=diff_limit)