Merge lp://qastaging/~vauxoo/addons-vauxoo/7.0-average_stock_valuation_journal_items-dev-yani into lp://qastaging/addons-vauxoo/7.0

Hola Yani,

Estuve intentando probar el modulo, en una bd nueva.

El procedimiento fue el siguiente:
1.- Creé bd nueva con datos de prueba.
2.- Instale el modulo: periodic_inventory_valuation.
3.- Asigne permisos al usuario admin para que pueda ver el modulo.
4.- Configuré diario en la compañía.
5.- Creeé un nuevo registro.
6.- Coloqué nombre.
7.- Coloqué fecha= 07/01/2013 (01 de Julio de 2013).
8.- Agregue productos en la pestaña: Products to Value.
9.- Al presionar "Save" o "Update Valuation" marca el siguiente error:

Please create one from the configuration of the accounting menu.
Traceback (most recent call last):
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/netsvc.py", line 292, in dispatch_rpc
    result = ExportService.getService(service_name).dispatch(method, params)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/service/web_services.py", line 626, in dispatch
    res = fn(db, uid, *params)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 188, in execute_kw
    return self.execute(db, uid, obj, method, *args, **kw or {})
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 131, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 197, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 185, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/gabriela/instancias/7_0/produccion_actual/addons-vauxoo/periodic_inventory_valuation/model/periodic_inventory_valuation.py", line 159, in create
    vals['period_id'] = self.get_period(cr, uid, False, vals.get('date'), context=context)
  File "/home/gabriela/instancias/7_0/produccion_actual/addons-vauxoo/periodic_inventory_valuation/model/periodic_inventory_valuation.py", line 104, in get_period
    raise osv.except_osv(_('Error!'), _('There is no fiscal year defined for this date.\nPlease create one from the configuration of the accounting menu.'))
except_osv: (u'Error!', u'There is no fiscal year defined for this date.\nPlease create one from the configuration of the accounting menu.')

PD. Si tengo el ejercicio fiscal con los periodos mensuales creados. No se, si se te llegó a presentar este error.

server: 5206.
addons: 9294
web:3994

Ya corregí ese error Yani, era la configuración del periodo como especial.

Ahora cuando estoy en estado "Ready to Valuate" e intento "Update Valuation" explota de la siguiente forma:

2013-07-12 05:29:28,690 4477 INFO INVENTORY_1 werkzeug: 127.0.0.1 - - [12/Jul/2013 05:29:28] "POST /web/dataset/call_kw HTTP/1.1" 200 -
2013-07-12 05:29:28,859 4477 INFO INVENTORY_1 werkzeug: 127.0.0.1 - - [12/Jul/2013 05:29:28] "POST /web/dataset/search_read HTTP/1.1" 200 -
2013-07-12 05:29:29,429 4477 ERROR INVENTORY_1 openerp.osv.osv: Uncaught exception
Traceback (most recent call last):
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 131, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 197, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/gabriela/instancias/7_0/produccion_actual/server/openerp/osv/osv.py", line 185, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/gabriela/instancias/7_0/produccion_actual/addons-vauxoo/periodic_inventory_valuation/model/periodic_inventory_valuation.py", line 345, in load_valuation_items
    inventario_final -= k.get('qty')
UnboundLocalError: local variable 'inventario_final' referenced before assignment

Hola Gaby, efectivamente faltaban validaciones, prueba de nuevo por favor, el campo de "products to valuate" está ahora readonly, ya que se cargan automáticamente al hacer click en el botón. Lo único que debes llenar es el nombre y la fecha.

Hola gaby, creo que no te llegó el correo del comentario anterior, ahora sí. Prueba y me cuentas por favor, Saludos.

En mi opinión, esta en condiciones para mezclar, luego podrían afinarse detalles.

Si ya lo mezclo, haga la última revisión y mezclo
On Jul 15, 2013 4:33 PM, "Yanina Aular (Vauxoo)" <email address hidden>
wrote:

> Review: Approve
>
> En mi opinión, esta en condiciones para mezclar, luego podrían afinarse
> detalles.
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-average_stock_valuation_journal_items-dev-yani/+merge/174475
> You are requested to review the proposed merge of
> lp:~vauxoo/addons-vauxoo/7.0-average_stock_valuation_journal_items-dev-yani
> into lp:addons-vauxoo/7.0.
>

Buenos Dias, Yani & Gaby, que otras cosas hacen falta para
mezclar esto al branch principal.

Saludos.

Hola Bto, ya Yanina me avisó que realizó los cambios, hace falta que yo los revisé para que se realice el merge. Una vez que lo revise les comento por este medio.

Saludos. :)

Hola Yani,

Una pregunta, los cambios que marcaste como DONE, como:

-Colocar el campo date a la derecha del campo name. (DONE)
-Ampliar la ayuda del modulo en el __openerp__.py de modo que indique cosas como: (DONE)

Los colocaste en otro branch, ya que no los visualizo desde el branch que mencionas aca.

Saludos.

Hola Gaby, si, si están los cambios, pero no había hecho resubmit proposal. Ya está : )