Code review comment for lp://staging/~inddiana/sisb/jose_l10n_ve_withholding_islr_final

> Por otra parte:
>
> A nivel técnico:
>
> 51 + def create(self,cr,uid,values,context=None):
> 52 + if values.get('type',False) == 'consu':
> 53 + raise osv.except_osv(('Error !'), ('No puede crear un producto como
> consumible !'))
> 54 + if values['purchase_requisition'] == True:
> 55 + raise osv.except_osv(('Error !'), ('Debe marcar la opcion solicitud
> de compra !'))
> 56 + return super(product_product,
> self).create(cr,uid,values,context=None)
>
> Esto es una mala práctica, los raise se deben _evitar_ en los creates al menos
> que estés validando errores de consistencia, es SIEMPRE preferible un
> contraint para lo que estás evaluando acá.
>

En este punto tienes toda la razón.

> Otro punto:
>
> Por que copias tooooodo el módulo y simplemente no heredan del original
> nuestro..... recuerden que nosotros lo tenemos corriendo en varios sitios que
> le vamos encontrando bugs día a día, creo que es mejor de esa manera o que
> bombardeen la localizacion con bugs + patchs, así creamos uno más fuerte y
> documentado, el copy tu pasteo es pan para hoy hambre para mañana (ya nos ha
> pasado.)
>
>

Simple, nuestro módulo de retención va a tomar otro enfoque muy distinto al que actualmente tiene el módulo de ustedes. En estos tres meses de experiencias con el módulo hemos conseguido muchas diferencias que tienen mas que ver a que sin duda ustedes partieron de las necesidades y procedimientos muy particulares de una empresa, asumiendo siempre los mejores escenario posibles. Al final la ley es una sola y la matemática es la misma.

> Propongo_ crear en SISB un módulo que se llame igual con sufijo: EXT (de

NO

> extendido) para poder estabilizarlo y portar lo s cambios poco a poco si
> quieren mantener uno propio o:
>
> HAcer tantas propuestas de merge como cambios requieran, si se fijan:
>
> Toda nuestra localización pasa cliente a cliente por una lista como ésta de
> validaciones:
>
> http://yfrog.com/h4au2ebj
>
> Si ustedes comparten una extensión de ñesta antes o despues de probar,
> avanzaríamos más rápido.
>
> Saludos y disculpen lo largo.....