vignesh/dolibarr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add a try catch on computed extrafields

Browse source
This commit is contained in:
Laurent Destailleur 2026-06-20 05:41:54 +02:00
parent ed379e4a7c
commit 7d8a6dfd44
1 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
htdocs/core/class/commonobject.class.php
View file

@ -6775,7 +6775,6 @@ abstract class CommonObject
if (!$this->table_element) {
return 0;
}
$this->array_options = array();
if (!is_array($optionsArray)) {
@ -6822,7 +6821,6 @@ abstract class CommonObject
$numrows = $this->db->num_rows($resql);
if ($numrows) {
$tab = $this->db->fetch_array($resql);
foreach ($tab as $key => $value) {
// Test fetch_array ! is_int($key) because fetch_array result is a mix table with Key as alpha and Key as int (depend db engine)
if ($key != 'rowid' && $key != 'tms' && $key != 'fk_member' && !is_int($key)) {
@ -6861,9 +6859,16 @@ abstract class CommonObject
if (!empty($extrafields->attributes[$this->table_element]) && !empty($extrafields->attributes[$this->table_element]['computed'][$key])) {
//var_dump($conf->disable_compute);
if (empty($conf->disable_compute)) {
global $objectoffield; // We set a global variable to $objectoffield so
$objectoffield = $this; // we can use it inside computed formula
$this->array_options['options_' . $key] = dol_eval((string) $extrafields->attributes[$this->table_element]['computed'][$key], 1, 0, '2');
// We set a global variable to $objectoffield so we can use it inside computed formula
$objectoffield = dol_clone($this, 2);
global $objectoffield;
try {
$this->array_options['options_' . $key] = dol_eval((string) $extrafields->attributes[$this->table_element]['computed'][$key], 1, 0, '2');
} catch (Exception $e) {
// If exception by dol_eval, we set value to null
dol_syslog('Error in dol_eval of a computed extrafield: '.$e->getMessage(), LOG_WARNING);
$this->array_options['options_' . $key] = null;
}
}
}
}
@ -6877,7 +6882,7 @@ abstract class CommonObject
return 0;
}
} else {
$this->errors[] = $this->db->lasterror;
$this->errors[] = $this->db->lasterror();
return -1;
}
}