Permalink
Browse files

[FIX] web: show page number in report

Following 4a3fd02 on the portal when displaying an invoice we may get
a report_type variable (with value 'html' or 'pdf') on the portal.

This was used to not display the page number when checking the report,
but when the report is printed in the backend this report_type was unset
so the page number was mistakenly removed too.

With this changeset, we only check the condition that we require to only
remove page number when report is html embedded on portal.

opw-1924729
closes #30095
  • Loading branch information...
nle-odoo committed Jan 10, 2019
1 parent f1df5f3 commit 42f2a22e700ec3daf1c5ce8df149991251f9d734
Showing with 3 additions and 3 deletions.
  1. +3 −3 addons/web/views/report_templates.xml
@@ -194,7 +194,7 @@
<li t-if="company.vat" class="list-inline-item"><i class="fa fa-building-o" role="img" aria-label="Fiscal number"/><t t-esc="company.country_id.vat_label or 'Tax ID'"/>: <span t-field="company.vat"/></li>
</ul>
<div t-field="company.report_footer"/>
<div t-if="report_type == 'pdf'" class="text-muted">
<div t-if="report_type != 'html'" class="text-muted">
Page:
<span class="page"/>
of
@@ -236,7 +236,7 @@
<li t-if="company.vat" class="list-inline-item"><t t-esc="company.country_id.vat_label or 'Tax ID'"/>: <span t-field="company.vat"/></li>
</ul>
<div t-field="company.report_footer"/>
<div t-if="report_type == 'pdf'">
<div t-if="report_type != 'html'">
Page: <span class="page"/> / <span class="topage"/>
</div>
</div>
@@ -329,7 +329,7 @@
<span t-field="company.report_footer"/>
</div>

<div t-if="report_type == 'pdf'" class="text-muted">
<div t-if="report_type != 'html'" class="text-muted">
Page: <span class="page"/> / <span class="topage"/>
</div>
</div>

0 comments on commit 42f2a22

Please sign in to comment.