From f621deac187e7482bffbf13c90f97e8fd06d9480 Mon Sep 17 00:00:00 2001
From: Mayank Patel <mayank@hibou.io>
Date: Wed, 11 Sep 2024 05:50:48 +0000
Subject: [PATCH] [REM] sale_payment_web: available in professional

H14528
---
 sale_payment_web/__init__.py                  |   2 -
 sale_payment_web/__manifest__.py              |  28 ----
 sale_payment_web/models/__init__.py           |   2 -
 sale_payment_web/models/payment.py            |  18 ---
 sale_payment_web/models/sale.py               |  35 -----
 sale_payment_web/security/ir.model.access.csv |   3 -
 sale_payment_web/security/sale_security.xml   |   9 --
 sale_payment_web/tests/__init__.py            |   1 -
 sale_payment_web/tests/test_sale_payment.py   |  49 -------
 sale_payment_web/views/payment_views.xml      |  15 --
 sale_payment_web/views/sale_views.xml         |  27 ----
 sale_payment_web/wizard/__init__.py           |   1 -
 .../wizard/account_payment_register.py        | 129 ------------------
 .../wizard/account_payment_register_views.xml |  28 ----
 14 files changed, 347 deletions(-)
 delete mode 100644 sale_payment_web/__init__.py
 delete mode 100755 sale_payment_web/__manifest__.py
 delete mode 100644 sale_payment_web/models/__init__.py
 delete mode 100644 sale_payment_web/models/payment.py
 delete mode 100644 sale_payment_web/models/sale.py
 delete mode 100644 sale_payment_web/security/ir.model.access.csv
 delete mode 100644 sale_payment_web/security/sale_security.xml
 delete mode 100644 sale_payment_web/tests/__init__.py
 delete mode 100644 sale_payment_web/tests/test_sale_payment.py
 delete mode 100644 sale_payment_web/views/payment_views.xml
 delete mode 100644 sale_payment_web/views/sale_views.xml
 delete mode 100644 sale_payment_web/wizard/__init__.py
 delete mode 100644 sale_payment_web/wizard/account_payment_register.py
 delete mode 100644 sale_payment_web/wizard/account_payment_register_views.xml

