Skip to Content
Menu
This question has been flagged
1 Reply
307 Views

EDIT: Thanks to Christoph for the answer, post refactored with the solution

Using Odoo 18 Entreprise, the goal is to modify the 'documents.FileViewer' QWeb template of Odoo Documents module, simply by adding the content 'TEST !' in the file viewer. 

The QWeb template is present in 'addons/documents/static/src/attachments/document_file_viewer.xml', and inherits from 'web.FileViewer' using 'primary' inheritance mode.

In order to validate that we can effectively extend this template, the aim is to add a simple text as a proof of success. The 'documents.FileViewer' template inherit from 'web.FileViewer' template available in 'odoo/addons/web/static/src/core/file_viewer/file_viewer.xml', which contains the div below

<div class="flex-grow-1"/>

Then our goal is to add the sequence below after the div above

<span>TEST !</span>

Thus, our QWeb template is as below

<?xml version="1.0" encoding="UTF-8"?>
<templates>
    <t-inherit="documents.FileViewer" t-inherit-mode="extension" t-name="al_documents.FileViewer">
        <xpath expr="//div[contains(@class, 'flex-grow-1')]" position="after">
            <span>TEST !</span>
        </xpath>
   </t>
</templates>

As can be read in the QWeb documentation, an in-place transformation (extension inheritance) will be applied on the existing 'documents.FileViewer' template. In this context 't-name' is not the name of a new template ! It is an optional reference added as a comment in the transformed template to retrace inheritance


As the Odoo Documents module manifest specifies 'documents/static/src/attachments/**/*' in the 'web.assets_backend', we will add our template in the same asset.

    'assets': {
        'web.assets_backend': [
            'al_documents/static/src/xml/patch_document_file_viewer.xml',
        ],
    },
    'depends':['base', 'web', 'documents'],



Avatar
Discard
Best Answer

If this is your full source, then no where are you telling Odoo to actually use al_documents.FileViewer as a template. You've just defined a second, additional template that is 'similar' to documents.FileViewer

Odoo will still only render documents.FileViewer, unless you explicitly tell it to use another one (https://github.com/odoo/enterprise/blob/18.0/documents/static/src/attachments/document_file_viewer.js#L8).

You are probably rather looking for this type of inheritance:

<t t-inherit="documents.FileViewer" t-inherit-mode="extension">
    <xpath expr="//div[contains(@class, 'flex-grow-1')]" position="after">
        <span>TEST !</span>
    </xpath>
</t>

See also https://www.odoo.com/documentation/18.0/developer/reference/frontend/qweb.html#template-inheritance


Avatar
Discard
Author

Dear Christoph,
many thanks for your contribution !
Amazing to learn the in-place Extension inheritance.
I wrongly though that the primary mode would apply the changes to the template specified by inherit_id.
I will edit the post with your solution.
Thanks

Related Posts Replies Views Activity
1
Jul 25
1578
1
Jun 25
951
1
May 25
1552
1
Apr 25
2179
3
Mar 25
1805