From 5998d1898e23c31a8ec05c1caaea6fdf3b104525 Mon Sep 17 00:00:00 2001
From: Dario Lodeiros <dario@aloxa.eu>
Date: Tue, 12 Mar 2019 18:02:40 +0100
Subject: [PATCH] [WIP] Views

---
 hotel/models/inherited_payment_return.py      |  4 +-
 hotel/views/hotel_folio_views.xml             | 38 +++++--------------
 .../inherited_hotel_reservation_views.xml     |  2 +-
 3 files changed, 14 insertions(+), 30 deletions(-)

diff --git a/hotel/models/inherited_payment_return.py b/hotel/models/inherited_payment_return.py
index 29cb0ca72..b137d5034 100644
--- a/hotel/models/inherited_payment_return.py
+++ b/hotel/models/inherited_payment_return.py
@@ -10,7 +10,7 @@ class PaymentReturn(models.Model):
 
     @api.multi
     def action_confirm(self):
-        pay = super(PaymentReturn,self).action_confirm()
+        pay = super(PaymentReturn, self).action_confirm()
         if pay:
             folio_ids = []
             folios = self.env['hotel.folio'].browse(folio_ids)
@@ -20,6 +20,8 @@ class PaymentReturn(models.Model):
                 ])
                 folios_line = self.env['hotel.folio'].browse(payments.mapped('folio_id.id'))
                 for folio in folios_line:
+                    if self.id not in folio.return_ids.ids:
+                        folio.update({'return_ids': [(4, self.id)]})
                     msg = _("Return of %s registered") % (line.amount)
                     folio.message_post(subject=_('Payment Return'), body=msg)
                 folios += folios_line
diff --git a/hotel/views/hotel_folio_views.xml b/hotel/views/hotel_folio_views.xml
index ea8a3dd5d..861a80e69 100644
--- a/hotel/views/hotel_folio_views.xml
+++ b/hotel/views/hotel_folio_views.xml
@@ -39,20 +39,6 @@
 
                 <sheet>
                     <div class=" oe_button_box">
-                       <button type="object" class="oe_stat_button"
-                            id="invoices_smart_button"
-                            icon="fa-thumbs-up"
-                            name="action_payments"
-                            attrs="{'invisible': [('invoices_paid','&lt;=',0)]}">
-                            <div class="o_form_field o_stat_info">
-                                <span class="o_stat_value">
-                                    <field name="invoices_paid" nolabel="1"
-                                    widget="monetary" options="{'currency_field': 'currency_id'}"/>
-                                </span>
-                                <span class="o_stat_text">Paid out</span>
-                            </div>
-                        </button>
-
                         <button type="object" class="oe_stat_button"
                             id="payment_smart_button"
                             icon="fa-money"
@@ -67,20 +53,6 @@
                             </div>
                         </button>
 
-                        <button type="object" class="oe_stat_button"
-                            id="refunds_smart_button"
-                            icon="fa-undo"
-                            name="action_return_payments"
-                            attrs="{'invisible': [('refund_amount','&lt;=',0)]}">
-                            <div class="o_form_field o_stat_info">
-                                <span class="o_stat_value">
-                                    <field name="refund_amount" nolabel="1"
-                                    widget="monetary" options="{'currency_field': 'currency_id'}"/>
-                                </span>
-                                <span class="o_stat_text">Refunds</span>
-                            </div>
-                        </button>
-
                         <button type="object" class="oe_stat_button" id="invoice_button"
                             icon="fa-pencil-square-o" name="open_invoices_folio"
                             attrs="{'invisible': [('invoice_count', '=', 0)]}"
@@ -140,6 +112,8 @@
                         <field name="invoice_ids" invisible="1"/>
                         <field name="invoice_status" invisible="1" />
                         <field name="currency_id" invisible="1"/>
+                        <field name="refund_amount" invisible="1" />
+                        <field name="invoices_paid" invisible="1" />
                     </group>
                     <notebook colspan="4" col="1">
                         <page string="Reservation Rooms">
@@ -194,6 +168,14 @@
                                 </group>
                             </group>
                         </page>
+                        <page name="payments" string="Payments" attrs="{'invisible': [('invoices_paid','&lt;=',0)]}">
+                            <field name="payment_ids"
+                                    options="{'no_create': True,'no_edit': True}" />
+                        </page>
+                        <page name="returns" string="Retun Payments" attrs="{'invisible': [('refund_amount','&lt;=',0)]}">
+                            <field name="return_ids"
+                                    options="{'no_create': True,'no_edit': True}" />
+                        </page>
                         <page string="Other data" invisible="1">
                             <group>
                                 <field name="user_id" />
diff --git a/hotel_channel_connector/views/inherited_hotel_reservation_views.xml b/hotel_channel_connector/views/inherited_hotel_reservation_views.xml
index 715ef5ce1..59f589ac3 100644
--- a/hotel_channel_connector/views/inherited_hotel_reservation_views.xml
+++ b/hotel_channel_connector/views/inherited_hotel_reservation_views.xml
@@ -21,7 +21,7 @@
                 /> -->
             </xpath>
             <xpath expr="//page[@name='days']" position="after">
-                <page name="connector" string="Channel Connector">
+                <page name="connector" string="Channel Connector" groups="connector.group_connector_manager">
                     <group string="Hotel Channel Bindings">
                         <field name="channel_bind_ids" nolabel="1">
                             <tree>