Merge pull request #8926 from fappels/develop_fix_setvaluefrom
FIX commonobject setValueFrom trigger
This commit is contained in:
commit
4b84b7ccf2
@ -1457,7 +1457,16 @@ abstract class CommonObject
|
|||||||
{
|
{
|
||||||
if ($trigkey)
|
if ($trigkey)
|
||||||
{
|
{
|
||||||
$result=$this->call_trigger($trigkey, (! empty($fuser) && is_object($fuser)) ? $fuser : $user); // This may set this->errors
|
// call trigger with updated object values
|
||||||
|
if (empty($this->fields) && method_exists($this, 'fetch'))
|
||||||
|
{
|
||||||
|
$result = $this->fetch($id);
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
$result = $this->fetchCommon($id);
|
||||||
|
}
|
||||||
|
if ($result >= 0) $result=$this->call_trigger($trigkey, (! empty($fuser) && is_object($fuser)) ? $fuser : $user); // This may set this->errors
|
||||||
if ($result < 0) $error++;
|
if ($result < 0) $error++;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -167,7 +167,7 @@ if (empty($reshook))
|
|||||||
|
|
||||||
if ($result >= 0)
|
if ($result >= 0)
|
||||||
{
|
{
|
||||||
$result = $object->setValueFrom('barcode', GETPOST('barcode'));
|
$result = $object->setValueFrom('barcode', GETPOST('barcode'), '', null, 'text', '', $user, 'PRODUCT_MODIFY');
|
||||||
header("Location: ".$_SERVER['PHP_SELF']."?id=".$object->id);
|
header("Location: ".$_SERVER['PHP_SELF']."?id=".$object->id);
|
||||||
exit;
|
exit;
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user