lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

NEW 16.0 - proposal for a prefix-based consistency mechanism for trigger names

Browse Source
This commit is contained in:
atm-florian 2022-04-01 19:20:47 +02:00
parent ab4e89d678
commit 96c432566d
1 changed files with 25 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
htdocs/core/class/commonobject.class.php
View File

@ -42,6 +42,7 @@
*/ */
abstract class CommonObject abstract class CommonObject
{ {
const TRIGGER_PREFIX = ''; // to be overriden in child class implementations, i.e. 'BILL', 'TASK', 'PROPAL', etc.
/** /**
* @var DoliDb Database handler (result of a new DoliDB) * @var DoliDb Database handler (result of a new DoliDB)
*/ */
@ -1987,6 +1988,10 @@ abstract class CommonObject
$result = $this->fetchCommon($id); $result = $this->fetchCommon($id);
} }
if ($result >= 0) { if ($result >= 0) {
if (!empty(self::TRIGGER_PREFIX) && strpos($trigkey, self::TRIGGER_PREFIX) !== 0) {
$trigkey = self::TRIGGER_PREFIX . '_' . $trigkey;
}
$result = $this->call_trigger($trigkey, (!empty($fuser) && is_object($fuser)) ? $fuser : $user); // This may set this->errors $result = $this->call_trigger($trigkey, (!empty($fuser) && is_object($fuser)) ? $fuser : $user); // This may set this->errors
} }
if ($result < 0) { if ($result < 0) {
@ -4269,6 +4274,9 @@ abstract class CommonObject
if ($trigkey) { if ($trigkey) {
// Call trigger // Call trigger
if (!empty(self::TRIGGER_PREFIX) && strpos($trigkey, self::TRIGGER_PREFIX) !== 0) {
$trigkey = self::TRIGGER_PREFIX . '_' . $trigkey;
}
$result = $this->call_trigger($trigkey, $user); $result = $this->call_trigger($trigkey, $user);
if ($result < 0) { if ($result < 0) {
$error++; $error++;
@ -5603,6 +5611,9 @@ abstract class CommonObject
// phpcs:enable // phpcs:enable
global $langs, $conf; global $langs, $conf;
if (!empty(self::TRIGGER_PREFIX) && strpos($triggerName, self::TRIGGER_PREFIX) !== 0) {
$triggerName = self::TRIGGER_PREFIX . '_' . $triggerName;
}
if (!is_object($langs)) { // If lang was not defined, we set it. It is required by run_triggers. if (!is_object($langs)) { // If lang was not defined, we set it. It is required by run_triggers.
include_once DOL_DOCUMENT_ROOT.'/core/class/translate.class.php'; include_once DOL_DOCUMENT_ROOT.'/core/class/translate.class.php';
$langs = new Translate('', $conf); $langs = new Translate('', $conf);
@ -6190,6 +6201,9 @@ abstract class CommonObject
if (!$error && $trigger) { if (!$error && $trigger) {
// Call trigger // Call trigger
$this->context = array('extrafieldaddupdate'=>1); $this->context = array('extrafieldaddupdate'=>1);
if (!empty(self::TRIGGER_PREFIX) && strpos($trigger, self::TRIGGER_PREFIX) !== 0) {
$trigger = self::TRIGGER_PREFIX . '_' . $trigger;
}
$result = $this->call_trigger($trigger, $userused); $result = $this->call_trigger($trigger, $userused);
if ($result < 0) { if ($result < 0) {
$error++; $error++;
@ -6308,6 +6322,9 @@ abstract class CommonObject
if (!$error && $trigger) { if (!$error && $trigger) {
// Call trigger // Call trigger
$this->context = array('extralanguagesaddupdate'=>1); $this->context = array('extralanguagesaddupdate'=>1);
if (!empty(self::TRIGGER_PREFIX) && strpos($trigger, self::TRIGGER_PREFIX) !== 0) {
$trigger = self::TRIGGER_PREFIX . '_' . $trigger;
}
$result = $this->call_trigger($trigger, $userused); $result = $this->call_trigger($trigger, $userused);
if ($result < 0) { if ($result < 0) {
$error++; $error++;
@ -6506,6 +6523,10 @@ abstract class CommonObject
if (!$error && $trigger) { if (!$error && $trigger) {
// Call trigger // Call trigger
$this->context = array('extrafieldupdate'=>1); $this->context = array('extrafieldupdate'=>1);
if (!empty(self::TRIGGER_PREFIX) && strpos($trigger, self::TRIGGER_PREFIX) !== 0) {
$trigger = self::TRIGGER_PREFIX . '_' . $trigger;
}
$result = $this->call_trigger($trigger, $userused); $result = $this->call_trigger($trigger, $userused);
if ($result < 0) { if ($result < 0) {
$error++; $error++;
@ -9426,6 +9447,10 @@ abstract class CommonObject
if (!$error && !$notrigger) { if (!$error && !$notrigger) {
// Call trigger // Call trigger
if (!empty(self::TRIGGER_PREFIX) && strpos($triggercode, self::TRIGGER_PREFIX) !== 0) {
$triggercode = self::TRIGGER_PREFIX . '_' . $triggercode;
}
$result = $this->call_trigger($triggercode, $user); $result = $this->call_trigger($triggercode, $user);
if ($result < 0) { if ($result < 0) {
$error++; $error++;