Imposible borrar multiples productos

Bug #941939 reported by Juan Márquez on 2012-02-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
account-management
Undecided
Unassigned

Bug Description

Al intentar borrar en un sólo paso múltiples productos el modulo elimina uno sólo (el primero).

el problema esta en el modulo account_product.py dentro de product_product.unlink

lo genera la linea:
    return super(product_product, self).unlink(cr, uid, ids, context=context)

Ya que al estar dento de un for el return sale de la ejecución del ciclo.

Lo otro es comentar que me parece innecesarias las validaciones de consistencia, ya que las propias dependencias de la base de datos arrojan el error (que tiene el mensaje en español)

Yo lo solucione así:

    def unlink(self, cr, uid, ids, context={}):

        for i in ids:
            print i
            aml_obj = self.pool.get('account.move.line')
            aml_ids = aml_obj.search(cr, uid, [('product_id','=',i)],context=context)

            ail_obj = self.pool.get('account.invoice.line')
            ail_ids = ail_obj.search(cr, uid, [('product_id','=',i)],context=context)

            pol_obj = self.pool.get('purchase.order.line')
            pol_ids = pol_obj.search(cr, uid, [('product_id','=',i)],context=context)

            sol_obj = self.pool.get('sale.order.line')
            sol_ids = sol_obj.search(cr, uid, [('product_id','=',i)],context=context)

            sm_obj = self.pool.get('stock.move')
            sm_ids = sm_obj.search(cr, uid, [('product_id','=',i)],context=context)

            if any([aml_ids,ail_ids,pol_ids,sol_ids,sm_ids]):
                raise osv.except_osv(_('Error'), _('El producto no puede ser borrado ya que esta siendo usado!'))

        return super(product_product, self).unlink(cr, uid, ids, context=context)

Pero si como creo esas validaciones son innecesarias lo correcto sería quitar por completo el metodo unlink (no hace nada distinto al original).

Related branches

Juan Márquez (jmarquez) wrote :
Juan Márquez (jmarquez) wrote :

El problema principal ya estaba resuelto en la ultima actualización pero insisto: debería borrarse toso el método unlink

Changed in account-management:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments