l10n_es_aeat: Falso positivo en get_certificates

[syci]

l10n-spain/l10n_es_aeat/models/aeat_certificate.py

Line 73 in e3ec0c5

else:

Según el código si no existe el certificado añadido desde el backend, comprueba que exista el ir.config_parameter , pero no realiza la comprobación de que exista realmente el fichero en la ruta definida en data de https://github.com/OCA/l10n-spain/blob/17.0/l10n_es_aeat/data/ir_config_parameter.xml

Por lo tanto, da un falso positivo si no se crean los ficheros (que es lo normal en una instalación nueva). Veo que está desde versiones atrás...

Veo dos alternativas:

• Añadir check de os.path.isfile
• Eliminar el fichero data

Por otro lado, aunque eliminemos los params que afectan el CRON de VERI*FACTU al lanzarse no captura el error correctamente porque pasa por alto la excepción en

l10n-spain/l10n_es_verifactu_oca/models/verifactu_invoice_entry.py

Line 190 in f0eadce

def _connect_verifactu(self):

Se propone:

try:
            public_crt, private_key = self.env["l10n.es.aeat.certificate"].get_certificates(
                company=self.company_id
            )
        except UserError:
            raise UserError(
                _("Please, configure the VERI*FACTU certificates for your company")
            )

Con este cambio se detecta un error que queda registrado en el planificador:

Pero se pierde la traza del error real...