OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
21 stars 51 forks source link

[FIX] point_of_sale,retentions_basic,perceptions_basic #10

Closed ssaid closed 9 years ago

ssaid commented 9 years ago

En los xml de data, agrego el forcecreate="0" para todos los registros. Es necesario para que en la migracion cuando tiro -u all no me tire constraint error.

sebastiken commented 9 years ago

Ok, @ssaid . Sería buena idea tal vez, que estos datos de configuración estén en otro módulo, ya que, si uno no utiliza el Plan de Cuentas de este repo, este módulo falla al querer insertar los datos de Percepciones. Esto es porque busca cuentas particulares de ese Plan de Cuentas. Ese xml con datos preconfigurados se hizo para tener ya varias percepciones cargadas de ejemplo que funcionan y se pueden utilizar sin tener que hacer todo un trabajo manual de configuración. El cambio que proponés acá, corrige el problema que ocurre al borrar una de estas configuraciones de percepción y luego actualizar el módulo.

ssaid commented 9 years ago

@sebastiken me parece correcta tu propuesta, te parece organizarlo asi?

  1. l10n_ar_perceptions_basic_data: requires l10n_ar_perceptions_basic, l10n_ar_chart_of_account
  2. l10n_ar_retentions_basic_data: requires l10n_ar_retentions_basic, l10n_ar_chart_of_account En l10n_ar_point_of_sale dejarlo como esta.
sebastiken commented 9 years ago

@ssaid me parece bien lo que proponés, ya que de esta manera, si tenés instalado el módulo l10n_ar_chart_of_account, ese módulo de datos va a funcionar correctamente. Perfecto, si lo hacés y completás este PR lo mergeamos. Gracias.