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

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

Open
pxdeoleo opened this issue Mar 4, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@pxdeoleo
Copy link

pxdeoleo commented Mar 4, 2024

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:

  • Accounting > Journal Entries > Create
  • Boton "Create Bill" desde una orden de compra

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?

  • Modificar las access rules referentes a account.move
  • Modificar las access rules referentes a Latam Document Types
  • Modificar los grupos de Accounting a los que pertenecen los usuarios (Billing, advisor, etc)
  • Actualizar los modulos l10n_do y l10n_do_accouting
  • Actualizar todos los modulos instalados

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.

@pxdeoleo pxdeoleo added the question Further information is requested label Mar 4, 2024
@jose-pcg
Copy link
Member

jose-pcg commented Mar 5, 2024

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?

@jose-pcg jose-pcg added bug Something isn't working and removed question Further information is requested labels Mar 5, 2024
@jose-pcg jose-pcg changed the title [13.0] l10n_do_accounting: No se pueden crear asientos contables luego de actualizar submodulo [13.0][BUG] l10n_do_accounting: No se pueden crear asientos contables luego de actualizar submodulo Mar 5, 2024
@pxdeoleo
Copy link
Author

pxdeoleo commented Mar 5, 2024

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
Copy link
Author

pxdeoleo commented Mar 5, 2024

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.

@jose-pcg
Copy link
Member

jose-pcg commented Mar 5, 2024

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

@jose-pcg
Copy link
Member

jose-pcg commented Mar 7, 2024

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

@pxdeoleo
Copy link
Author

pxdeoleo commented Mar 7, 2024

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants