indexa-git / l10n-dominicana

Repositorio con las funcionalidades básicas para la emisión de facturas fiscales dominicanas en Odoo.
GNU Lesser General Public License v3.0
98 stars 121 forks source link

[13.0][BUG] l10n_do_accounting: No se pueden crear asientos contables luego de actualizar submodulo #1149

Open pxdeoleo opened 5 months ago

pxdeoleo commented 5 months ago

Describe your question Luego de actualizar el 29 de febrero al commit [FIX] l10n_do_accounting: ValueError: day is out of range for month, no se ha podido crear ningún asiento contable/factura de proveedor que utilice NCF.

Al tratar de crear un asiento desde:

Al tratar de hacer una factura de proveedor o un asiento contable individual, se presenta el error:

The requested operation ("read" on "Journal Entries" (account.move)) was rejected because of the following rules:

  • All Journal Entries
  • Personal Invoice
  • All Invoices
  • Purchase User Account Move
  • Team Invoice

(Records: Draft Entry (* <NewId 0x7f339e1f38d0>) (id=<NewId 0x7f339e1f38d0>), User: Pedro De Oleo (id=6))

Esto se mantiene independientemente de los grupos a los que pertenezca el usuario o las reglas de acceso que tenga. Incluyendo al usuario Administrator.

Al continuar con la creación de un Bill luego del mensaje, no tiene líneas creadas (ya que es un bill, debería tener las líneas a facturar de la orden de compra desde la que se genera).

What have you tried?

Additional context Indagando en el código de l10n_do_accounting, pude aislar la línea que dispara la excepción de los accesos:

self.write({"l10n_do_company_in_contingency": False}) Módulo l10n_do_accounting Archivo account.move.py Línea 275 Función _compute_company_in_contingency

Desconozco cómo funciona esta parte del código, qué hace, o por qué empezó a dar problemas luego de actualizar el submódulo, pero es la única pista que he logrado encontrar. Cuando la comento se pueden crear facturas y asientos de nuevo sin inconveniente.

Edit: No estoy seguro de si esto cae dentro de la categoría question o bug. Por favor moverlo a donde corresponda si es necesario.

jlopezg commented 5 months ago

Hola @pxdeoleo

Gracias por reportar este issue de manera tan detallada.

Estuve intentando reproducir el error en una base de datos nueva pero no lo logré. ¿Tienes posibilidad de reproducirlo en una base de datos nueva?

pxdeoleo commented 5 months ago

Hola @jlopezg.

Lamentablemente no, he estado tratando tambien de reproducirlo en una bd nueva pero no lo he logrado tampoco.

Nosotros utilizamos Odoo.sh, asi que subí un build con el último commit que estabamos usando de l10n-dominicana 13.0 antes de actualizar, que era el b25643. Para luego actualizar al mas reciente, pero tampoco logré nada.

Al momento simplemente estoy tratando de encontrar cualquier solucion posible, aunque sea temporal, ya que debido a esto no podemos registrar facturas, y los usuarios que tienen acceso read-only a las facturas también estan teniendo el mismo problema.

Aquí algunos de los permisos de los account.move. Entiendo que con esto no deberían al menos surgir problema de permisos. image

pxdeoleo commented 5 months ago

Por el momento, haciendo un rollback al commit b25643, todo vuelve a funcionar correctamente.

Estaré actualizando gradualmente al commit más reciente para probar exactamente en cuál empieza a dar problemas. Aunque esto sería en una base de datos de pruebas (un clon de producción), ya que en una base de datos completamente nueva no parece haber problemas.

Edit: Es exactamente al aplicar el commit ca4c5b que vuelve el error de los permisos. Que es cuando se agrega la linea que mencioné más arriba.

jlopezg commented 5 months ago

Entendido @pxdeoleo Por favor déjanos saber si te podemos ayudar de alguna otra manera. Mientras tanto dejaré este issue abierto.

jlopezg commented 5 months ago

Hola @pxdeoleo He subido el branch 13.0-fix-003-jl con el posible fix. ¿Podrías probarlo por favor?

pxdeoleo commented 5 months ago

Hola @jlopezg Probé y ocurrió lo mismo del principio.

Aquí parte del log:

2024-03-07 19:27:40,794 4 INFO db-odoo werkzeug: 127.0.0.1 - - [07/Mar/2024 19:27:40] "POST /web/dataset/call_kw/account.move/default_get HTTP/1.0" 200 - 17 0.014 0.019
2024-03-07 19:27:41,014 4 INFO db-odoo odoo.addons.base.models.ir_rule: Access Denied by record rules for operation: read on record ids: [], uid: 2, model: account.move 
2024-03-07 19:27:41,060 4 INFO db-odoo odoo.addons.base.models.ir_rule: Access Denied by record rules for operation: read on record ids: [], uid: 2, model: account.move 
2024-03-07 19:27:41,065 4 WARNING db-odoo odoo.http: ('The requested operation ("read" on "Journal Entries" (account.move)) was rejected because of the following rules:\n- All Journal Entries\n- Personal Invoice\n- All Invoices\n- Purchase User Account Move\n- Team Invoice\n\n(Records: Draft Entry (* <NewId 0x7f121ebe6cf8>) (id=<NewId 0x7f121ebe6cf8>), User: Administrator (id=2))', None)