From 73987067fc9dbfe9f68d2445b273e31f87ddec04 Mon Sep 17 00:00:00 2001
From: qsm-odoo <qsm@odoo.com>
Date: Wed, 5 Dec 2018 10:02:20 +0000
Subject: [PATCH] [REF] website_sale: use new colored cards

Since https://github.com/odoo/odoo/pull/29253, we change the way cards
can be colored by always using the background color class on the card
element itself instead of its header and footer (and by changing the
way it looks by doing so).

This commit removes the use of the old way (introduced by
https://github.com/odoo/odoo/commit/64719b71d5997c0463a9f8cde4fb8d1f5c88967a).
Note that this commit leaves the `bg-*-light` definitions for now; they
are meant to be improved in the future anyway.

closes odoo/odoo#29253
---
 addons/website_sale/views/templates.xml | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/addons/website_sale/views/templates.xml b/addons/website_sale/views/templates.xml
index 401966fc0f3..527ea5c862d 100644
--- a/addons/website_sale/views/templates.xml
+++ b/addons/website_sale/views/templates.xml
@@ -1532,12 +1532,12 @@
     <template id="payment_confirmation_status">
         <div class="oe_website_sale_tx_status mt-3" t-att-data-order-id="order.id">
             <t t-set="payment_tx_id" t-value="order.get_portal_last_transaction()"/>
-            <div class="card">
-                <div t-att-class="'card-header %s' % (
-                    (payment_tx_id.state == 'pending' and 'bg-info') or
-                    (payment_tx_id.state == 'done' and 'bg-success') or
-                    (payment_tx_id.state == 'authorized' and 'bg-success') or
-                    'bg-danger')">
+            <div t-attf-class="card #{
+                (payment_tx_id.state == 'pending' and 'bg-info') or
+                (payment_tx_id.state == 'done' and 'bg-success') or
+                (payment_tx_id.state == 'authorized' and 'bg-success') or
+                'bg-danger'}">
+                <div class="card-header">
                     <a role="button" groups="base.group_system" class="btn btn-sm btn-link text-white float-right" target="_blank" aria-label="Edit" title="Edit"
                             t-att-href="'/web#model=%s&amp;id=%s&amp;action=%s&amp;view_type=form' % ('payment.acquirer', payment_tx_id.acquirer_id.id, 'payment.action_payment_acquirer')">
                         <i class="fa fa-pencil"></i>
@@ -1560,11 +1560,7 @@
                         <span t-else="">Your payment has been authorized.</span>
                     </t>
                 </div>
-                <div t-if="payment_tx_id.acquirer_id.post_msg" t-att-class="'card-body %s' % (
-                    (payment_tx_id.state == 'pending' and 'bg-info-light') or
-                    (payment_tx_id.state == 'done' and 'bg-success-light') or
-                    (payment_tx_id.state == 'authorized' and 'bg-success-light') or
-                    'bg-danger-light')">
+                <div t-if="payment_tx_id.acquirer_id.post_msg" class="card-body">
                     <t t-raw="payment_tx_id.acquirer_id.post_msg" />
                     <div t-if="payment_tx_id.acquirer_id.provider == 'transfer' and order.reference">
                         <b>Communication: </b><span t-esc='order.reference'/>