From 716ff76d5e639654171a8795461282f3e80a74dc Mon Sep 17 00:00:00 2001
From: Julien Castiaux <juc@odoo.com>
Date: Tue, 19 Mar 2019 15:28:24 +0000
Subject: [PATCH] [FIX] sale: force maximum section/subtotal width

Step to reproduce:
- Install sale (sale_management) and studio (web_studio)
- Change the document template (settings > configuration > general
settings) to select the second one
- Go to an order form
- Enable studio, select "Reports" in the right tab
- Edit the "Quotation/Order" report, add a new colomn
- Back to the order form, enable dev mode and add a section to the order
lines.
- Print the report => the section is not using all the available place.

The section/subtotal are using a colspan to use all the place in the
table, this colspan is hardcoded to 5, the number of columns without
customization. Adding a new column doesn't change that value.

The fix has been to force the colspan at 99, the maximum value for
colspans. Using that value will always render the row to fill the table.

opw-1936088

closes odoo/odoo#31951

Signed-off-by: Julien Castiaux <Julien00859@users.noreply.github.com>
---
 addons/sale/report/sale_report_templates.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/sale/report/sale_report_templates.xml b/addons/sale/report/sale_report_templates.xml
index b438cbd3db2..edce182dcfc 100644
--- a/addons/sale/report/sale_report_templates.xml
+++ b/addons/sale/report/sale_report_templates.xml
@@ -116,14 +116,14 @@
                                 </td>
                             </t>
                             <t t-if="line.display_type == 'line_section'">
-                                <td t-att-colspan="colspan">
+                                <td t-att-colspan="99">
                                     <span t-field="line.name"/>
                                 </td>
                                 <t t-set="current_section" t-value="line"/>
                                 <t t-set="current_subtotal" t-value="0"/>
                             </t>
                             <t t-if="line.display_type == 'line_note'">
-                                <td t-att-colspan="colspan">
+                                <td t-att-colspan="99">
                                     <span t-field="line.name"/>
                                 </td>
                             </t>
@@ -131,7 +131,7 @@
 
                         <t t-if="current_section and (line_last or doc.order_line[line_index+1].display_type == 'line_section')">
                             <tr class="is-subtotal text-right">
-                                <td t-att-colspan="colspan">
+                                <td t-att-colspan="99">
                                     <strong class="mr16">Subtotal</strong>
                                     <span
                                         t-esc="current_subtotal"