diff --git a/sale_payment_web/__init__.py b/sale_payment_web/__init__.py
deleted file mode 100644
index 9b429614..00000000
--- a/sale_payment_web/__init__.py
+++ /dev/null
@@ -1,2 +0,0 @@
-from . import models
-from . import wizard
diff --git a/sale_payment_web/__manifest__.py b/sale_payment_web/__manifest__.py
deleted file mode 100755
index 22d6e9b0..00000000
--- a/sale_payment_web/__manifest__.py
+++ /dev/null
@@ -1,28 +0,0 @@
-{
-    'name': 'Sale Payment Web',
-    'author': 'Hibou Corp. <hello@hibou.io>',
-    'category': 'Sales',
-    'version': '15.0.1.0.0',
-    'description':
-        """
-Sale Payment Web
-================
-
-Allow sales people to register payments for sale orders.
-
-Electronic payments will create transactions and automatically reconcile on the invoice.
-
-        """,
-    'depends': [
-        'payment',
-        'sale',
-    ],
-    'auto_install': False,
-    'data': [
-        'security/sale_security.xml',
-        'security/ir.model.access.csv',
-        'views/payment_views.xml',
-        'views/sale_views.xml',
-        'wizard/account_payment_register_views.xml',
-    ],
-}
diff --git a/sale_payment_web/models/__init__.py b/sale_payment_web/models/__init__.py
deleted file mode 100644
index b70704c4..00000000
--- a/sale_payment_web/models/__init__.py
+++ /dev/null
@@ -1,2 +0,0 @@
-from . import payment
-from . import sale
diff --git a/sale_payment_web/models/payment.py b/sale_payment_web/models/payment.py
deleted file mode 100644
index b205575c..00000000
--- a/sale_payment_web/models/payment.py
+++ /dev/null
@@ -1,18 +0,0 @@
-from odoo import api, fields, models
-
-
-class PaymentTransaction(models.Model):
-    _inherit = 'payment.transaction'
-
-    @api.model
-    def create(self, values):
-        active_ids = self._context.get('active_ids')
-        if active_ids and self._context.get('active_model') == 'sale.order':
-            values['sale_order_ids'] = [(6, 0, active_ids)]
-        return super(PaymentTransaction, self).create(values)
-
-
-class AccountPayment(models.Model):
-    _inherit = 'account.payment'
-
-    sale_order_id = fields.Many2one('sale.order', string="Sale Order")
diff --git a/sale_payment_web/models/sale.py b/sale_payment_web/models/sale.py
deleted file mode 100644
index dd002d90..00000000
--- a/sale_payment_web/models/sale.py
+++ /dev/null
@@ -1,35 +0,0 @@
-from odoo import api, fields, models, _
-from odoo.tools.safe_eval import safe_eval
-
-
-class SaleOrder(models.Model):
-    _inherit = 'sale.order'
-
-    manual_payment_ids = fields.One2many('account.payment', 'sale_order_id', string='Manual Payments')
-    manual_amount_registered_payment = fields.Monetary('Manually Registered Amount', compute='_compute_manual_amount_registered_payment')
-    manual_amount_remaining = fields.Monetary('Remaining Amount Due', compute='_compute_manual_amount_registered_payment')
-
-    @api.depends('manual_payment_ids.amount', 'amount_total')
-    def _compute_manual_amount_registered_payment(self):
-        for so in self:
-            so.manual_amount_registered_payment = sum(so.manual_payment_ids.mapped('amount'))
-            so.manual_amount_remaining = so.amount_total - so.manual_amount_registered_payment
-
-
-    def action_manual_payments(self):
-        action = self.env.ref('account.action_account_payments').read()[0]
-        domain = action['domain'] or '[]'
-        domain = safe_eval(domain)
-        domain.append(('id', 'in', self.manual_payment_ids.ids))
-        action['domain'] = domain
-        return action
-
-    def action_payment_register(self):
-        return {
-            'name': _('Register Payment'),
-            'res_model': 'account.payment.register',
-            'view_mode': 'form',
-            'context': {'active_ids': self.ids, 'active_model': 'sale.order'},
-            'target': 'new',
-            'type': 'ir.actions.act_window',
-        }
diff --git a/sale_payment_web/security/ir.model.access.csv b/sale_payment_web/security/ir.model.access.csv
deleted file mode 100644
index 1cd7e0de..00000000
--- a/sale_payment_web/security/ir.model.access.csv
+++ /dev/null
@@ -1,3 +0,0 @@
-id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_payment_method,account.payment.method,account.model_account_payment_method,group_payment_web,1,0,0,0
-access_payment,account.payment,account.model_account_payment,group_payment_web,1,1,1,1
diff --git a/sale_payment_web/security/sale_security.xml b/sale_payment_web/security/sale_security.xml
deleted file mode 100644
index 7234cc38..00000000
--- a/sale_payment_web/security/sale_security.xml
+++ /dev/null
@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<odoo>
-
-    <record id="group_payment_web" model="res.groups">
-        <field name="name">Sale Order Payments</field>
-        <field name="category_id" ref="base.module_category_hidden"/>
-    </record>
-
-</odoo>
diff --git a/sale_payment_web/tests/__init__.py b/sale_payment_web/tests/__init__.py
deleted file mode 100644
index 35d768d5..00000000
--- a/sale_payment_web/tests/__init__.py
+++ /dev/null
@@ -1 +0,0 @@
-from . import test_sale_payment
diff --git a/sale_payment_web/tests/test_sale_payment.py b/sale_payment_web/tests/test_sale_payment.py
deleted file mode 100644
index 50d180cf..00000000
--- a/sale_payment_web/tests/test_sale_payment.py
+++ /dev/null
@@ -1,49 +0,0 @@
-from odoo.addons.sale.tests.test_sale_to_invoice import TestSaleToInvoice
-from odoo.exceptions import UserError
-from odoo.tests import tagged
-
-
-@tagged('post_install', '-at_install')
-class TestSalePayment(TestSaleToInvoice):
-
-    def _sale_context(self):
-        return {
-            'active_model': 'sale.order',
-            'active_ids': [self.sale_order.id],
-            'active_id': self.sale_order.id,
-        }
-
-    def test_00_payment(self):
-        self.sale_order.action_confirm()
-
-        payment_wizard = self.env['account.payment.register'].with_context(self._sale_context()).create({'amount': -15})
-        self.assertTrue(payment_wizard.journal_id)
-        with self.assertRaises(UserError):
-            payment_wizard._create_payments()
-
-        payment_wizard = self.env['account.payment.register'].with_context(self._sale_context()).create({'amount': 0})
-        self.assertTrue(payment_wizard.journal_id)
-        with self.assertRaises(UserError):
-            payment_wizard._create_payments()
-
-        payment_wizard = self.env['account.payment.register'].with_context(self._sale_context()).create({})
-        self.assertTrue(payment_wizard.journal_id)
-
-        payment = payment_wizard._create_payments()
-        self.assertEqual(payment.amount, self.sale_order.amount_total)
-        self.assertEqual(payment.sale_order_id, self.sale_order)
-        self.assertEqual(self.sale_order.manual_amount_remaining, 0.0)
-
-        payment_wizard = self.env['account.payment.register'].with_context(self._sale_context()).create({'amount': 15})
-        self.assertTrue(payment_wizard.journal_id)
-        with self.assertRaises(UserError):
-            payment_wizard._create_payments()
-            
-    def test_10_partial_payment(self):
-        payment_wizard = self.env['account.payment.register'].with_context(self._sale_context()).create({'amount': 50})
-        self.assertTrue(payment_wizard.journal_id)
-
-        payment = payment_wizard._create_payments()
-        self.assertEqual(payment.amount, 50)
-        self.assertEqual(payment.sale_order_id, self.sale_order)
-        self.assertEqual(self.sale_order.manual_amount_remaining, self.sale_order.amount_total - 50)
diff --git a/sale_payment_web/views/payment_views.xml b/sale_payment_web/views/payment_views.xml
deleted file mode 100644
index ea0b879c..00000000
--- a/sale_payment_web/views/payment_views.xml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<odoo>
-
-    <record id="view_account_payment_form_inherit" model="ir.ui.view">
-        <field name="name">account.payment.form.inherit</field>
-        <field name="model">account.payment</field>
-        <field name="inherit_id" ref="account.view_account_payment_form"/>
-        <field name="arch" type="xml">
-            <xpath expr="//field[@name='destination_journal_id']" position="after">
-                <field name="sale_order_id" attrs="{'invisible': [('state', '!=', 'draft'), ('sale_order_id', '=', False)], 'readonly': [('state', '!=', 'draft')]}" groups="account.group_account_readonly"/>
-            </xpath>
-        </field>
-    </record>
-
-</odoo>
diff --git a/sale_payment_web/views/sale_views.xml b/sale_payment_web/views/sale_views.xml
deleted file mode 100644
index da5012bd..00000000
--- a/sale_payment_web/views/sale_views.xml
+++ /dev/null
@@ -1,27 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<odoo>
-
-    <record id="view_order_form_inherit" model="ir.ui.view">
-        <field name="name">sale.order.form.inherit</field>
-        <field name="model">sale.order</field>
-        <field name="inherit_id" ref="sale.view_order_form"/>
-        <field name="groups_id" eval="[(4, ref('group_payment_web'))]"/>
-        <field name="arch" type="xml">
-            <xpath expr="//button[@name='action_draft']" position="after">
-                <button name="action_payment_register" 
-                        type="object" 
-                        context="{'dont_redirect_to_payments': True}"
-                        string="Register Payment"/>
-            </xpath>
-            <xpath expr="//field[@name='tax_totals_json']" position="after">
-                <field name="manual_amount_remaining" widget="monetary"/>
-            </xpath>
-            <xpath expr="//div[@name='button_box']" position="inside">
-                <field name="manual_payment_ids" invisible="1"/>
-                <button name="action_manual_payments" string="Manual Payments" type="object" class="oe_stat_button"
-                        icon="fa-dollar" attrs="{'invisible':[('manual_payment_ids', '=', [])]}"/>
-            </xpath>
-        </field>
-    </record>
-
-</odoo>
\ No newline at end of file
diff --git a/sale_payment_web/wizard/__init__.py b/sale_payment_web/wizard/__init__.py
deleted file mode 100644
index 019698a8..00000000
--- a/sale_payment_web/wizard/__init__.py
+++ /dev/null
@@ -1 +0,0 @@
-from . import account_payment_register
diff --git a/sale_payment_web/wizard/account_payment_register.py b/sale_payment_web/wizard/account_payment_register.py
deleted file mode 100644
index 31730f42..00000000
--- a/sale_payment_web/wizard/account_payment_register.py
+++ /dev/null
@@ -1,129 +0,0 @@
-from odoo import api, fields, models, _
-from odoo.exceptions import UserError
-
-
-class AccountPaymentRegister(models.TransientModel):
-    _inherit = 'account.payment.register'
-
-    sale_order_id = fields.Many2one('sale.order', string='Sale Order')
-    so_amount_remaining = fields.Monetary(related='sale_order_id.manual_amount_remaining')
-    so_amount_over = fields.Boolean()
-
-    @api.onchange('amount')
-    def _compute_amount_remaining(self):
-        self.so_amount_over = self.sale_order_id and self.amount > self.so_amount_remaining
-
-    @api.model
-    def default_get(self, fields):
-        # super implementation checks active_ids, but not active_model
-        active_ids = self._context.get('active_ids')
-        if self._context.get('active_model') == 'sale.order' and 'line_ids' in fields:
-            fields.remove('line_ids')
-        rec = super(AccountPaymentRegister, self).default_get(fields)
-        if self._context.get('active_model') != 'sale.order':
-            return rec
-        sale_order = self.env['sale.order'].browse(active_ids)
-        if len(sale_order) != 1:
-            raise UserError('Register payment must be called on a sale order.')
-
-        if 'sale_order_id' not in rec:
-            rec['sale_order_id'] = sale_order.id
-        return rec
-    
-    def _compute_from_lines(self):
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_from_lines()
-        for wizard in wizards:
-            sale_order = wizard.sale_order_id
-            account_receivable = sale_order.partner_id.property_account_receivable_id
-            if not account_receivable:
-                raise UserError('Your partner must have an Account Receivable setup.')
-            company = sale_order.company_id
-            amount_currency = sale_order.manual_amount_remaining
-            if sale_order.currency_id == company.currency_id:
-                amount = amount_currency
-            else:
-                amount = sale_order.currency_id._convert(amount_currency, company.currency_id, company, fields.Date.context_today(self))
-            wizard.update({
-                'company_id': company.id,
-                'partner_id': sale_order.partner_id.id,
-                'partner_type': 'customer',
-                'payment_type': 'inbound',
-                'source_currency_id': sale_order.currency_id.id,
-                'source_amount': amount,
-                'source_amount_currency': amount_currency,
-                'can_edit_wizard': True,
-                'can_group_payments': False,
-            })
-    
-    def _compute_communication(self):
-        # The communication can't be computed in '_compute_from_lines' because
-        # it's a compute editable field and then, should be computed in a separated method.    
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_communication()
-        for wizard in wizards:
-            so = wizard.sale_order_id
-            wizards.communication = so.reference or so.name
-        
-    def _compute_group_payment(self):
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_group_payment()
-        wizards.write({'group_payment': False})
-        
-    def _compute_journal_id(self):
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_journal_id()
-        for wizard in wizards:
-            wizard.journal_id = self.env['account.journal'].search([
-                    ('type', 'in', ('bank', 'cash')),
-                    ('company_id', '=', wizard.company_id.id),
-                ], limit=1)
-        
-    def _compute_available_partner_bank_ids(self):
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_available_partner_bank_ids()
-        for wizard in wizards:
-            wizard.available_partner_bank_ids = wizard.journal_id.bank_account_id
-    
-    def _compute_partner_bank_id(self):
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_partner_bank_id()
-        wizards.write({'partner_bank_id': None})
-    
-    def _compute_suitable_payment_token_partner_ids(self):
-        """ Override from `payment_fix_register_token` """
-        wizards = self.filtered('sale_order_id')
-        super(AccountPaymentRegister, self - wizards)._compute_suitable_payment_token_partner_ids()
-        for wizard in wizards:
-            partners = wizard.sale_order_id.partner_id
-            commercial_partners = partners.commercial_partner_id
-            children_partners = commercial_partners.child_ids
-            wizard.suitable_payment_token_partner_ids = (partners + commercial_partners + children_partners)._origin
-            
-    def _create_payment_vals_from_sale_order(self):
-        if self.amount <= 0:
-            raise UserError("You must enter a positive amount.")
-        elif self.amount > self.so_amount_remaining:
-            raise UserError("You cannot make a payment for more than the difference of the total amount and existing "
-                            "payments: %.2f" % self.so_amount_remaining)
-        return {
-            'date': self.payment_date,
-            'amount': self.amount,
-            'payment_type': self.payment_type,
-            'partner_type': self.partner_type,
-            'ref': self.communication,
-            'journal_id': self.journal_id.id,
-            'currency_id': self.currency_id.id,
-            'partner_id': self.partner_id.id,
-            'partner_bank_id': self.partner_bank_id.id,
-            'payment_method_line_id': self.payment_method_line_id.id,
-            'sale_order_id': self.sale_order_id.id,
-        }
-            
-    def _create_payments(self):
-        self.ensure_one()
-        if not self.sale_order_id:
-            return super(AccountPaymentRegister, self)._create_payments()
-        payments = self.env['account.payment'].create(self._create_payment_vals_from_sale_order())
-        payments.action_post()
-        return payments
diff --git a/sale_payment_web/wizard/account_payment_register_views.xml b/sale_payment_web/wizard/account_payment_register_views.xml
deleted file mode 100644
index b5e3671d..00000000
--- a/sale_payment_web/wizard/account_payment_register_views.xml
+++ /dev/null
@@ -1,28 +0,0 @@
-<?xml version="1.0" encoding="UTF-8" ?>
-<odoo>
-
-    <record id="view_account_payment_register_form_inherit" model="ir.ui.view">
-        <field name="name">account.payment.form.multi.inherit</field>
-        <field name="model">account.payment.register</field>
-        <field name="inherit_id" ref="account.view_account_payment_register_form"/>
-        <field name="arch" type="xml">
-            <xpath expr="//field[@name='company_currency_id']" position="after">
-                <field name="sale_order_id" invisible="1"/>
-                <field name="so_amount_over" invisible="1"/>
-                <field name="so_amount_remaining" invisible="1"/>
-            </xpath>
-            <xpath expr="//field[@name='payment_difference_handling']" position="attributes">
-                <attribute name="attrs">{'invisible': [('sale_order_id', '!=', False)]}</attribute>
-            </xpath>
-            <xpath expr="//group[@name='group3']" position="after">
-                <group name="group_so_amount_over" attrs="{'invisible':[('so_amount_over', '=', False)]}" class="text-danger" colspan="4">
-                    <p>
-                        You cannot make a payment for more than the difference of the total amount and existing payments:
-                        <field name="so_amount_remaining"/>
-                    </p>
-                </group>
-            </xpath>
-        </field>
-    </record>
-
-</odoo>