Merge lp://qastaging/~pexego/openerp-spain/6.1_fix_plantillas_pyg into lp://qastaging/~openerp-spain-team/openerp-spain/6.1

Proposed by Omar (Pexego)
Status: Merged
Merged at revision: 400
Proposed branch: lp://qastaging/~pexego/openerp-spain/6.1_fix_plantillas_pyg
Merge into: lp://qastaging/~openerp-spain-team/openerp-spain/6.1
Diff against target: 1096 lines (+528/-528)
2 files modified
l10n_es_account_balance_report/data/pyg_abreviado.xml (+261/-261)
l10n_es_account_balance_report/data/pyg_pymes.xml (+267/-267)
To merge this branch: bzr merge lp://qastaging/~pexego/openerp-spain/6.1_fix_plantillas_pyg
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review Approve
Alejandro Santana code review (no test) Approve
Review via email: mp+220987@code.qastaging.launchpad.net

Description of the change

Se incluye la siguiente corrección en las plantillas:

- Plantilla pyg pymes:

En el epígrafe de Resultado de explotación no se estaba sumando el epígrafe 41000.

Además antes de aprobar el merge deberíamos hablar de si cambiar o no otras cosas que encontré en las plantillas:

- Plantilla pyg pymes:

No se están teniendo en cuenta en el epígrafe de Gastos de personal, las cuentas 7950 y 7957, si se tienen en cuenta en el abrevaiado. Está de acuerdo a la plantilla: http://cuentasanuales.info/index.php/P%C3%A9rdidas-y-ganancias/cuenta-de-perdidas-y-ganancias.html

- Plantilla pyg abreviado

Hay dos epígrafes que no deberían estar ya que sino no se distingue en casi nada la abreviada de la de pymes, los epígrafes en cuestión son, el 41300 y el 41200 Está de acuerdo a: http://cuentasanuales.info/index.php/PyG/cuenta-de-perdidas-y-ganancias-abreviada.html pero consultando otros modelos, no se usan: https://docs.google.com/viewer?url=http%3A%2F%2Fwww.gabilos.com%2Fcomosehace%2Fcuentasanuales_nuevopgc%2FEstructura_cuenta_perdidas_y_ganancias_modelo_abrev.pdf

Se está usando la cuenta 746 en dos sitios, en el epígrafe 40900 y en 41430, está de acuerdo a la pantilla: http://cuentasanuales.info/index.php/PyG/cuenta-de-perdidas-y-ganancias-abreviada.html
pero hay otras que no hacen este doble apunte y creo que sería lo normal: https://docs.google.com/viewer?url=http%3A%2F%2Fwww.gabilos.com%2Fcomosehace%2Fcuentasanuales_nuevopgc%2FEstructura_cuenta_perdidas_y_ganancias_modelo_abrev.pdf

To post a comment you must log in.
Revision history for this message
Alejandro Santana (alejandrosantana) wrote :

Hola, me han pasado este análisis de los cambios, a nivel contable, que os pego abajo. Los comentarios empiezan por ">>>".

"
Se incluye la siguiente corrección en las plantillas:
1. Plantilla pyg pymes:
En el epígrafe de Resultado de explotación no se estaba sumando el epígrafe 41000.

>>> Respuesta 1: Si, aparentemente viendo la plantilla si lo suma pero luego he realizado una prueba y no lo suma, con lo que la rectificación es correcta.

2. Plantilla pyg pymes:
No se están teniendo en cuenta en el epígrafe de Gastos de personal, las cuentas 7950 y 7957, si se tienen en cuenta en el abreviado.

>>> Respuesta 2: 7950 (Exceso de provisión para retribuciones y otras prestaciones al personal) y 7957 (Exceso de provisión por transacciones con pagos basados en instrumentos de patrimonio) son dos cuentas que según he visto en las plantillas más utilizadas para realizar PyG para Pymes no figuran dentro del epígrafe 40600 (Gastos de Personal), con lo que yo lo dejaría tal y como está, es decir que siga sin tenerse en cuenta.

3. Plantilla pyg abreviado
Hay dos epígrafes que no deberían estar ya que sino no se distingue en casi nada la abreviada de la de pymes, los epígrafes en cuestión son, el 41300 y el 41200.

>>> Respuesta 3: El BOE página 119 del documento que adjunto (http://www.boe.es/boe/dias/2009/02/10/pdfs/BOE-A-2009-2276.pdf) dice que estas dos cuentas deben figurar, con lo que no hay mucho más que decir. Deben estar, para mi la rectificación es errónea.

4. Plantilla pyg abreviado
Se está usando la cuenta 746 en dos sitios, en el epígrafe 40900 y en 41430.

>>> Respuesta 4: Ídem respuesta 3. El que propone la rectificación argumenta que ha visto plantilla que no utilizan esta doble contabilización de las subvenciones. Esto puede ser debido a que la subvención en función de su uso, de su objetivo, si se utiliza para financiar un gasto, una inversión, un inmovilizado... tiene una contabilización distinta y esto en realidad es muy difícil de definir puesto que una misma subvención puede utilizarse para todas estas cosas conjuntamente y quizás por eso la gente aglutina todo en una sola cuenta contable.
De todos modos el BOE dice que debe figurar, con lo que para mi la rectificación es errónea.

"

En función de eso, habría que revisar bien qué cuentas incluir. Alguien más con experiencia contable que argumente también.

review: Needs Fixing (read proposal)
Revision history for this message
Omar (Pexego) (omar7r) wrote :

Hola Alejandro,

Gracias por la respuesta, si te fijas en el código lo único que está corregido es lo primero lo demás eran observacioens que hice revisando el módulo, por que ya puestos, si estuviaran mal las incluía con nuevos commits, se ve mucho cambio en el diff pero es por las tabulaciones de los fichero que se conviertieron en espacios al editarlos.
Vengo a decir que, el merge no puede estar como need fixing porque no hay nada que solucionar, simplemente descartamos incluir el resto de cambios si estamos todos de acuerdo.

Un saludo

Revision history for this message
Alejandro Santana (alejandrosantana) wrote :

Hola, Omar.

Debería haberle dado a "Needs information". Los comentarios eran ya de cara a los futuribles, para que quedara documentado lo que me han dicho al respecto.

Ahora reviso el código y dejo la revisión "efectiva" hecha.

Revision history for this message
Alejandro Santana (alejandrosantana) wrote :

Revisando el código que afecta al punto 1 (Resultado de explotación), todo correcto.

Lo demás, que comente alguien con más conocimientos contables.

review: Approve (code review (no test))
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Aunque no consigo ver cuál es la diferencia en la fórmula para el resultado de explotación, paso a hacer el merge, ya que el archivo XML se queda mejor con espacios, pero me gustaría que me lo dijeras, Omar, para ver si hace falta replicarlo en la v7.

Un saludo.

review: Approve (code review)
Revision history for this message
Omar (Pexego) (omar7r) wrote :

La corrección está en la plantilla de pymes, en el diff linea 735:
40100+40200+40300+40400+40500+40600+40700+40800+40900+40000+41100+41300
y se corrige en la linea del diff 979:
40100+40200+40300+40400+40500+40600+40700+40800+40900+40000+41000+41100+41300

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Omar, muchas gracias por apuntar el problema. Ya lo he visto, y he hecho estos dos PRs a la rama 7.0 y 8.0:

https://github.com/OCA/l10n-spain/pull/3
https://github.com/OCA/l10n-spain/pull/4

¿Puedes por favor revisarlos y aprobarlos?

Un saludo.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.