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.

Sign up for GitHub

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

Jump to bottom

Error en metodo para validar varias facturas juntas. #28

Open
santcosta opened this issue Apr 15, 2021 · 1 comment
Open

Error en metodo para validar varias facturas juntas. #28

santcosta opened this issue Apr 15, 2021 · 1 comment
Assignees
@sotolucas
Labels
bug Something isn't working

Comments

@santcosta
Copy link

santcosta commented Apr 15, 2021 •
edited
Loading

Version afectada:

13.0

Basicamente al querer confirmar varias facturas el sistema arroja un error .

imagen

Lo que pasa actualmente:

Throws an error:

"File "/opt/odoo/modulos/odoo-argentina-ce/l10n_ar_afipws_fe/models/account_move.py", line 225, in do_pyafipws_request_cae
    amounts = self._l10n_ar_get_amounts()
  File "/opt/odoo/odoo/addons/l10n_ar/models/account_move.py", line 193, in _l10n_ar_get_amounts
    self.ensure_one()
  File "/opt/odoo/odoo/odoo/models.py", line 5082, in ensure_one
    raise ValueError("Expected singleton: %s" % self)"

En si estuve viendo el codigo y creo que es algo sencillo... en la funcion do_pyafipws_request_cae se realiza un for por cada inv
pero al momento de solicitar los montos linea #244 se lo llama con SELF , entonces se genera el error si tenes mas de una factura seleccionada.

Despues revisaria las lineas donde tambien se utiliza el SELF en vez de inv
#393
#401
#408
#414
#427
#434
#441

Saludos
@sotolucas
Copy link
Contributor

sotolucas commented May 31, 2021 •
edited
Loading

Hola @santcosta , podés proponer una solución mediante un PR y lo testeo?

@sotolucas sotolucas self-assigned this May 31, 2021
@sotolucas sotolucas added the bug Something isn't working label May 31, 2021
maq-adhoc pushed a commit to adhoc-dev/odoo-argentina-ce that referenced this issue Jan 15, 2023
@filoquin
[FIX] Fix issue ingadhoc#28 and ingadhoc#32
bd114fd
roboadhoc pushed a commit that referenced this issue Jan 30, 2023
@filoquin
[FIX] Fix issue #28 and #32
b6f0701 
Part-of: #47
fw-bot-adhoc pushed a commit to adhoc-dev/odoo-argentina-ce that referenced this issue Jan 30, 2023
@filoquin
Cherry pick of bd114fd failed
8aa240f 
stdout:
Auto-merging l10n_ar_afipws_fe/models/account_move.py
CONFLICT (content): Merge conflict in l10n_ar_afipws_fe/models/account_move.py

stderr:
20:09:22.272723 git.c:444               trace: built-in: git cherry-pick bd114fd
error: could not apply bd114fd... [FIX] Fix issue ingadhoc#28 and  ingadhoc#32
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'
----------
status:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sotolucas sotolucas

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@santcosta @sotolucas