Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][MIG] l10n_es_aeat_sii_oca: Migration to 16.0 #2813

Merged
merged 216 commits into from
Feb 16, 2023

Conversation

zamberjo
Copy link
Member

@zamberjo zamberjo commented Feb 1, 2023

No description provided.

acysos and others added 30 commits February 1, 2023 09:03
l10n_es_aeat_sii: añadidas facturas de bienes de inversión e intracomunitarias. Se traspasa la configuración del WSDL a parametros
* Ocultar la contraseña del certificado
* Plantilla de cuentas solo obligatorio si "activar SII" activado
* Habilitar un certificado por compañía
* Permisos de acceso
* Tener en cuenta las fechas de los certificados

�[IMP] l10n_es_aeat_sii: No utilizar los parámetros del sistema y buscar solo certificados activos
* Añadir filtros a las facturas para saber si el SII se ha enviado o no
* Corregir la agrupacion de los impuestos si en varias línias de factura hay el mismo impuestos
…& añadir en la difinicion del producto la configuración de la causa exenta y para poderla enviar a la agencia
…ión de ImporteTotal

De momento se aplica solo a emitidas
…SII, pensada para ser heredada en otros módulos como l10n_es_dua_sii
…ínea de factura. De esta forma permitimos que otros módulos puedan heredar e intervenir en el cálculo del precio subtotal por línea, como por ejemplo para incluir el importe de un tasa, segundos descuentos, etc.
* filtro facturas sii fallidas + clave registro en posición fiscal
* catch job error on invoice
…laves por defecto, una para ventas y otra para compras. También se corrigen los problemas de redondeo si se aplica redondeo global en la contabilización de facturas. [FIX] PEP8. [UPD] Traducciones.
* Onchanges renamed for avoiding conflicts with other modules
* Methods renamed for avoiding conflicts with other modules
* Some method docstrings.
* Use recordset instead of lists.
* Remove incorrect mix of @api.multi and @api.model decorators
* Remove method arguments that are passed when there are other data that are taken from self.
* Place in the proper model some of the methods for using self reference
* Variables renamed according their use.
* POT file removed.
* Reduce cyclomatic complexity of some methods.
* Code style more homogeneous.
* Uniformize file headers.
* Full Spanish translation.
* Add icon.
* TipoImpositivo
* Default del refund_type
* Clave de registro facturas proveedor
… + otros

* Ajuste DesgloseFactura/DesgloseTipoOperacion en emitidas

  Evita/adapta que se incluyan DesgloseFactura/DesgloseTipoOperacion (excluyentes).
  Si es obligatorio usa DesgloseTipoOperacion para ciertas operaciones.
  Se hace al final de _get_sii_out_taxes para tocar lo menos posible lo que ya hay.

* Mapeado S_IVA0_E en SFESBE
* Se añade a recibidas clave ImporteTotal.
- Cambio del campo "Carpeta" al asistente de obtención de claves.
- Incluir el tipo de rectificación en el asistente de factura rectificativa.
- Renombrados dos campos que faltaban sin prefijo `sii_` para evitar colisiones con otros módulos.
- Solventadas algunas inconsistencias del UI de la factura.
- Campo readonly para related para evitar efectos laterales
- PEP8
… de los intracomunitarios. Para no presentar el 100% del padre.
- Se usa float-round en el redondeo
- También se redondean los valores en taxes_to
- Se incluye el redondeo de la BaseImponible
- Se aplica también a recibidas
jabelchi and others added 17 commits February 16, 2023 10:25
Currently translated at 99.4% (191 of 192 strings)

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_sii_oca
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_sii_oca/ca/
…ance in other addons.

Hide thirdparty fields in some cases:
- Hide thirdparty fields in moves when it is not an invoice.
- Hide thirdparty field in journals when it is not sale/purchase type.
- Avoid modifying third-party number when an invoice has already been sent to the SII.
- Avoid modifying customer invoice name when an invoice has already been sent to the SII.
TT32612
Following odoo/odoo#95967, the mapping is done
in the SII corresponding files.

TT38004
- pyOpenSSL was replaced by cryptography and is no longer used.
- cryptography has a specific version dependency specified in `l10n_es_aeat`, which in turn is dependency of all these modules. So there's no real need to indicate that dependency in these files.

@moduon MT-1067
…egative

Including l10n_es.account_tax_template_p_iva*_isp taxes in
NotIncludedInTotalNegative map section actually adds tax amount
to ImporteTotal. Removing them ImporteTotal becomes BaseImponible,
that is the expected behavior according AEAT.

Fixes OCA#2669
* Before this fix, when an invoice fiscal position is marked as Export, SII
  exempt cause is currently forced to E2.
* E3 exempt cause should also be considered in this situation. This commit fix
  it. E2 is still selected as default in other cases, anyway.
Refinement of OCA#2836

- Missing tax "0% IVA soportado (servicios corrientes)"
- Bad tax group for taxes "Importaciones servicios corrientes"
- Missing "Recargo equivalencia 0%" taxes
- Missing new taxes in NotIncludeInTotalNegative
@zamberjo zamberjo force-pushed the 16.0-mig-l10n_es_aeat_sii_oca branch from 5180be9 to 4bedecb Compare February 16, 2023 09:28
@zamberjo
Copy link
Member Author

Te puse algunos ejemplos, pero hay más commits administrativos antes. ¿Puedes rematarlo? Gracias

Había hecho unos cuantos más, pero no todos, ahora he hecho hasta que he dejado de ver commits de bots.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gracias!

Vamos con ello, y si se detecta algo más en pruebas, se hacen PRs consecutivos:

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-2813-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b9c6516 into OCA:16.0 Feb 16, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 553cf7c. Thanks a lot for contributing to OCA. ❤️

@zamberjo zamberjo deleted the 16.0-mig-l10n_es_aeat_sii_oca branch February 18, 2023 07:35
@ljsalvatierra-factorlibre
Copy link
Contributor

Gracias!

Vamos con ello, y si se detecta algo más en pruebas, se hacen PRs consecutivos:

/ocabot merge nobump

Hola @pedrobaeza , tengo una pregunta sobre el asistente de reversión account.move.reversal y el campo sii_refund_type, ¿aceptaríais introducir el valor "I" por defecto? Si os parece bien abro un PR nuevo.

@pedrobaeza
Copy link
Member

Vale, creo que es razonable, puesto que al final solo hay soporte para un tipo.

@ljsalvatierra-factorlibre
Copy link
Contributor

Vale, creo que es razonable, puesto que al final solo hay soporte para un tipo.

Genial gracias, me pongo con ello.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.