##// END OF EJS Templates
decrease column size for gist_access_id.
decrease column size for gist_access_id.

File last commit:

r3821:ce4b7023 beta
r3870:b08a3f8a beta
Show More
compare.py
269 lines | 10.8 KiB | text/x-python | PythonLexer
Basic compare-view controller with ref parsing
r2241 # -*- coding: utf-8 -*-
"""
rhodecode.controllers.compare
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Mads Kiilerich
compare and diff: remove unused "bundle" functionality...
r3304 compare controller for pylons showing differences between two
Basic compare-view controller with ref parsing
r2241 repos, branches, bookmarks or tips
:created_on: May 6, 2012
:author: marcink
:copyright: (C) 2010-2012 Marcin Kuzminski <marcin@python-works.com>
:license: GPLv3, see COPYING for more details.
"""
# This program is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721
Basic compare-view controller with ref parsing
r2241 import logging
import traceback
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721 import re
Basic compare-view controller with ref parsing
r2241
Created base for diffing two repositories inside rhodecode
r2337 from webob.exc import HTTPNotFound
Basic compare-view controller with ref parsing
r2241 from pylons import request, response, session, tmpl_context as c, url
from pylons.controllers.util import abort, redirect
Add check for existing revisions in compare view
r2593 from pylons.i18n.translation import _
Basic compare-view controller with ref parsing
r2241
Add check for existing revisions in compare view
r2593 from rhodecode.lib.vcs.exceptions import EmptyRepositoryError, RepositoryError
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721 from rhodecode.lib.vcs.utils import safe_str
from rhodecode.lib.vcs.utils.hgcompat import scmutil
Implemented basic compare view(for pull requests) for mercurial.
r2348 from rhodecode.lib import helpers as h
Basic compare-view controller with ref parsing
r2241 from rhodecode.lib.base import BaseRepoController, render
from rhodecode.lib.auth import LoginRequired, HasRepoPermissionAnyDecorator
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721 from rhodecode.lib import diffs, unionrepo
Created base for diffing two repositories inside rhodecode
r2337
from rhodecode.model.db import Repository
swap the select values with revisions dynamically when switching the repos on pull request view...
r2847 from webob.exc import HTTPBadRequest
Optimized look of compare view of changesets....
r3011 from rhodecode.lib.diffs import LimitedDiffContainer
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721
Basic compare-view controller with ref parsing
r2241
log = logging.getLogger(__name__)
class CompareController(BaseRepoController):
def __before__(self):
super(CompareController, self).__before__()
swap the select values with revisions dynamically when switching the repos on pull request view...
r2847 def __get_cs_or_redirect(self, rev, repo, redirect_after=True,
partial=False):
Add check for existing revisions in compare view
r2593 """
Safe way to get changeset if error occur it redirects to changeset with
swap the select values with revisions dynamically when switching the repos on pull request view...
r2847 proper message. If partial is set then don't do redirect raise Exception
instead
Add check for existing revisions in compare view
r2593
:param rev: revision to fetch
:param repo: repo instance
"""
try:
type_, rev = rev
return repo.scm_instance.get_changeset(rev)
except EmptyRepositoryError, e:
if not redirect_after:
return None
h.flash(h.literal(_('There are no changesets yet')),
category='warning')
redirect(url('summary_home', repo_name=repo.repo_name))
except RepositoryError, e:
Bumped mercurial version to 2.3...
r2684 log.error(traceback.format_exc())
Add check for existing revisions in compare view
r2593 h.flash(str(e), category='warning')
swap the select values with revisions dynamically when switching the repos on pull request view...
r2847 if not partial:
redirect(h.url('summary_home', repo_name=repo.repo_name))
raise HTTPBadRequest()
Add check for existing revisions in compare view
r2593
auth decorators are not used anymore on __before__...
r3749 def _get_changesets(self, alias, org_repo, org_ref, other_repo, other_ref, merge):
"""
Returns a list of changesets that can be merged from org_repo@org_ref
to other_repo@other_ref ... and the ancestor that would be used for merge
:param org_repo:
:param org_ref:
:param other_repo:
:param other_ref:
:param tmp:
"""
ancestor = None
if alias == 'hg':
# lookup up the exact node id
_revset_predicates = {
'branch': 'branch',
'book': 'bookmark',
'tag': 'tag',
'rev': 'id',
}
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 org_rev_spec = "max(%s(%%s))" % _revset_predicates[org_ref[0]]
org_revs = org_repo._repo.revs(org_rev_spec, safe_str(org_ref[1]))
auth decorators are not used anymore on __before__...
r3749 org_rev = org_repo._repo[org_revs[-1] if org_revs else -1].hex()
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 other_revs_spec = "max(%s(%%s))" % _revset_predicates[other_ref[0]]
other_revs = other_repo._repo.revs(other_revs_spec, safe_str(other_ref[1]))
auth decorators are not used anymore on __before__...
r3749 other_rev = other_repo._repo[other_revs[-1] if other_revs else -1].hex()
#case two independent repos
if org_repo != other_repo:
hgrepo = unionrepo.unionrepository(other_repo.baseui,
other_repo.path,
org_repo.path)
# all the changesets we are looking for will be in other_repo,
# so rev numbers from hgrepo can be used in other_repo
#no remote compare do it on the same repository
else:
hgrepo = other_repo._repo
if merge:
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 revs = hgrepo.revs("ancestors(id(%s)) and not ancestors(id(%s)) and not id(%s)",
other_rev, org_rev, org_rev)
auth decorators are not used anymore on __before__...
r3749
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 ancestors = hgrepo.revs("ancestor(id(%s), id(%s))", org_rev, other_rev)
auth decorators are not used anymore on __before__...
r3749 if ancestors:
# pick arbitrary ancestor - but there is usually only one
ancestor = hgrepo[ancestors[0]].hex()
else:
# TODO: have both + and - changesets
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 revs = hgrepo.revs("id(%s) :: id(%s) - id(%s)",
org_rev, other_rev, org_rev)
auth decorators are not used anymore on __before__...
r3749
Mads Kiilerich
hg: use 'revset injection safe' repo.revs for revsets
r3811 changesets = [other_repo.get_changeset(rev) for rev in revs]
auth decorators are not used anymore on __before__...
r3749
elif alias == 'git':
It's better to use Exception here than assertion. It plays better with the exception handling software like sentry or errormator
r3783 if org_repo != other_repo:
raise Exception('Comparing of different GIT repositories is not'
'allowed. Got %s != %s' % (org_repo, other_repo))
auth decorators are not used anymore on __before__...
r3749 so, se = org_repo.run_git_command(
removed assertion about ancestor revision....
r3771 'log --reverse --pretty="format: %%H" -s -p %s..%s'
% (org_ref[1], other_ref[1])
auth decorators are not used anymore on __before__...
r3749 )
changesets = [org_repo.get_changeset(cs)
for cs in re.findall(r'[0-9a-fA-F]{40}', so)]
return changesets, ancestor
@LoginRequired()
@HasRepoPermissionAnyDecorator('repository.read', 'repository.write',
'repository.admin')
Rewrote url routes to make all validations and parsing for compare view + added compare fork button into forked repos
r2363 def index(self, org_ref_type, org_ref, other_ref_type, other_ref):
Mads Kiilerich
compare: minor refactoring and comments
r3443 # org_ref will be evaluated in org_repo
Rewrote url routes to make all validations and parsing for compare view + added compare fork button into forked repos
r2363 org_repo = c.rhodecode_db_repo.repo_name
org_ref = (org_ref_type, org_ref)
Mads Kiilerich
compare: minor refactoring and comments
r3443 # other_ref will be evaluated in other_repo
Rewrote url routes to make all validations and parsing for compare view + added compare fork button into forked repos
r2363 other_ref = (other_ref_type, other_ref)
Mads Kiilerich
compare: rename optional compare_url parameter repo to other_repo...
r3317 other_repo = request.GET.get('other_repo', org_repo)
Mads Kiilerich
compare/pullrequest: introduce merge parameter...
r3486 # If merge is True:
# Show what org would get if merged with other:
# List changesets that are ancestors of other but not of org.
# New changesets in org is thus ignored.
# Diff will be from common ancestor, and merges of org to other will thus be ignored.
# If merge is False:
# Make a raw diff from org to other, no matter if related or not.
# Changesets in one and not in the other will be ignored
merge = bool(request.GET.get('merge'))
Mads Kiilerich
compare: minor refactoring and comments
r3443 # fulldiff disables cut_off_limit
c.fulldiff = request.GET.get('fulldiff')
Mads Kiilerich
compare: cleanup of as_form handling...
r3442 # partial uses compare_cs.html template directly
partial = request.environ.get('HTTP_X_PARTIAL_XHR')
# as_form puts hidden input field with changeset revisions
c.as_form = partial and request.GET.get('as_form')
# swap url for compare_diff page - never partial and never as_form
c.swap_url = h.url('compare_url',
Mads Kiilerich
compare: rename optional compare_url parameter repo to other_repo...
r3317 repo_name=other_repo,
org_ref_type=other_ref[0], org_ref=other_ref[1],
fixed broken swap url, and added a test to detect it in feature
r3320 other_repo=org_repo,
Mads Kiilerich
compare/pullrequest: introduce merge parameter...
r3486 other_ref_type=org_ref[0], other_ref=org_ref[1],
merge=merge or '')
Rewrote url routes to make all validations and parsing for compare view + added compare fork button into forked repos
r2363
fixed pull-requests with cherry picking changesets...
r3380 org_repo = Repository.get_by_repo_name(org_repo)
other_repo = Repository.get_by_repo_name(other_repo)
Improved cross repos diffs...
r2362
fixed pull-requests with cherry picking changesets...
r3380 if org_repo is None:
Mads Kiilerich
compare: more helpful error messages when a repo wasn't found
r3143 log.error('Could not find org repo %s' % org_repo)
raise HTTPNotFound
fixed pull-requests with cherry picking changesets...
r3380 if other_repo is None:
Mads Kiilerich
compare: more helpful error messages when a repo wasn't found
r3143 log.error('Could not find other repo %s' % other_repo)
Improved cross repos diffs...
r2362 raise HTTPNotFound
fixed pull-requests with cherry picking changesets...
r3380 if org_repo != other_repo and h.is_git(org_repo):
Enabled compare engine for tags...
r3010 log.error('compare of two remote repos not available for GIT REPOS')
data checks
r2444 raise HTTPNotFound
Enabled compare engine for tags...
r3010
fixed pull-requests with cherry picking changesets...
r3380 if org_repo.scm_instance.alias != other_repo.scm_instance.alias:
Enabled compare engine for tags...
r3010 log.error('compare of two different kind of remote repos not available')
raise HTTPNotFound
move the existing changeset checks bellow other checks else it can throw exceptions for non...
r3615 self.__get_cs_or_redirect(rev=org_ref, repo=org_repo, partial=partial)
self.__get_cs_or_redirect(rev=other_ref, repo=other_repo, partial=partial)
fixed pull-requests with cherry picking changesets...
r3380 c.org_repo = org_repo
c.other_repo = other_repo
c.org_ref = org_ref[1]
c.other_ref = other_ref[1]
c.org_ref_type = org_ref[0]
c.other_ref_type = other_ref[0]
Add check for existing revisions in compare view
r2593
Mads Kiilerich
compare: move get_changesets to compare controller where it is used...
r3721 c.cs_ranges, c.ancestor = self._get_changesets(org_repo.scm_instance.alias,
org_repo.scm_instance, org_ref,
other_repo.scm_instance, other_ref,
merge)
Created base for diffing two repositories inside rhodecode
r2337
added changeset review status into repo/branch compare view
r2393 c.statuses = c.rhodecode_db_repo.statuses([x.raw_id for x in
c.cs_ranges])
removed assertion about ancestor revision....
r3771 if not c.ancestor:
log.warning('Unable to find ancestor revision')
swap the select values with revisions dynamically when switching the repos on pull request view...
r2847 if partial:
created pull-request overview
r2395 return render('compare/compare_cs.html')
added changeset review status into repo/branch compare view
r2393
Mads Kiilerich
compare/pullrequest: introduce merge parameter...
r3486 if c.ancestor:
assert merge
Mads Kiilerich
compare: show aggregated diff of what will be merged to other repo, using merge ancestor...
r3323 # case we want a simple diff without incoming changesets,
# previewing what will be merged.
Mads Kiilerich
compare/pullrequest: introduce merge parameter...
r3486 # Make the diff on the other repo (which is known to have other_ref)
fixed pull-requests with cherry picking changesets...
r3380 log.debug('Using ancestor %s as org_ref instead of %s'
Mads Kiilerich
compare/pullrequest: introduce merge parameter...
r3486 % (c.ancestor, org_ref))
org_ref = ('rev', c.ancestor)
Mads Kiilerich
compare: swap org and other when they refer to different repos, ie are pull request style...
r3322 org_repo = other_repo
#590 Add GET flag that controls the way the diff are generated, for pull requests we want to use non-bundle based diffs,...
r2892
Mads Kiilerich
compare: minor refactoring and comments
r3443 diff_limit = self.cut_off_limit if not c.fulldiff else None
Basic implementation of cherry picking changesets...
r3023
Mads Kiilerich
diff: cleanup of variables and logging in compare and pullrequests
r3812 log.debug('running diff between %s and %s in %s'
% (org_ref, other_ref, org_repo.scm_instance.path))
txtdiff = org_repo.scm_instance.get_diff(rev1=safe_str(org_ref[1]), rev2=safe_str(other_ref[1]))
#590 Add GET flag that controls the way the diff are generated, for pull requests we want to use non-bundle based diffs,...
r2892
Mads Kiilerich
diff: cleanup of variables and logging in compare and pullrequests
r3812 diff_processor = diffs.DiffProcessor(txtdiff or '', format='gitdiff',
Optimized look of compare view of changesets....
r3011 diff_limit=diff_limit)
Implemented basic compare view(for pull requests) for mercurial.
r2348 _parsed = diff_processor.prepare()
Created base for diffing two repositories inside rhodecode
r2337
Optimized look of compare view of changesets....
r3011 c.limited_diff = False
if isinstance(_parsed, LimitedDiffContainer):
c.limited_diff = True
Implemented basic compare view(for pull requests) for mercurial.
r2348 c.files = []
c.changes = {}
Multiple changes for compare system...
r3015 c.lines_added = 0
c.lines_deleted = 0
added changeset review status into repo/branch compare view
r2393 for f in _parsed:
Multiple changes for compare system...
r3015 st = f['stats']
diff parser: redefined operations stats for changes...
r3821 if not st['binary']:
c.lines_added += st['added']
c.lines_deleted += st['deleted']
Implemented basic compare view(for pull requests) for mercurial.
r2348 fid = h.FID('', f['filename'])
c.files.append([fid, f['operation'], f['filename'], f['stats']])
Mads Kiilerich
diff: cleanup of variables and logging in compare and pullrequests
r3812 htmldiff = diff_processor.as_html(enable_comments=False, parsed_lines=[f])
c.changes[fid] = [f['operation'], f['filename'], htmldiff]
Created base for diffing two repositories inside rhodecode
r2337
return render('compare/compare_diff.html')