diff --git a/sale_payment_deposit/__init__.py b/sale_payment_deposit/__init__.py new file mode 100755 index 00000000..0650744f --- /dev/null +++ b/sale_payment_deposit/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/sale_payment_deposit/__manifest__.py b/sale_payment_deposit/__manifest__.py new file mode 100755 index 00000000..4689959c --- /dev/null +++ b/sale_payment_deposit/__manifest__.py @@ -0,0 +1,24 @@ +{ + 'name': 'Sale Payment Deposit', + 'author': 'Hibou Corp. ', + 'category': 'Sales', + 'version': '12.0.1.0.0', + 'description': + """ +Sale Deposits +============= + +Automates the creation of 'Deposit' invoices and payments. For example, someone confirming +a sale with "50% Deposit, 50% on Delivery" payment terms, will pay the +50% deposit payment up front instead of the entire order. + """, + 'depends': [ + 'sale', + 'payment', + ], + 'auto_install': False, + 'data': [ + 'views/account_views.xml', + 'views/sale_portal_templates.xml', + ], +} diff --git a/sale_payment_deposit/models/__init__.py b/sale_payment_deposit/models/__init__.py new file mode 100644 index 00000000..13088db7 --- /dev/null +++ b/sale_payment_deposit/models/__init__.py @@ -0,0 +1,3 @@ +from . import account +from . import sale +from . import sale_patch diff --git a/sale_payment_deposit/models/account.py b/sale_payment_deposit/models/account.py new file mode 100644 index 00000000..111e748f --- /dev/null +++ b/sale_payment_deposit/models/account.py @@ -0,0 +1,31 @@ +from odoo import api, fields, models + + +class AccountPaymentTerm(models.Model): + _inherit = 'account.payment.term' + + deposit_percentage = fields.Float(string='Deposit Percentage', + help='Require deposit when paying on the front end.') + + +class PaymentTransaction(models.Model): + _inherit = 'payment.transaction' + + @api.multi + def _post_process_after_done(self): + now = fields.Datetime.now() + res = super(PaymentTransaction, self)._post_process_after_done() + + # Post Process Payments made on the front of the website, reconciling to Deposit. + for transaction in self.filtered(lambda t: t.payment_id + and t.sale_order_ids + and sum(t.sale_order_ids.mapped('amount_total_deposit')) + and not t.invoice_ids and (now - t.date).seconds < 1200): + if not transaction.sale_order_ids.mapped('invoice_ids'): + # don't process ones that we might still be able to process later... + transaction.write({'is_processed': False}) + else: + # we have a payment and could attempt to reconcile to an invoice + # Leave the payment in 'is_processed': True + transaction.sale_order_ids._auto_deposit_payment_match() + return res diff --git a/sale_payment_deposit/models/sale.py b/sale_payment_deposit/models/sale.py new file mode 100644 index 00000000..293a03bb --- /dev/null +++ b/sale_payment_deposit/models/sale.py @@ -0,0 +1,47 @@ +from odoo import api, fields, models +from json import loads as json_loads + + +class SaleOrder(models.Model): + _inherit = 'sale.order' + + amount_total_deposit = fields.Monetary(string='Deposit', compute='_amount_total_deposit') + + @api.depends('amount_total', 'payment_term_id.deposit_percentage') + def _amount_total_deposit(self): + for order in self: + order.amount_total_deposit = order.amount_total * float(order.payment_term_id.deposit_percentage) / 100.0 + + def action_confirm(self): + res = super(SaleOrder, self).action_confirm() + self._auto_deposit_invoice() + return res + + def _auto_deposit_invoice(self): + wizard_model = self.env['sale.advance.payment.inv'].sudo() + for sale in self.sudo().filtered(lambda o: o.amount_total_deposit): + # Create Deposit Invoices + wizard = wizard_model.create({ + 'advance_payment_method': 'percentage', + 'amount': sale.payment_term_id.deposit_percentage, + }) + wizard.with_context(active_ids=sale.ids).create_invoices() + # Validate Invoices + sale.invoice_ids.filtered(lambda i: i.state == 'draft').action_invoice_open() + # Attempt to reconcile + sale._auto_deposit_payment_match() + + def _auto_deposit_payment_match(self): + # Attempt to find payments that could be used on this new invoice and reconcile them. + # Note that this probably doesn't work for a payment made on the front, see .account.PaymentTransaction + aml_model = self.env['account.move.line'].sudo() + for sale in self.sudo(): + for invoice in sale.invoice_ids.filtered(lambda i: i.state == 'open'): + outstanding = json_loads(invoice.outstanding_credits_debits_widget) + if isinstance(outstanding, dict) and outstanding.get('content'): + for line in outstanding.get('content'): + if abs(line.get('amount', 0.0) - invoice.residual) < 0.01 and line.get('id'): + aml = aml_model.browse(line.get('id')) + aml += invoice.move_id.line_ids.filtered(lambda l: l.account_id == aml.account_id) + if aml.reconcile(): + break diff --git a/sale_payment_deposit/models/sale_patch.py b/sale_payment_deposit/models/sale_patch.py new file mode 100644 index 00000000..c306a21b --- /dev/null +++ b/sale_payment_deposit/models/sale_patch.py @@ -0,0 +1,86 @@ +from odoo import api, fields, models +from odoo.exceptions import ValidationError +from odoo.addons.sale.models.sale import SaleOrder + + +@api.multi +def _create_payment_transaction(self, vals): + # This is a copy job from odoo.addons.sale.models.sale due to the closed nature of the vals.update(dict) call + # Ultimately, only the 'vals.update' with the new amount is really used. + '''Similar to self.env['payment.transaction'].create(vals) but the values are filled with the + current sales orders fields (e.g. the partner or the currency). + :param vals: The values to create a new payment.transaction. + :return: The newly created payment.transaction record. + ''' + # Ensure the currencies are the same. + + # extract variable for use later. + sale = self[0] + + currency = sale.pricelist_id.currency_id + if any([so.pricelist_id.currency_id != currency for so in self]): + raise ValidationError(_('A transaction can\'t be linked to sales orders having different currencies.')) + + # Ensure the partner are the same. + partner = sale.partner_id + if any([so.partner_id != partner for so in self]): + raise ValidationError(_('A transaction can\'t be linked to sales orders having different partners.')) + + # Try to retrieve the acquirer. However, fallback to the token's acquirer. + acquirer_id = vals.get('acquirer_id') + acquirer = False + payment_token_id = vals.get('payment_token_id') + + if payment_token_id: + payment_token = self.env['payment.token'].sudo().browse(payment_token_id) + + # Check payment_token/acquirer matching or take the acquirer from token + if acquirer_id: + acquirer = self.env['payment.acquirer'].browse(acquirer_id) + if payment_token and payment_token.acquirer_id != acquirer: + raise ValidationError(_('Invalid token found! Token acquirer %s != %s') % ( + payment_token.acquirer_id.name, acquirer.name)) + if payment_token and payment_token.partner_id != partner: + raise ValidationError(_('Invalid token found! Token partner %s != %s') % ( + payment_token.partner.name, partner.name)) + else: + acquirer = payment_token.acquirer_id + + # Check an acquirer is there. + if not acquirer_id and not acquirer: + raise ValidationError(_('A payment acquirer is required to create a transaction.')) + + if not acquirer: + acquirer = self.env['payment.acquirer'].browse(acquirer_id) + + # Check a journal is set on acquirer. + if not acquirer.journal_id: + raise ValidationError(_('A journal must be specified of the acquirer %s.' % acquirer.name)) + + if not acquirer_id and acquirer: + vals['acquirer_id'] = acquirer.id + + # Override for Deposit + amount = sum(self.mapped('amount_total')) + # This is a patch, all databases will run this code even if this field doesn't exist. + if hasattr(sale, 'amount_total_deposit') and sum(self.mapped('amount_total_deposit')): + amount = sum(self.mapped('amount_total_deposit')) + + vals.update({ + 'amount': amount, + 'currency_id': currency.id, + 'partner_id': partner.id, + 'sale_order_ids': [(6, 0, self.ids)], + }) + + transaction = self.env['payment.transaction'].create(vals) + + # Process directly if payment_token + if transaction.payment_token_id: + transaction.s2s_do_transaction() + + return transaction + + +# Patch core implementation. +SaleOrder._create_payment_transaction = _create_payment_transaction diff --git a/sale_payment_deposit/views/account_views.xml b/sale_payment_deposit/views/account_views.xml new file mode 100644 index 00000000..aa174032 --- /dev/null +++ b/sale_payment_deposit/views/account_views.xml @@ -0,0 +1,13 @@ + + + + account.payment.term.form.inherit + account.payment.term + + + + + + + + \ No newline at end of file diff --git a/sale_payment_deposit/views/sale_portal_templates.xml b/sale_payment_deposit/views/sale_portal_templates.xml new file mode 100644 index 00000000..cd52552d --- /dev/null +++ b/sale_payment_deposit/views/sale_portal_templates.xml @@ -0,0 +1,22 @@ + + + + \ No newline at end of file