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_mod347: Migration to 16.0 #2679

Merged
merged 83 commits into from
Dec 17, 2022

Conversation

ramiadavid
Copy link
Contributor

No description provided.

Borja L.S and others added 30 commits December 15, 2022 14:21
- Se han renombrado los módulos para usar la nomenclatura propuesta
  por OpenERP: l10n_es para el módulo base de localización (plan de
  cuentas), l10n_es_* para el resto de módulos.

- Se eliminan los módulos extra_addons/* que deberían moverse a
  los extra-addons genéricos (no son específicos de España).

- Se renombran los __terp__.py por __openerp__.py
…ciertos módulos, por los correspondientes que se modificaron para esta versión 6.0 y en ciertos __init__ adaptamos los imports a los nuevos nombres de los ficheros. Renombrado de los archivos de traducción españoles de es_ES.po a es.po y pequeñas refactorizaciones
* FIX: Renombrado de los archivos de traducción catalanas de ca_ES.po a ca.po
* IMP: Añadidos avisos NO ADAPTADO TODAVÍA A LA VERSIÓN 6.0 a varios módulos, limpieza l10n_es_partner_mercantil
…c module for aeat models, 347 module was portedto v6.0 and adds new module to print AEAT model 349.
…ueñas mejoras generales (vistas, traducciones, código) y corrección pequeño bug por un olvido en la adaptación de la v5 a la v6
… módulo del 347, que incluían los cambios hechos por la AEAT para la declaración 2010, para la versión 6.0 del módulo, además corregimos un bug encontrado por Jordi en la versión 5.0 hoy. También añado una nueva comprobación de los registros de empresas, ya que el cif de la empresa es requerido, por lo que no va a dejar confirmar el informe, mientras no se rellene este campo en todos los registros de empresa. La agrupación de pagos en efectivo se dejó igual que en la 5.0, ya que se sigue comportando como en la 5.0, sigo animando a que si alguien conoce como mejorarla adelante.
… las mejoras introducidas en el anterior commit
…recía Pexego en la licencia, quedan los compartidos.
…elo con el detalle de los trimestes obligatorio para la declaración del 2011.
…r Cif/Nif en el cálculo del 347, por si hay más de una empresa con el mismo cif..., pequeña refactorización usando rec_name en lugar de name_get
… los nuevos strings después de los recientes cambios en el 347.
…as. Necesario para las Sociedades Irregulares, Sociedades Civiles y Comunidad de Bienes sin regimen de módulos en el IRPF. Al resto de las empresas no les afecta ya que no facturan con IRPF. l10n_es_aeat_mod340: Correción de los autores en el modelo 340
…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True
[FIX]: Add security definition for objects l10n_es_aeat_mod347_calculate_records and l10n_es_aeat_mod347_export_to_boe.
[FIX]: Changed types to orm.Model, orm.TransientModel and orm.AbstractModel.
[FIX]: Contributions have been written in the standard format of the community.
[FIX]: Remove 'init_xml' keys, because it's no longer needed in v7.
[FIX]: Remove __author__ variables in files, because authors are put on manifest file (__openerp__.py).
[FIX]: Increased compatibility with standard PEP8.
[FIX]: Imports in standard format.
[IMP]: Add "state" field in tree view.
[IMP]: Option to generate report suppliers only.
[FIX]: Update address partner search to V7.
[FIX]: Update state in address partner to V7.
[FIX]: Update dependencies, now required the module documents to view generated file in V7.
[FIX]: Translate .pot file to v7.
[FIX]: Translate field "only_supplier" to spanish language.
[FIX]: Format the text that explains the module.
[IMP]: Add icon and screencapture.
- Cambiado flag de los partners para no incluirlos en el 347 (opción menos común), que incluirlos (que suelen ser todos).
- Eliminado flag para incluir los diarios, ya que no se activa por defecto y el usuario se confunde con los resultados. Se cogen todos los diarios de caja.
- Otras mejoras menores.
…ds, variables, classes and translations, and related PEP8 fixes.
manuelregidor and others added 18 commits December 15, 2022 14:34
Previously, if there are cancel or draft moves with the corresponding
taxes, they were included in the report.

We should filter them out.

TT33910
The provided field name was not correct.

Fixes OCA#2001
In a customer invoice, the field `name` contains the number of the
invoice, and the field `ref` can contain any other arbitrary content
put by the user, so we should only use this second one in the case of
vendor bills, as this is the invoice number for the vendor.

TT34430
When populating move records for a partner, an abs operation is
performed, as AEAT requires to express amount as positive for all
records.

Previous code trusts on the journal entry type for determining later
the sign of the operation, but there can be the possibility of having
a big journal entry where the type is of one sign, but the specific
operation for that partner is another. The canonical example is on PoS,
with the closing entry including together all the operations of the
session.

So let's keep the balance sign all the time, and remove the checks on
move type, converting the amount to positive where needed (BOE export),
and this also makes a bit lighter the quarter computation, not needing
to filter by move_type.

It includes the migration scripts for adapting following the same
previous approach the existing amounts.

TT34561
Currently translated at 100.0% (249 of 249 strings)

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod347
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod347/ca/
@ramiadavid ramiadavid mentioned this pull request Dec 15, 2022
54 tasks
@ramiadavid ramiadavid marked this pull request as ready for review December 15, 2022 13:40
@pedrobaeza
Copy link
Member

/ocabot merge nobump
/ocabot migration l10n_es_aeat_mod347

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Dec 17, 2022
@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-2679-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3281000 into OCA:16.0 Dec 17, 2022
@OCA-git-bot
Copy link
Contributor

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

@ramiadavid ramiadavid deleted the 16.0-mig-l10n_es_aeat_mod347 branch December 17, 2022 18:15
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.