[FIX] l10n_be,point_of_sale: exclude non-trade receivable partners from POS

This commit removes the partners with non-trade receivable
accounts from the POS customer's list. Additionally, 2 new
VAT accounts were added in BE COA to be added on
"Collection Center" partner.

task-5148985

closes odoo#233360

Related: odoo/enterprise#98224
Signed-off-by: John Laterre (jol) <[email protected]>

Author: shgabr

addons/l10n_be/data/template/account.account-be.csv

@@ -164,6 +164,7 @@
 "a409","Trade Debtors - Write-Downs (-)","409","asset_current","","False","","Créances commerciales - Réductions de valeur (-)","Handelsvorderingen - Waardeverminderingen (-)","Forderungen aus Lieferungen und Leistungen - Wertminderungen (-)","","","","","",""
 "a411","VAT Recoverable","411","asset_current","","False","","TVA à récupérer","Terug te vorderen btw","Zurück zu erhaltende Mehrwertsteuer","","","","","",""
 "a4112","VAT Recoverable: VAT Current Account (C/A)","4112","asset_receivable","","True","True","TVA à récupérer : Compte courant TVA (C/C)","Terug te vorderen btw: Rekening-courant btw (RC)","Zurück zu erhaltende Mehrwertsteuer: Girokonto-MwSt","","","","","",""
+"a4121","Advance Tax Payment","4121","asset_receivable","","True","True","Paiement anticipé de l'impôt","Vooruitbetaling van belasting","Vorauszahlung von Steuern","","","","","",""
 "a4125","Other Belgian Taxes to Be Recovered","4125","asset_current","","False","","Autres impôts et taxes belges à récupérer","Terug te vorderen andere Belgische belastingen","Zurück zu erhaltende sonstige belgische Steuern","","","","","",""
 "a4128","Foreign Taxes to Be Recovered","4128","asset_current","","False","","Impôts et taxes étrangers à récupérer","Terug te vorderen buitenlandse belastingen","Zurück zu erhaltende ausländische Steuern","","","","","",""
 "a414","Income Receivable","414","asset_current","","False","","Produits à recevoir","Te innen opbrengsten","Zu erhaltende Erträge","","","","","",""
@@ -200,6 +201,7 @@
 "a451800","VAT Payable - Insufficient Taxation","451800","liability_current","","False","","TVA à payer - Taxation insuffisante","Te betalen btw - Ontoereikende heffing","Zu zahlende Mehrwertsteuer - Unzureichende Besteuerung","","","","","",""
 "a451820","VAT Payable - Deduction Revisions","451820","liability_current","","False","","TVA à payer - Révisions des déductions","Te betalen btw - Herzieningen aftrek","Zu zahlende Mehrwertsteuer - Abzugsrevisionen","","","","","",""
 "a451830","VAT Payable - Regularizations","451830","liability_current","","False","","TVA à payer - Régularisations","Te betalen btw - Regularisaties","Zu zahlende Mehrwertsteuer - Regularisierungen","","","","","",""
+"a4521","Advance Tax Payment","4521","liability_payable","","True","True","Paiement anticipé de l'impôt","Vooruitbetaling van belasting","Vorauszahlung von Steuern","","","","","",""
 "a4525","Other Belgian Taxes Payable","4525","liability_current","","False","","Autres impôts et taxes belges à payer","Andere te betalen Belgische belastingen en taksen","Sonstige zu zahlende belgische Steuern und Abgaben","","","","","",""
 "a4528","Foreign Taxes Payable","4528","liability_current","","False","","Impôts et taxes étrangers à payer","Te betalen buitenlandse belastingen en taksen","Zu zahlende ausländische Steuern und Abgaben","","","","","",""
 "a453","Taxes Withheld","453","liability_current","","False","","Précomptes retenus","Ingehouden voorheffingen","Einbehaltene Steuervorhabzüge","","","","","",""

addons/point_of_sale/models/__init__.py

@@ -2,6 +2,7 @@
 
 from . import account_bank_statement
 from . import pos_load_mixin
+from . import account_account
 from . import account_cash_rounding
 from . import account_payment
 from . import account_journal

addons/point_of_sale/models/account_account.py

@@ -0,0 +1,17 @@
+from odoo import api, models
+
+
+class AccountAccount(models.Model):
+    _name = 'account.account'
+    _inherit = ['account.account', 'pos.load.mixin']
+
+    @api.model
+    def _load_pos_data_fields(self, config):
+        return [
+            'id', 'non_trade',
+        ]
+
+    @api.model
+    def _load_pos_data_domain(self, data, config):
+        property_account_receivable_ids = {partner['property_account_receivable_id'] for partner in data['res.partner']}
+        return [('id', 'in', property_account_receivable_ids)]

addons/point_of_sale/models/pos_session.py

@@ -141,7 +141,7 @@ def _load_pos_data_models(self, config):
             'pos.category', 'pos.bill', 'res.company', 'account.tax', 'account.tax.group', 'product.template', 'product.product', 'product.attribute', 'product.attribute.custom.value',
             'product.template.attribute.line', 'product.template.attribute.value', 'product.template.attribute.exclusion', 'product.combo', 'product.combo.item', 'res.users', 'res.partner', 'product.uom',
             'decimal.precision', 'uom.uom', 'res.country', 'res.country.state', 'res.lang', 'product.category', 'product.pricelist', 'product.pricelist.item',
-            'account.cash.rounding', 'account.fiscal.position', 'stock.picking.type', 'res.currency', 'pos.note', 'product.tag', 'ir.module.module', 'account.move']
+            'account.cash.rounding', 'account.fiscal.position', 'stock.picking.type', 'res.currency', 'pos.note', 'product.tag', 'ir.module.module', 'account.move', 'account.account']
 
     @api.model
     def _load_pos_data_domain(self, data, config):

addons/point_of_sale/models/res_partner.py

@@ -74,7 +74,7 @@ def _load_pos_data_fields(self, config):
         return [
             'id', 'name', 'street', 'street2', 'city', 'state_id', 'country_id', 'vat', 'lang', 'phone', 'zip', 'email',
             'barcode', 'write_date', 'property_product_pricelist', 'parent_name', 'pos_contact_address',
-            'invoice_emails', 'fiscal_position_id', 'is_company',
+            'invoice_emails', 'fiscal_position_id', 'is_company', 'property_account_receivable_id',
         ]
 
     def _compute_pos_order(self):

addons/point_of_sale/static/src/app/screens/partner_list/partner_list.js

@@ -29,7 +29,10 @@ export class PartnerList extends Component {
         this.modalRef = useChildRef();
         this.modalContent = null;
         this.state = useState({
-            initialPartners: this.pos.models["res.partner"].getAll(),
+            initialPartners: this.pos.models["res.partner"].filter((p) => {
+                const par = p.property_account_receivable_id;
+                return !par || par.non_trade !== true;
+            }),
             loadedPartners: [],
             query: "",
             loading: false,