From b08d7ef6bd0f1b8a36d82e238d563a37c7fa9057 Mon Sep 17 00:00:00 2001
From: Vo Minh Thu <vmt@openerp.com>
Date: Fri, 22 Mar 2013 16:39:55 +0100
Subject: [PATCH] [REF] ir.actions.report.xml: register_all do not make sense
 any more:

auto=True reports are looked up in the database for each rendering, so they do
no have to be in the report registry any longer.
auto=False reports will register themselves but at this point
they can be updated to use the new `parser` XML attribute.

bzr revid: vmt@openerp.com-20130322153955-s6nyux2pyez6c01w
---
 openerp/addons/base/ir/ir_actions.py | 22 +---------------------
 openerp/modules/registry.py          |  1 -
 2 files changed, 1 insertion(+), 22 deletions(-)

diff --git a/openerp/addons/base/ir/ir_actions.py b/openerp/addons/base/ir/ir_actions.py
index 9a76d5c28fc..b32617d0134 100644
--- a/openerp/addons/base/ir/ir_actions.py
+++ b/openerp/addons/base/ir/ir_actions.py
@@ -85,27 +85,6 @@ class report_xml(osv.osv):
                 res[report.id] = False
         return res
 
-    def register_all(self, cr):
-        """Report registration handler that may be overridden by subclasses to
-           add their own kinds of report services.
-           Loads all reports with no manual loaders (auto==True) and
-           registers the appropriate services to implement them.
-        """
-        opj = os.path.join
-        cr.execute("SELECT * FROM ir_act_report_xml WHERE auto=%s ORDER BY id", (True,))
-        result = cr.dictfetchall()
-        reports = openerp.report.interface.report_int._reports
-        for r in result:
-            if reports.has_key('report.'+r['report_name']):
-                continue
-            if r['report_rml'] or r['report_rml_content_data']:
-                report_sxw('report.'+r['report_name'], r['model'],
-                        opj('addons',r['report_rml'] or '/'), header=r['header'])
-            if r['report_xsl']:
-                report_rml('report.'+r['report_name'], r['model'],
-                        opj('addons',r['report_xml']),
-                        r['report_xsl'] and opj('addons',r['report_xsl']))
-
     def render_report(self, cr, uid, ids, name, data, context=None):
         """
         Look up a report definition and render the report for the provided IDs.
@@ -117,6 +96,7 @@ class report_xml(osv.osv):
 
         # First lookup in the deprecated place, because if the report definition
         # has not been updated, it is more likely the correct definition is there.
+        # Only reports with custom parser sepcified in Python are still there.
         if 'report.' + name in openerp.report.interface.report_int._reports:
             new_report = openerp.report.interface.report_int._reports['report.' + name]
         else:
diff --git a/openerp/modules/registry.py b/openerp/modules/registry.py
index b06c500a24c..89cb6143a59 100644
--- a/openerp/modules/registry.py
+++ b/openerp/modules/registry.py
@@ -225,7 +225,6 @@ class RegistryManager(object):
             try:
                 Registry.setup_multi_process_signaling(cr)
                 registry.do_parent_store(cr)
-                registry.get('ir.actions.report.xml').register_all(cr)
                 cr.commit()
             finally:
                 cr.close()