Improve logging on LSobject & LSattribute triggers executions

This commit is contained in:
Benjamin Renard 2020-08-26 18:08:14 +02:00
parent 0de62c218d
commit 4c427b26d6
3 changed files with 50 additions and 14 deletions

View file

@ -80,6 +80,14 @@ class LSattribute extends LSlog_staticLoggerClass {
return true; return true;
} }
/**
* Allow conversion of LSattribute to string
*
* @retval string The string representation of the LSattribute
*/
public function __toString() {
return strval($this -> ldapObject)." -> <LSattribute ".$this -> name.">";
}
/** /**
* Retourne la valeur du label de l'attribut * Retourne la valeur du label de l'attribut
@ -625,25 +633,25 @@ class LSattribute extends LSlog_staticLoggerClass {
* @retval boolean True si tout c'est bien passé, false sinon * @retval boolean True si tout c'est bien passé, false sinon
*/ */
public function fireEvent($event) { public function fireEvent($event) {
self :: log_debug(strval($this)." -> fireEvent($event)");
$return = true; $return = true;
if(isset($this -> config[$event])) { if(isset($this -> config[$event])) {
if (!is_array($this -> config[$event])) { $funcs = (!is_array($this -> config[$event])?array($this -> config[$event]):$this -> config[$event]);
$funcs = array($this -> config[$event]);
}
else {
$funcs = $this -> config[$event];
}
foreach($funcs as $func) { foreach($funcs as $func) {
if(function_exists($func)) { if(function_exists($func)) {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable($func));
if(!call_user_func_array($func, array(&$this -> ldapObject))) { if(!call_user_func_array($func, array(&$this -> ldapObject))) {
$return = false; $return = false;
} }
} }
else { else {
self :: log_warning(strval($this)." -> fireEvent($event): function '".format_callable($func)."' doesn't exists.");
$return = false; $return = false;
} }
} }
} }
else
self :: log_trace(strval($this)." -> fireEvent($event): no configured trigger for this event.");
if (isset($this -> _events[$event]) && is_array($this -> _events[$event])) { if (isset($this -> _events[$event]) && is_array($this -> _events[$event])) {
foreach ($this -> _events[$event] as $e) { foreach ($this -> _events[$event] as $e) {
@ -652,35 +660,37 @@ class LSattribute extends LSlog_staticLoggerClass {
$obj = new $e['class'](); $obj = new $e['class']();
if (method_exists($obj,$e['fct'])) { if (method_exists($obj,$e['fct'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable(array($obj, $e['fct'])));
call_user_func_array(array($obj, $e['fct']), array(&$e['params'])); call_user_func_array(array($obj, $e['fct']), array(&$e['params']));
} }
catch(Exception $er) { catch(Exception $er) {
$return = false; self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable(array($obj, $e['fct'])));
LSdebug("Event ".$event." : Erreur durant l'execution de la méthode ".$e['fct']." de la classe ".$e['class']);
}
}
else {
LSdebug("Event ".$event." : La méthode ".$e['fct']." de la classe ".$e['class']." n'existe pas.");
$return = false; $return = false;
} }
} }
else { else {
self :: log_warning(strval($this)." -> fireEvent($event): method '".$e['fct']."' of the class '".$e['class']."' doesn't exists.");
$return = false;
}
}
else {
self :: log_warning(strval($this)." -> fireEvent($event): the class '".$e['class']."' doesn't exists.");
$return = false; $return = false;
LSdebug("Event ".$event." : La classe ".$e['class']." n'existe pas");
} }
} }
else { else {
if (function_exists($e['fct'])) { if (function_exists($e['fct'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable($e['fct']));
call_user_func_array($e['fct'], array(&$e['params'])); call_user_func_array($e['fct'], array(&$e['params']));
} }
catch(Exception $er) { catch(Exception $er) {
LSdebug("Event ".$event." : Erreur durant l'execution de la function ".$e['fct']); self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable(e['fct']));
$return = false; $return = false;
} }
} }
else { else {
LSdebug("Event ".$event." : la function ".$e['fct']." n'existe pas"); self :: log_warning(strval($this)." -> fireEvent($event): the function '".$e['fct']."' doesn't exists.");
$return = false; $return = false;
} }
} }
@ -689,17 +699,18 @@ class LSattribute extends LSlog_staticLoggerClass {
if (isset($this -> _objectEvents[$event]) && is_array($this -> _objectEvents[$event])) { if (isset($this -> _objectEvents[$event]) && is_array($this -> _objectEvents[$event])) {
foreach ($this -> _objectEvents[$event] as $e) { foreach ($this -> _objectEvents[$event] as $e) {
if (method_exists($e['obj'],$e['meth'])) { if (method_exists($e['obj'], $e['meth'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable(array($e['obj'], $e['meth'])));
call_user_func_array(array($e['obj'], $e['meth']),array(&$e['params'])); call_user_func_array(array($e['obj'], $e['meth']),array(&$e['params']));
} }
catch(Exception $er) { catch(Exception $er) {
self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable(array($e['obj'], $e['meth'])));
$return = false; $return = false;
LSdebug("Event ".$event." : Erreur durant l'execution de la méthode ".$e['meth']." sur l'objet.");
} }
} }
else { else {
LSdebug("Event ".$event." : La méthode ".$e['meth']." de l'objet n'existe pas."); self :: log_warning(strval($this)." -> fireEvent($event): the method '".$e['meth']."' of the object doesn't exists.");
$return = false; $return = false;
} }
} }

View file

@ -1680,6 +1680,7 @@ class LSldapObject extends LSlog_staticLoggerClass {
* @retval boolean True si tout c'est bien passé, false sinon * @retval boolean True si tout c'est bien passé, false sinon
*/ */
public function fireEvent($event) { public function fireEvent($event) {
self :: log_debug(strval($this)." -> fireEvent($event)");
// Object event // Object event
$return = $this -> fireObjectEvent($event); $return = $this -> fireObjectEvent($event);
@ -1691,6 +1692,7 @@ class LSldapObject extends LSlog_staticLoggerClass {
$funcs = array($this -> config[$event]); $funcs = array($this -> config[$event]);
foreach($funcs as $func) { foreach($funcs as $func) {
if(function_exists($func)) { if(function_exists($func)) {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable($func));
if(!call_user_func_array($func, array(&$this))) { if(!call_user_func_array($func, array(&$this))) {
$return = false; $return = false;
LSerror :: addErrorCode('LSldapObject_07',array('func' => $func,'event' => $event)); LSerror :: addErrorCode('LSldapObject_07',array('func' => $func,'event' => $event));
@ -1711,9 +1713,11 @@ class LSldapObject extends LSlog_staticLoggerClass {
$obj = new $e['class'](); $obj = new $e['class']();
if (method_exists($obj,$e['fct'])) { if (method_exists($obj,$e['fct'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable(array($obj, $e['fct'])));
call_user_func_array(array($obj,$e['fct']),array(&$e['param'])); call_user_func_array(array($obj,$e['fct']),array(&$e['param']));
} }
catch(Exception $er) { catch(Exception $er) {
self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable(array($obj, $e['fct'])));
LSerror :: addErrorCode('LSldapObject_10',array('class' => $e['class'],'meth' => $e['fct'],'event' => $event)); LSerror :: addErrorCode('LSldapObject_10',array('class' => $e['class'],'meth' => $e['fct'],'event' => $event));
$return = false; $return = false;
} }
@ -1731,9 +1735,11 @@ class LSldapObject extends LSlog_staticLoggerClass {
else { else {
if (function_exists($e['fct'])) { if (function_exists($e['fct'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable($e['fct']));
call_user_func_array($e['fct'],array(&$e['param'])); call_user_func_array($e['fct'],array(&$e['param']));
} }
catch(Exception $er) { catch(Exception $er) {
self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable($e['fct']));
LSerror :: addErrorCode('LSldapObject_27',array('func' => $e['fct'],'event' => $event)); LSerror :: addErrorCode('LSldapObject_27',array('func' => $e['fct'],'event' => $event));
$return = false; $return = false;
} }
@ -1751,9 +1757,11 @@ class LSldapObject extends LSlog_staticLoggerClass {
foreach ($this -> _objectEvents[$event] as $e) { foreach ($this -> _objectEvents[$event] as $e) {
if (method_exists($e['obj'],$e['meth'])) { if (method_exists($e['obj'],$e['meth'])) {
try { try {
self :: log_debug(strval($this)." -> fireEvent($event): run ".format_callable(array($e['obj'], $e['meth'])));
call_user_func_array(array($e['obj'], $e['meth']),array(&$e['param'])); call_user_func_array(array($e['obj'], $e['meth']),array(&$e['param']));
} }
catch(Exception $er) { catch(Exception $er) {
self :: log_exception($er, strval($this)." -> fireEvent($event): exception occured running ".format_callable(array($e['obj'], $e['meth'])));
LSerror :: addErrorCode('LSldapObject_29',array('meth' => $e['meth'],'event' => $event)); LSerror :: addErrorCode('LSldapObject_29',array('meth' => $e['meth'],'event' => $event));
$return = false; $return = false;
} }

View file

@ -747,3 +747,20 @@ function dumpFile($file_path, $mime_type=null, $max_age=3600, $force_download=fa
exit(); exit();
} }
} }
/**
* Format a callable object for logging
* @param callable $callable The callable object
* @return string The callable object string representation
*/
function format_callable($callable) {
if (is_array($callable))
if (is_string($callable[0]))
return $callable[0]."::".$callable[1]."()";
elseif (is_object($callable[0]))
return get_class($callable[0])."->".$callable[1]."()";
else
return "Unkown->".$callable[1]."()";
else
return $callable."()";
}