Imposible borrar multiples productos

Bug #941939 reported by Juan Márquez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
account-management
Fix Committed
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

Revision history for this message
Juan Márquez (jmarquez) wrote :
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.