ReportPluginPatch: trac-0.10.4-report-plugin.patch

trac/ticket/report.py

@@ -32 +32 @@
-from trac.web.chrome import add_link, add_stylesheet, INavigationContributor
-from trac.wiki import wiki_to_html, IWikiSyntaxProvider, Formatter
-
+class ITicketReportRenderer(Interface):  
+    """Extension point interface for components that implement new report   
+    writing formats."""  
+  
+    def get_report_format():  
+        """Called to get the id for a report format, for example: .xls"""  
+          
+        def get_report_mimetype():  
+                """Called to get the mimetype string for a report format"""  
+          
+        def get_report_linkname():  
+                """Called to get the report link name that will appear on the UI"""  
+                    
+        def get_report_linkclass():  
+                """Called to get the class of the link that will appear on the UI"""  
+  
+        def render(req, cols, rows):  
+                """Render the report, takes the request, columns and rows"""     
+
+
-class ReportModule(Component):
-
-    implements(INavigationContributor, IPermissionRequestor, IRequestHandler,
@@ -111 +131 @@
-        add_stylesheet(req, 'common/css/report.css')
-        return 'report.cs', None
-
+    renderers = ExtensionPoint(ITicketReportRenderer)
+
-    # Internal methods
-
-    def _do_create(self, req, db):
@@ -369 +391 @@
-                             mimetype='text/tab-separated-values',
-                             filename=filename)
-            return None
+        else:  
+            for renderer in self.renderers:  
+                if renderer.get_report_format()==format:  
+                    renderer.render(req,cols,rows)  
+                    return None  
-
-        return 'report.cs', None
-
@@ -390 +417 @@
-        if req.perm.has_permission('REPORT_SQL_VIEW'):
-            add_link(req, 'alternate', '?format=sql', 'SQL Query',
-                     'text/plain')
+        # add link for every loaded component   
+        # that implements ITicketReportRenderer               
+          
+        for renderer in self.renderers:  
+            format = renderer.get_report_format()  
+            mimetype = renderer.get_report_mimetype()  
+            linkname = renderer.get_report_linkname()  
+            linkclass = renderer.get_report_linkclass()  
+                          
+            add_link(req,'alternate','?format='+format+href,  
+                linkname,mimetype,linkclass)  
-
-    def execute_report(self, req, db, id, sql, args):
-        sql, args = self.sql_sub_vars(req, sql, args, db)