Merge pull request #791 from odoo/revert-483-saas-5-report-translation-fix-sle
authorChristophe Simonis <KangOl@users.noreply.github.com>
Thu, 26 Jun 2014 16:23:39 +0000 (18:23 +0200)
committerChristophe Simonis <KangOl@users.noreply.github.com>
Thu, 26 Jun 2014 16:23:39 +0000 (18:23 +0200)
In stable version, the code (js and python) must **not** rely on new data existance (here qweb template).

addons/report/controllers/main.py
addons/report/models/report.py
addons/report/views/layouts.xml
addons/sale/views/report_saleorder.xml
addons/website/views/website_templates.xml
addons/website_livechat/views/website_livechat.xml
addons/website_report/views/layouts.xml

index 01992d6..c6bc13d 100644 (file)
@@ -50,12 +50,7 @@ class ReportController(Controller):
         if data.get('options'):
             options_data = simplejson.loads(data['options'])
         if data.get('context'):
-            # Ignore 'lang' here, because the context in data is the one from the webclient *but* if
-            # the user explicitely wants to change the lang, this mechanism overwrites it. 
-            data_context = simplejson.loads(data['context'])
-            if data_context.get('lang'):
-                del data_context['lang']
-            context.update(data_context)
+            context.update(simplejson.loads(data['context']))
 
         if converter == 'html':
             html = report_obj.get_html(cr, uid, docids, reportname, data=options_data, context=context)
index 5bfc797..cbbee7f 100644 (file)
@@ -34,8 +34,10 @@ import lxml.html
 import cStringIO
 import subprocess
 from distutils.version import LooseVersion
-from functools import partial
-from pyPdf import PdfFileWriter, PdfFileReader
+try:
+    from pyPdf import PdfFileWriter, PdfFileReader
+except ImportError:
+    PdfFileWriter = PdfFileReader = None
 
 
 _logger = logging.getLogger(__name__)
@@ -69,6 +71,23 @@ class Report(osv.Model):
 
     public_user = None
 
+    MINIMAL_HTML_PAGE = """
+<base href="{base_url}">
+<!DOCTYPE html>
+<html style="height: 0;">
+    <head>
+        <link href="/report/static/src/css/reset.min.css" rel="stylesheet"/>
+        <link href="/web/static/lib/bootstrap/css/bootstrap.css" rel="stylesheet"/>
+        <link href="/website/static/src/css/website.css" rel="stylesheet"/>
+        <link href="/web/static/lib/fontawesome/css/font-awesome.css" rel="stylesheet"/>
+        <style type='text/css'>{css}</style>
+        {subst}
+    </head>
+    <body class="container" onload="subst()">
+        {body}
+    </body>
+</html>"""
+
     #--------------------------------------------------------------------------
     # Extension of ir_ui_view.render with arguments frequently used in reports
     #--------------------------------------------------------------------------
@@ -115,7 +134,6 @@ class Report(osv.Model):
         website = None
         if request and hasattr(request, 'website'):
             website = request.website
