# HG changeset patch # User Marcin Kuzminski # Date 1351009316 -7200 # Node ID c0a6a2e6bb2ee16936b04f3e4cac96a2e9d179ee # Parent 15dc5b2ff672f5f8f44837c6612930b2edd8142e Use lightweight revisions in journal by default diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/controllers/admin/settings.py --- a/rhodecode/controllers/admin/settings.py Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/controllers/admin/settings.py Tue Oct 23 18:21:56 2012 +0200 @@ -202,11 +202,6 @@ form_result['rhodecode_lightweight_dashboard'] Session().add(sett4) - sett5 = RhodeCodeSetting.get_by_name_or_create('lightweight_journal') - sett5.app_settings_value = \ - form_result['rhodecode_lightweight_journal'] - Session().add(sett5) - Session().commit() set_rhodecode_config(config) h.flash(_('Updated visualisation settings'), diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/lib/base.py --- a/rhodecode/lib/base.py Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/lib/base.py Tue Oct 23 18:21:56 2012 +0200 @@ -248,7 +248,6 @@ c.visual.show_private_icon = str2bool(rc_config.get('rhodecode_show_private_icon')) c.visual.stylify_metatags = str2bool(rc_config.get('rhodecode_stylify_metatags')) c.visual.lightweight_dashboard = str2bool(rc_config.get('rhodecode_lightweight_dashboard')) - c.visual.lightweight_journal = str2bool(rc_config.get('rhodecode_lightweight_dashboard')) c.repo_name = get_repo_slug(request) c.backends = BACKENDS.keys() diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/lib/helpers.py --- a/rhodecode/lib/helpers.py Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/lib/helpers.py Tue Oct 23 18:21:56 2012 +0200 @@ -64,7 +64,7 @@ def html_escape(text): """Produce entities within text.""" - return "".join(html_escape_table.get(c,c) for c in text) + return "".join(html_escape_table.get(c, c) for c in text) def shorter(text, size=20): @@ -491,7 +491,7 @@ return value -def action_parser(user_log, feed=False, parse_cs=True): +def action_parser(user_log, feed=False, parse_cs=False): """ This helper will action_map the specified string action into translated fancy names with icons and links @@ -500,9 +500,7 @@ :param feed: use output for feeds (no html and fancy icons) :param parse_cs: parse Changesets into VCS instances """ - from pylons import tmpl_context as c - if c.visual.lightweight_journal: - parse_cs = False + action = user_log.action action_params = ' ' @@ -522,42 +520,42 @@ repo_name = user_log.repository.repo_name def lnk(rev, repo_name): - if isinstance(rev, BaseChangeset) or isinstance(rev, AttributeDict): - if rev.revision: - lbl = 'r%s:%s' % (rev.revision, rev.short_id) - else: - lbl = '%s' % (rev.short_id) + lbl = '%s' % (rev.short_id[:8]) _url = url('changeset_home', repo_name=repo_name, revision=rev.raw_id) - title = tooltip(rev.message) if parse_cs else '' + title = tooltip(rev.message) else: + ## changeset cannot be found/striped/removed etc. lbl = ('%s' % rev)[:12] _url = '#' title = _('Changeset not found') - - return link_to(lbl, _url, title=title, - class_='tooltip' if parse_cs else '',) + if parse_cs: + return link_to(lbl, _url, title=title, class_='tooltip') + return link_to(lbl, _url, raw_id=rev.raw_id, class_='journal-cs') revs = [] if len(filter(lambda v: v != '', revs_ids)) > 0: - if parse_cs: - repo = user_log.repository.scm_instance + repo = None for rev in revs_ids[:revs_top_limit]: + # we want parsed changesets, or new log store format is bad if parse_cs: try: - rev = repo.get_changeset(rev) - revs.append(rev) + if repo is None: + repo = user_log.repository.scm_instance + _rev = repo.get_changeset(rev) + revs.append(_rev) except ChangesetDoesNotExistError: log.error('cannot find revision %s in this repo' % rev) revs.append(rev) continue else: - rev = AttributeDict({ + _rev = AttributeDict({ 'short_id': rev[:12], 'raw_id': rev, + 'message': '', }) - revs.append(rev) + revs.append(_rev) cs_links = [] cs_links.append(" " + ', '.join( [lnk(rev, repo_name) for rev in revs[:revs_limit]] diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/lib/hooks.py --- a/rhodecode/lib/hooks.py Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/lib/hooks.py Tue Oct 23 18:21:56 2012 +0200 @@ -36,7 +36,7 @@ from rhodecode.lib.vcs.backends.base import EmptyChangeset from rhodecode.lib.compat import json from rhodecode.lib.exceptions import HTTPLockedRC -from rhodecode.lib.utils2 import safe_str +from rhodecode.lib.utils2 import safe_str, datetime_to_time from rhodecode.model.db import Repository, User diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/templates/admin/settings/settings.html --- a/rhodecode/templates/admin/settings/settings.html Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/templates/admin/settings/settings.html Tue Oct 23 18:21:56 2012 +0200 @@ -131,10 +131,6 @@ ${h.checkbox('rhodecode_lightweight_dashboard','True')} -
- ${h.checkbox('rhodecode_lightweight_journal','True')} - -
diff -r 15dc5b2ff672 -r c0a6a2e6bb2e rhodecode/templates/pullrequests/pullrequest_show.html --- a/rhodecode/templates/pullrequests/pullrequest_show.html Tue Oct 23 18:10:07 2012 +0200 +++ b/rhodecode/templates/pullrequests/pullrequest_show.html Tue Oct 23 18:21:56 2012 +0200 @@ -70,14 +70,20 @@ <%include file="/compare/compare_cs.html" /> ## FILES -
${_('Files affected')}
-
+
+ % if c.files: +
${_('Files affected')}
+
%for fid, change, f, stat in c.files:
${h.link_to(h.safe_unicode(f),h.url.current(anchor=fid))}
${h.fancy_file_stats(stat)}
%endfor +
+ %else: +
${_('Click to load diff details')}
+ %endif
@@ -131,10 +137,12 @@ ## diff block +
<%namespace name="diff_block" file="/changeset/diff_block.html"/> %for fid, change, f, stat in c.files: ${diff_block.diff_block_simple([c.changes[fid]])} %endfor +
## template for inline comment form <%namespace name="comment" file="/changeset/changeset_file_comment.html"/>