Merge pull request #5848 from atm-florian/develop

FIX delete line with no triggers
This commit is contained in:
Laurent Destailleur 2016-10-07 17:18:07 +02:00 committed by GitHub
commit 4ae3d7e034

View File

@ -3747,9 +3747,10 @@ class PropaleLigne extends CommonObjectLine
/** /**
* Delete line in database * Delete line in database
* *
* @param int $notrigger 1=Does not execute triggers, 0= execute triggers
* @return int <0 if ko, >0 if ok * @return int <0 if ko, >0 if ok
*/ */
function delete() function delete($notrigger=0)
{ {
global $conf,$user; global $conf,$user;
@ -3773,13 +3774,16 @@ class PropaleLigne extends CommonObjectLine
} }
} }
// Call trigger if (! $error && ! $notrigger)
$result=$this->call_trigger('LINEPROPAL_DELETE',$user); {
if ($result < 0) // Call trigger
{ $result=$this->call_trigger('LINEPROPAL_DELETE',$user);
$this->db->rollback(); if ($result < 0)
return -1; {
} $this->db->rollback();
return -1;
}
}
// End call triggers // End call triggers
$this->db->commit(); $this->db->commit();