-            context.update(translatable=context.get('lang') != request.website.default_lang_code)
         values.update({
             'time': time,
             'translate_doc': translate_doc,
@@ -179,11 +197,6 @@ class Report(osv.Model):
         footerhtml = []
         base_url = self.pool['ir.config_parameter'].get_param(cr, uid, 'web.base.url')
 
-        # Minimal page renderer
-        view_obj = self.pool['ir.ui.view']
-        render_minimal = partial(view_obj.render, cr, uid, 'report.minimal_layout', context=context)
-
-
         # The received html report must be simplified. We convert it in a xml tree
         # in order to extract headers, bodies and footers.
         try:
@@ -194,12 +207,12 @@ class Report(osv.Model):
 
             for node in root.xpath("//div[@class='header']"):
                 body = lxml.html.tostring(node)
-                header = render_minimal(dict(css=css, subst=subst, body=body, base_url=base_url))
+                header = self.MINIMAL_HTML_PAGE.format(css=css, subst=subst, body=body, base_url=base_url)
                 headerhtml.append(header)
 
             for node in root.xpath("//div[@class='footer']"):
                 body = lxml.html.tostring(node)
-                footer = render_minimal(dict(css=css, subst=subst, body=body, base_url=base_url))
+                footer = self.MINIMAL_HTML_PAGE.format(css=css, subst=subst, body=body, base_url=base_url)
                 footerhtml.append(footer)
 
             for node in root.xpath("//div[@class='page']"):
@@ -216,7 +229,7 @@ class Report(osv.Model):
                     reportid = False
 
                 body = lxml.html.tostring(node)
-                reportcontent = render_minimal(dict(css=css, subst='', body=body, base_url=base_url))
+                reportcontent = self.MINIMAL_HTML_PAGE.format(css=css, subst='', body=body, base_url=base_url)
 
                 # FIXME: imo the best way to extract record id from html reports is by using the
                 # qweb branding. As website editor is not yet splitted in a module independant from
index e5ce9a8..f9c0ea7 100644 (file)
@@ -1,41 +1,55 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
 <data>
-<record model="ir.ui.view" id="layout">
-    <field name="name">report.layout</field>
-    <field name="inherit_id" ref="web.layout"></field>
-    <field name="mode">primary</field>
-    <field name="type">qweb</field>
-    <field name="arch" type="xml">
-        <!-- Add report attributes -->
-        <xpath expr="//html" position="attributes">
-            <attribute name="t-att-data-report-margin-top">data_report_margin_top if data_report_margin_top else None</attribute>
-            <attribute name="t-att-data-report-header-spacing">data_report_header_spacing if data_report_header_spacing else None</attribute>
-            <attribute name="t-att-data-report-dpi">data_report_dpi if data_report_dpi else None</attribute>
-        </xpath>
-        <!-- Add report style -->
-        <xpath expr="//head" position="inside">
-            <link href="/web/static/lib/bootstrap/css/bootstrap.css" rel="stylesheet"/>
-            <link href="/website/static/src/css/website.css" rel="stylesheet"/>
-            <link href="/web/static/lib/fontawesome/css/font-awesome.css" rel="stylesheet"/>
+<template id="html_container">
+    &lt;!DOCTYPE html&gt;
+    <html t-att-lang="lang and lang.replace('_', '-')"
+          t-att-data-editable="'1' if editable else None"
+          t-att-data-view-xmlid="xmlid if editable else None"
+          t-att-data-main-object="repr(main_object) if editable else None"
+          t-att-data-report-margin-top="data_report_margin_top if data_report_margin_top else None"
+          t-att-data-report-header-spacing="data_report_header_spacing if data_report_header_spacing else None"
+          t-att-data-report-dpi="data_report_dpi if data_report_dpi else None"
+          t-att-data-oe-company-name="res_company.name">
+        <head>
+            <meta name="viewport" content="width=device-width, initial-scale=1"/>
+
+            <t t-if="main_object and 'website_meta_title' in main_object">
+                <t t-set="title" t-value="main_object.website_meta_title"/>
+            </t>
+            <t t-if="not title and main_object and 'name' in main_object">
+                <t t-set="additional_title" t-value="main_object.name"/>
+            </t>
+            <meta name="description" t-att-value="main_object and 'website_meta_description' in main_object
+                and main_object.website_meta_description or website_meta_description"/>
+            <meta name="keywords" t-att-value="main_object and 'website_meta_keywords' in main_object
+                and main_object.website_meta_keywords or website_meta_keywords"/>
+
+            <link id="bootstrap_css" rel='stylesheet' href='/web/static/lib/bootstrap/css/bootstrap.css' t-ignore="true"/>
+            <link rel="stylesheet" type="text/css"  href='/website/static/src/css/website.css'/>
             <style type="text/css">
                 <t t-call="report.style"/>
             </style>
-        </xpath>
-        <!-- Remove conflicting style -->
-        <xpath expr="//head/link[@href='/web/static/src/css/full.css']" position="replace"></xpath>
-        <!-- Correct view inheritance -->
-        <xpath expr="//t[@t-name='web.layout']" position="attributes">
-            <attribute name="t-name">report.layout</attribute>
-        </xpath>
-    </field>
-</record>
 
-<template id="html_container">
-    <t t-set="body_classname" t-value="'container'"/>
-    <t t-call="report.layout">
-        <t t-raw="0"/>
-    </t>
+            <link rel='stylesheet' href='/web/static/lib/fontawesome/css/font-awesome.css'/>
+
+            <script type="text/javascript" src="/web/static/lib/es5-shim/es5-shim.min.js"></script>
+            <script type="text/javascript" src="/web/static/lib/underscore/underscore.js"></script>
+            <script type="text/javascript" src="/web/static/lib/underscore.string/lib/underscore.string.js"></script>
+            <script type="text/javascript" src="/web/static/lib/jquery/jquery.js"></script>
+            <script type="text/javascript" src="/web/static/lib/bootstrap/js/bootstrap.js"></script>
+            <script type="text/javascript" src="/web/static/lib/qweb/qweb2.js"></script>
+            <script type="text/javascript" src="/web/static/src/js/openerpframework.js"></script>
+            <script type="text/javascript" src="/website/static/src/js/website.js"></script>
+            <script t-if="not translatable" type="text/javascript" src="/website/static/src/js/website.snippets.animation.js"></script>
+            <t t-raw="head or ''" name='layout_head'/>
+        </head>
+        <body class="container">
+            <div id="wrapwrap">
+                <t t-raw="0"/>
+            </div>
+        </body>
+    </html>
 </template>
 
 <template id="style">
     <t t-raw="0" />
 </template>
 
-<template id="minimal_layout">
-    <t t-raw="'&lt;base href=%s&gt;' % base_url"/>
-    &lt;!DOCTYPE html&gt;
-    <html style="height: 0;">
-        <head>
-            <link href="/report/static/src/css/reset.min.css" rel="stylesheet"/>
-            <link href="/web/static/lib/bootstrap/css/bootstrap.css" rel="stylesheet"/>
-            <link href="/website/static/src/css/website.css" rel="stylesheet"/>
-            <link href="/web/static/lib/fontawesome/css/font-awesome.css" rel="stylesheet"/>
-            <style type='text/css'><t t-raw="css"/></style>
-            <t t-raw="subst"/>
-        </head>
-        <body class="container" onload="subst()">
-            <t t-raw="body"/>
-        </body>
-    </html>
-</template>
-
 </data>
 </openerp>
index a677182..5c198c6 100644 (file)
@@ -65,7 +65,7 @@
                <tbody class="sale_tbody">
                     <tr t-foreach="o.order_line" t-as="l">
                         <td>
-                           <span t-field="l.name"/>
+                           <span  t-field="l.name"/>
                         </td>
                         <td>
                             <span t-esc="', '.join(map(lambda x: x.name, l.tax_id))"/>
index 1c96529..df3a429 100644 (file)
@@ -57,7 +57,7 @@
 
 <template id="layout" name="Main layout">&lt;!DOCTYPE html&gt;
     <html t-att-lang="lang and lang.replace('_', '-')"
-          t-att-data-website-id="website.id if editable and website else None"
+          t-att-data-website-id="website.id if editable else None"
           t-att-data-editable="'1' if editable else None"
           t-att-data-translatable="'1' if translatable else None"
           t-att-data-view-xmlid="xmlid if editable else None"
                 and main_object.website_meta_keywords or website_meta_keywords"/>
             <title><t t-esc="title"/></title>
 
-            <t t-set="languages" t-value="website.get_languages() if website else None"/>
-            <t t-if="request and request.website_multilang">
+            <t t-set="languages" t-value="website.get_languages()"/>
+            <t t-if="request.website_multilang">
                 <t t-foreach="languages" t-as="lg">
-                    <t t-set="force_lang" t-value="lg[0] if website and lg[0] != website.default_lang_code else None"/>
+                    <t t-set="force_lang" t-value="lg[0] if lg[0] != website.default_lang_code else None"/>
                     <link rel="alternate" t-att-href="url_for(request.httprequest.path + '?' + keep_query(), lang=force_lang)" t-att-hreflang="lg[0].replace('_', '-').lower()" />
                 </t>
             </t>
                     </div>
                 </footer>
             </div>
-            <t t-if="website and website.google_analytics_key">
+            <t t-if="website.google_analytics_key">
                 <script>
                     (function(b,o,i,l,e,r){b.GoogleAnalyticsObject=l;b[l]||(b[l]=
                     function(){(b[l].q=b[l].q||[]).push(arguments)});b[l].l=+new Date;
index ef6b2a9..374e0d5 100644 (file)
@@ -4,7 +4,7 @@
 
 <template id="header" inherit_id="website.layout" name="LiveChat Snippet">
     <xpath expr="//body" position="inside">
-        <t t-if="website and website.channel_id">
+        <t t-if="website.channel_id">
             <t t-raw="website.channel_id.script"/>
         </t>
     </xpath>
index 3f8722b..9978d99 100644 (file)
@@ -1,57 +1,93 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
 <data>
+<template id="editor_head" inherit_id="report.html_container" name="Editor" groups="base.group_website_publisher">
+    <xpath expr="//html" position="attributes">
+        <attribute name="t-att-data-website-id">website.id if editable and website else None</attribute>
+        <attribute name="t-att-data-translatable">'1' if translatable else None</attribute>
+    </xpath>
+
+    <xpath expr="//body" position="attributes">
+        <attribute name="style">padding-top: 51px;</attribute>
+    </xpath>
+    <xpath expr="//body" position="inside">
+        <div id="website-top-navbar-placeholder" class="navbar navbar-inverse navbar-fixed-top hidden-xs">
+            <div class="navbar-header">
+                <form class="navbar-form navbar-left">
+                    <button type="button" class="btn btn-primary">Edit</button>
+                </form>
+            </div>
+            <div class="collapse navbar-collapse navbar-edit-collapse">
+                <ul class="nav navbar-nav navbar-right">
+                    <li><a href="#" onclick="return false;"><i class="fa fa-mobile" title="Mobile preview"/></a></li>
+                    <li class="divider-vertical"/>
+                    <li><a href="#" onclick="return false;"><span title="Promote page on the web">Promote</span></a></li>
+                    <li class="dropdown">
+                        <a href="#" onclick="return false;">Content <span class="caret"/></a>
+                    </li>
+                    <li class="dropdown">
+                        <a href="#" onclick="return false;">Customize <span class="caret"/></a>
+                    </li>
+                    <li class="dropdown">
+                        <a href="#" onclick="return false;">Help <span class="caret"/></a>
+                    </li>
+                </ul>
+            </div>
+        </div>
+    </xpath>
+    <xpath expr='//t[@name="layout_head"]' position="before">
+        <link rel='stylesheet' href='/website/static/src/css/snippets.css'/>
+        <link rel='stylesheet' href='/website/static/src/css/editor.css'/>
+
+
+        <link rel="stylesheet" href="/web/static/lib/select2/select2.css"/>
+        <link rel="stylesheet" href="/website/static/lib/select2-bootstrap-css/select2-bootstrap.css"/>
+        <link rel='stylesheet' href="/web/static/lib/jquery.ui/css/smoothness/jquery-ui-1.9.1.custom.css"/>
 
-<record model="ir.ui.view" id="layout">
-    <field name="name">website_report.layout</field>
-    <field name="inherit_id" ref="website.layout"></field>
-    <field name="mode">primary</field>
-    <field name="type">qweb</field>
-    <field name="arch" type="xml">
-        <!-- Add report attributes -->
-        <xpath expr="//html" position="attributes">
-            <attribute name="t-att-data-report-margin-top">data_report_margin_top if data_report_margin_top else None</attribute>
-            <attribute name="t-att-data-report-header-spacing">data_report_header_spacing if data_report_header_spacing else None</attribute>
-            <attribute name="t-att-data-report-dpi">data_report_dpi if data_report_dpi else None</attribute>
-        </xpath>
-        <!-- Add report style -->
-        <xpath expr="//head" position="inside">
-            <style type="text/css">
-                <t t-call="report.style"/>
-            </style>
-        </xpath>
-        <xpath expr="//body" position="attributes">
-            <attribute name="class">container</attribute>
-        </xpath>
-        <xpath expr="//header" position="replace"/>
-        <xpath expr="//footer" position="replace">
-            <ul class="list-inline js_language_selector mt16" t-if="(request and request.website_multilang and len(languages) &gt; 1) or (website and editable)">
+        <script type="text/javascript" src="/web/static/lib/select2/select2.js"></script>
+        <script type="text/javascript" src="/web/static/lib/ckeditor/ckeditor.js"></script>
+        <script type="text/javascript" src="/website/static/lib/ace/ace.js"></script>
+        <script type="text/javascript" src="/website/static/lib/vkbeautify/vkbeautify.0.99.00.beta.js"></script>
+        <script type="text/javascript" src="/web/static/lib/jquery.ui/js/jquery-ui-1.9.1.custom.js"></script>
+        <!-- mutation observers shim backed by mutation events (8 < IE < 11, Safari < 6, FF < 14, Chrome < 17) -->
+        <script type="text/javascript" src="/website/static/lib//jquery.mjs.nestedSortable/jquery.mjs.nestedSortable.js"></script>
+        <script type="text/javascript" src="/website/static/lib/MutationObservers/test/sidetable.js"></script>
+        <script type="text/javascript" src='/website/static/lib/nearest/jquery.nearest.js'></script>
+        <script type="text/javascript" src="/website/static/lib/MutationObservers/MutationObserver.js"></script>
+
+        <script type="text/javascript" src="/website/static/lib/jquery.placeholder/jquery.placeholder.js"></script>
+
+        <script type="text/javascript" src="/website/static/src/js/website.editor.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.editor.newpage.js"></script> <!-- groups="base.group_website_designer" -->
+        <script type="text/javascript" src="/website/static/src/js/website.menu.js"></script> <!-- groups="base.group_website_designer" -->
+        <script type="text/javascript" src="/website/static/src/js/website.mobile.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.seo.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.tour.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.tour.banner.js"></script> <!-- groups="base.group_website_designer" -->
+        <script type="text/javascript" src="/website/static/src/js/website.snippets.editor.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.ace.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/website.translator.js"></script>
+        <script type="text/javascript" src="/website/static/src/js/jQuery.transfo.js"></script>
+    </xpath>
+
+    <xpath expr='//body[@class="container"]/div[@id="wrapwrap"]' position="before">
+        <t t-set="languages" t-value="website.get_languages() if website else None"/>
+        <t t-if="languages">
+            <ul class="list-inline js_language_selector mt16" t-if="(len(languages) &gt; 1 or editable)">
                 <li t-foreach="languages" t-as="lg">
-                    <a t-att-href="url_for(request.httprequest.path + '?' + keep_query(), lang=lg[0])"
-                       t-att-data-default-lang="editable and 'true' if website and lg[0] == website.default_lang_code else None">
+                    <a t-att-href="url_for('', lang=lg[0]) + '?' + keep_query()"
+                       t-att-data-default-lang="editable and 'true' if lg[0] == website.default_lang_code else None">
                         <t t-esc="lg[1].split('/').pop()"/>
                     </a>
                 </li>
                 <li groups="base.group_website_publisher">
                     <t t-set="url_return" t-value="url_for('', '[lang]') + '?' + keep_query()"/>
-                    <a t-attf-href="/web#action=base.action_view_base_language_install&amp;website_id=#{website.id if website else ''}&amp;url_return=#{url_return}">
+                    <a t-attf-href="/web#action=base.action_view_base_language_install&amp;website_id=#{website.id}&amp;url_return=#{url_return}">
                         <i class="fa fa-plus-circle"/>
                         Add a language...
                     </a>
                 </li>
             </ul>
-        </xpath>
-        <!-- Correct view inheritance -->
-        <xpath expr="//t[@t-name='website.layout']" position="attributes">
-            <attribute name="t-name">website_report.layout</attribute>
-        </xpath>
-    </field>
-</record>
-
-<template id="website_html_container" inherit_id="report.html_container">
-    <xpath expr="//t[@t-call='report.layout']" position="replace">
-        <t t-call="website_report.layout">          
-            <t t-raw="0"/>
         </t>
     </xpath>
 </template>