Ticket #7715: 7715-mergeinfo-display-r8263.patch

trac/htdocs/css/browser.css

@@ -174 +174 @@
  margin: 0 0 .4em 1.6em;
  padding: 0;
 }
-#info .props li { padding: 0; overflow: auto; }
+#info .props > li { padding: 2px 0; overflow: auto; }
 
 /* Styles for the HTML preview */
 #preview { background: #fff; clear: both; margin: 0 }

trac/versioncontrol/svn_fs.py

@@ -323 +323 @@
         if name == 'svn:externals':
             return self._render_externals(props[name])
         elif name == 'svn:mergeinfo' or name.startswith('svnmerge-'):
-            return self._render_mergeinfo(props[name])
+            return self._render_mergeinfo(name, mode, context, props)
         elif name == 'svn:needs-lock':
             return self._render_needslock(context)
 
@@ -389 +389 @@
         return tag.ul([tag.li(tag.a(label, href=href, title=title))
                        for label, href, title in externals_data])
 
-    def _render_mergeinfo(self, prop):
-        prop = prop.rsplit(':', 1)
-        if len(prop) == 2:
-            prop[1] = prop[1].replace(',', u',\u200b')
-        return ':'.join(prop)
-
+    def _render_mergeinfo(self, name, mode, context, props):
+        """Parse svn:mergeinfo and svnmerge-* properties converting branch
+        names and revisions to links."""
+        ul = []
+        for line in props[name].splitlines():
+            try:
+                path, revs = line.split(':', 1)
+                if 'LOG_VIEW' in context.perm('source', path):
+                    li = [tag.a(path, href=context.href.browser(path)), ':']
+                    for rev in revs.split(','):
+                        if len(li) > 2:
+                            li.append(u',\u200b')
+                        range = rev.split('-', 1)
+                        if len(range) > 1:
+                            href=context.href.log(path, rev=range[1], 
+                                                  stop_rev=range[0])
+                        else:
+                            href=context.href.changeset(rev, path)
+                        li.append(tag.a(rev, href=href))
+                    ul.append(tag.li(li))
+                else:
+                    revs = revs.replace(',', u',\u200b')
+                    ul.append(tag.li(':'.join([path, revs])))
+            except Exception, e:
+                self.log.warning('Rendering of property %s failed: %s', name,
+                                 exception_to_unicode(e))
+                ul.append(tag.li(line))
+        return tag.ul(ul)      
+     
     def _render_needslock(self, context):
         return tag.img(src=context.href.chrome('common/lock-locked.png'),
                        alt="needs lock", title="needs lock")