[19.0][MIG] l10n_es_account_asset: Migration to version 19.0 - #5124
Open
carlos-lopez-tecnativa wants to merge 53 commits into
Open
[19.0][MIG] l10n_es_account_asset: Migration to version 19.0#5124carlos-lopez-tecnativa wants to merge 53 commits into
carlos-lopez-tecnativa wants to merge 53 commits into
Conversation
…s prevision tesoreria y activos/amortizaciones.
…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
…hecho para aprovechar el estándar al máximo. * Incluido nuevo método de depreciación por porcentaje fijo.
* PEP8 * Icono del módulo * Valor por defecto para ext_method_time
* Nos traemos unas corecciones aplicadas en la versión 7.0 que no estaban en la 8.0, necesarias para hacer correctamente amortizaciones mensuales sin prorrata.
…rsiones de fecha. Fixes OCA#208
…sual ya incuidas en 7.0 + arreglado problema en la vista para la fecha de inicio de cálculo
[IMP] l10n_es_account_asset: tests added.
…e + remove # depreciations compensation. Fixes OCA#146 + Delete empty last depreciation line
…category_id (OCA#671) Pasos para reproducir el error: * Crear un tipo de activo * Al crear un nuevo activo, los datos de la periodicidad configurados en el tipo de activo no se importan al nuevo activo. Corrección: El método a sobreescribir debe ser el "onchange_category_id_values" y no el "onchange_category_id" tal y como es actualmente.
…CA#804) Corregido _compute_board_undone_dotation_nb para casos de prorrateo
…ins object instead of ids
…sset_management As `account_asset` is now enterprise, we switch the dependency to `account_asset_management` OCA module. This requires a heavy refactoring on code and tests, but with some hooks and fixes provided to OCA module, it's still very light. Other changes: * README by fragments * Promote to Production/Stable * Set maintainer
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_account_asset/es/
In some cases, the amount in annual_percentage is not synchronized with the one in the period one due to a non hunted bug in v10/v11. As in v12 now the important field is the annual one, we must synchronize it properly.
* fix: falta llamada super
* Not possible to remove Spanish assets It should be possible and there's no real reason for that limitation. * Avoid incorrect values in annual_percentage When generating an asset from the invoice with other computation method, it can be that the method_percentage is put as 100, inversing the value for annual percentage as something incorrect. We avoid it preventing a higher value than 100. * Avoid infinite recursion When creating an asset from invoice with percentage method, there's a infinite recursion due to the trick of changing the method type on the fly, which activates another board computation on the write method. That can be avoided passing a special context key that avoids the recomputation. * Compute correctly number of depreciations The depreciation table is computed and stripped in _compute_depreciation_amount_per_fiscal_year, but previously the computation in that method was not intercepted, and so, computations was done with year method data.
…w computed writable Since OCA/account-financial-tools#952, the onchange method for profile no longer exists, so there's an error when changing such profile when this module is installed. This commit fixes this and switch to new computed writable style, adding also tests for the profile assignation for better regressions detection. Closes OCA#1455
…tage As it alters computations for other methods. TT25453
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_account_asset/es/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_account_asset/ca/
SavepointCase > TransactionCase
- Standard procedure - Adapt tests - Some compute methods were not totally multi-record TT41118
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_account_asset/
When a tax is non deductible (because it's directly non deductible VAT, or due to the VAT prorate), the amount to depreciate should be increased with this non deductible amount, which was not taken into account before this commit. For achieving this without requiring glue modules, we are now depending on `l10n_es_aeat`, which is a reasonable trade-off, as almost all the instances with this module will have the other installed as well. As we only know the deductible amount at invoice level, we get the general tax info, and then apply the percentage corresponding to each line. It includes a new test for covering this case.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_account_asset/
Currently translated at 100.0% (7 of 7 strings) Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_account_asset/es/
Currently translated at 100.0% (7 of 7 strings) Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_account_asset/ca/
OCA-git-bot
added
series:19.0
mod:l10n_es_account_asset
Contributor
|
/ocabot migration l10n_es_account_asset |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TT58648
Ping @Tecnativa @pedrobaeza @manuelcalerosolis
Reemplaza #5055