Skip to content

[FIX] stock_voucher_ux: asignar números de remito antes de imprimir#951

Open
fw-bot-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-18.0-t-67613-mav-6202-fw
Open

[FIX] stock_voucher_ux: asignar números de remito antes de imprimir#951
fw-bot-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-18.0-t-67613-mav-6202-fw

Conversation

@fw-bot-adhoc

Copy link
Copy Markdown
Contributor

Problema

Al usar el botón Print Vouchers en un remito con autoprinted=False (formulario pre-impreso), los números de remito se asignaban después de generar el PDF (en el controller HTTP), por lo que la primera impresión salía sin número. El usuario tenía que imprimir dos veces.

Solución

En do_print_and_assign, para el caso autoprinted=False, se llama assign_numbers(get_estimated_number_of_pages(), book_id) antes de invocar do_print_voucher(), igual que el flujo de autoprinted=True.

El controller conserva la guarda if not voucher_ids que evita doble asignación cuando ya fueron pre-asignados.

Task

https://www.adhoc.inc/odoo/knowledge/180/902/tasks/67613

Forward-Port-Of: #941

stdout:
Auto-merging stock_declared_value/__manifest__.py
CONFLICT (content): Merge conflict in stock_declared_value/__manifest__.py
CONFLICT (modify/delete): stock_voucher_ux/models/stock_picking.py deleted in 8b1493d and modified in ff458cb.  Version ff458cb of stock_voucher_ux/models/stock_picking.py left in tree.

stderr:
@roboadhoc

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@fw-bot-adhoc

Copy link
Copy Markdown
Contributor Author

@mav-adhoc @les-adhoc cherrypicking of pull request #941 failed.

stdout:

Auto-merging stock_declared_value/__manifest__.py
CONFLICT (content): Merge conflict in stock_declared_value/__manifest__.py
CONFLICT (modify/delete): stock_voucher_ux/models/stock_picking.py deleted in 8b1493de9906b9f991195e66de149c8aa7f66608 and modified in ff458cb0fca8277705f76a82d8fc6d68fd56ca4f.  Version ff458cb0fca8277705f76a82d8fc6d68fd56ca4f of stock_voucher_ux/models/stock_picking.py left in tree.

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

:shipit: you can use git-fw to re-do the forward-port for you locally.

⚠️ after resolving this conflict, you will need to merge it via @roboadhoc.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants