openerp-community-reviewer team mailing list archive

["Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>]

Holger Brunn (Therp) has proposed merging lp:~therp-nl/web-addons/lp1242637 into lp:web-addons.

Requested reviews:
  Web-Addons Core Editors (webaddons-core-editors)
Related bugs:
  Bug #1242637 in Web addons for OpenERP: "web_export_view creates extra entry in export format list"
  https://bugs.launchpad.net/web-addons/+bug/1242637

For more details, see:
https://code.launchpad.net/~therp-nl/web-addons/lp1242637/+merge/191969
-- 
https://code.launchpad.net/~therp-nl/web-addons/lp1242637/+merge/191969
Your team Web-Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/web-addons/lp1242637 into lp:web-addons.

=== modified file 'web_export_view/controllers.py'
--- web_export_view/controllers.py	2013-07-23 13:25:54 +0000
+++ web_export_view/controllers.py	2013-10-21 12:01:00 +0000
@@ -31,6 +31,11 @@
 class ExcelExportView(ExcelExport):
     _cp_path = '/web/export/xls_view'
 
+    def __getattribute__(self, name):
+        if name == 'fmt':
+            raise AttributeError()
+        return super(ExcelExportView, self).__getattribute__(name)
+
     @openerpweb.httprequest
     def index(self, req, data, token):
         data = json.loads(data)