adhoc-dev / odoo-addons

DEPRECEATED, see README
GNU Affero General Public License v3.0
71 stars 124 forks source link

[FIX][8.0][stock_voucher] Default value 'book_id' when 'book_required' is true #253

Closed ivantodorovich closed 8 years ago

ivantodorovich commented 8 years ago

Copie la funcionalidad de stock_print_remit a stock_transfer_details, para que complete automáticamente el book_id de acuerdo a lo configurado en stock.picking.type.

Revisando el código se sugiere que esa era la intención original (@api.onchange('picking_id')), pero por alguna razón nunca funcionó. Haciendo algunas pruebas, parece que picking_id nunca está definido, lo cual es raro, pero por eso no funcionaba. Me tomé el atrevimiento de eliminar ese bloque, ya que parece no tener finalidad.

Pero la verdad no tengo experiencia con odoo así que puedo estar terriblemente equivocado. Por esa razón, además, me limité únicamente a copiar y pegar parte del código de un archivo a otro, sin modificar nada que no sea necesario.

Pero, sin entender demasiado, estas líneas me hacen mucho ruido:

        if not active_id:
            active_id = 24

¿¿Está bien eso ahí??

Por las dudas lo dejé tal cual lo encontré, pero si es en efecto algo que no debería estar ahí, hay que corregirlo también en stock_print_remit.

jjscarafia commented 8 years ago

Buenas! Gracias por el PR.

  1. Si, claramente está muy mal eso de
        if not active_id:
            active_id = 24
  1. Borralo por favor en ambos archivos "stock print remit" y "stock transfer details" Borrado eso aceptamos el PR. Gracias!
ivantodorovich commented 8 years ago

@jjscarafia Listo!

jjscarafia commented 8 years ago

Gracias!