pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] account_peppol: disable peppol for certain edi formats

Browse Source 
If the format set on a partner does not work for our peppol implementation, `enable_peppol` should be `False` and the checkbox should be hidden.
This commit checks invalid partners when computing `enable_peppol` on account_move_send wizard.
Since we hide the checkbox in this case, we don't need to show the peppol warning when opening send&print to such partners.

closes odoo/odoo#143565

Signed-off-by: Laurent Smet (las) <las@odoo.com>
Signed-off-by: Aliya Tastemirova (alta) <alta@odoo.com>
This commit is contained in:
aliya
2023-11-25 05:41:40 +00:00
parent 9a3450d396
commit 9224c42487
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/account_peppol/views/res_partner_views.xml
+2 -2
View File
@@ -19,9 +19,9 @@
<field name="account_peppol_is_endpoint_valid" invisible="1"/>
<field name="account_peppol_validity_last_check" invisible="1"/>
<label for="account_peppol_verification_label"
invisible="ubl_cii_format in (False, 'facturx') or not peppol_endpoint"/>
invisible="ubl_cii_format in (False, 'facturx', 'oioubl_201') or not peppol_endpoint"/>
<div class="row"
invisible="ubl_cii_format in (False, 'facturx') or not peppol_endpoint">
invisible="ubl_cii_format in (False, 'facturx', 'oioubl_201') or not peppol_endpoint">
<div class="col-4">
<field name="account_peppol_verification_label"/>
</div>
addons/account_peppol/wizard/account_move_send.py
+5 -2
View File
@@ -55,7 +55,7 @@ class AccountMoveSend(models.TransientModel):
def _compute_peppol_warning(self):
for wizard in self:
invalid_partners = wizard.move_ids.partner_id.filtered(
lambda partner: not partner.account_peppol_is_endpoint_valid or partner.ubl_cii_format in {False, 'facturx'})
lambda partner: not partner.account_peppol_is_endpoint_valid)
if not invalid_partners:
wizard.peppol_warning = False
else:
@@ -68,13 +68,16 @@ class AccountMoveSend(models.TransientModel):
def _compute_enable_peppol(self):
for wizard in self:
# show peppol option if either the ubl option is available or any move already has a ubl file generated
# and moves are not processing/done
# and moves are not processing/done and if partners have an edi format set to one that works for peppol
invalid_partners = wizard.move_ids.partner_id.filtered(
lambda partner: partner.ubl_cii_format in {False, 'facturx', 'oioubl_201'})
wizard.enable_peppol = (
wizard.company_id.account_peppol_proxy_state == 'active' \
and (
wizard.enable_ubl_cii_xml
or any(m.ubl_cii_xml_id and m.peppol_move_state not in ('processing', 'done') for m in wizard.move_ids)
)
and not invalid_partners
)
@api.depends('company_id.account_edi_proxy_client_ids.edi_mode')