[PATCH] improve compatibility with Trac 0.12 / 1.0

[Genie]

Hi~, This is patch for Trac 0.12 / 1.0 compatibility.

tracmetrixplugin/web_ui.py

@@ -28 +28 @@
 from trac.util.datefmt import to_datetime, utc
 from trac.web import IRequestFilter, IRequestHandler, ITemplateStreamFilter
 from trac.web.chrome import add_stylesheet, INavigationContributor, ITemplateProvider
+from trac import __version__
 
 from tracmetrixplugin.model import ChangesetsStats, TicketGroupMetrics
 
@@ -86 +87 @@
         milestoneName = u"".join(e[1] for e in self.buffer.events)
         title = "Go to TracMetrix for %s" % milestoneName
         href = self.baseHref.mdashboard(milestoneName)
+        func = ( __version__ < '1.0') and tag.dd or tag.span # Trac >= 1.0dev
 
-        return iter(tag.dd('[', tag.a('TracMetrix', href=href, title=title), ']'))
+        return iter( func('[', tag.a('TracMetrix', href=href, title=title), ']'))
 
 
 
@@ -104 +106 @@
 
         if filename in ('roadmap.html', ):
 
+            tag = ( __version__ < '0.12.1') and 'li' or 'div' # Trac >= 0.12.1
+            cls = ( __version__ < '1.0') and ''   or ' trac-progress'     # Trac >= 1.0dev
+            obj = ( __version__ < '1.0') and 'dl' or 'p[@class="legend"]' # Trac >= 1.0dev
+
             buffer = StreamBuffer()
-            t = Transformer('//li[@class="milestone"]/div[@class="info"]/h2/a/em/text()')
+            t = Transformer('//'+ tag +'[@class="milestone"]/div[@class="info'+ cls +'"]/h2/a/em/text()')
             t = t.copy(buffer).end()
-            t = t.select('//li[@class="milestone"]/div[@class="info"]/dl')
+            t = t.select('//'+ tag +'[@class="milestone"]/div[@class="info'+ cls +'"]/'+ obj)
             t = t.append(GenerateMetrixLink(buffer, req.href))
             stream |= t
 
@@ -125 +131 @@
 
         if filename in ('milestone_view.html', ):
 
+            cls = ( __version__ < '1.0') and ''   or ' trac-progress'     # Trac >= 1.0dev
+            obj = ( __version__ < '1.0') and 'dl' or 'p[@class="legend"]' # Trac >= 1.0dev
+
             buffer = StreamBuffer()
             t = Transformer('//div[@class="milestone"]/h1/text()[2]')
             t = t.copy(buffer).end()
-            t = t.select('//div[@class="milestone"]/div[@class="info"]/dl')
+            t = t.select('//div[@class="milestone"]/div[@class="info'+ cls +'"]/'+ obj)
             t = t.append(GenerateMetrixLink(buffer, req.href))
             stream |= t

[Ryan J Ollos]

Thanks for the patch! Could you give a brief explanation of what the patch accomplishes or the changes in 0.12/1.0 that you are trying to address?

[Genie]

Roadmap(Milestone) view Layout has changed.

Trac >= 0.12.1:

. roadmap.html ( li[@class="milestone"] -> div[@class="milestone"] )

​r9662

[Genie]

Trac >= 1.0dev (0.13dev):

. roadmap.html ( div[@class="info" -> div[@class="info trac-progress" )

. milestone_view.html ( div[@class="info" -> div[@class="info trac-progress" )

​r10751

. progress_bar.html ( dl -> p[@class="legend"] )

​r10662