lucderheld / php-greylog-exception

src/GreyLogException/GreyLogException.php

Doc Comments +7 added lines, -7 removed lines

@@ -233,7 +233,7 @@ 
      * Seaches the key of the given class constant value.
      * @param array $_aExceptionDetails The value of an exception constant.
      * @param string $_sExceptionClass The class the exception constant has to be searched.
-     * @return mixed the key for <i>needle</i> if it is found in the array, <b>FALSE</b> otherwise.
+     * @return string the key for <i>needle</i> if it is found in the array, <b>FALSE</b> otherwise.
      * @todo Could be cached with storing all exceptions in opCache
      * @todo Inner exception should be added when getExceptionCode fails. But at the moment innerNotice function has circular reference to ExceptinShortMessage
      */
@@ -388,7 +388,7 @@ 
 
     /**
      * Sets the ShortMessage and returns message object.
-     * @return Gelf\Message Sets the ShortMessage and returns message object.
+     * @return \Gelf\Message Sets the ShortMessage and returns message object.
      */
     private function setShortMessage() {
         return $this->gelfMessage->setShortMessage($this->iExceptionCode . ":" . $this->sExceptionShortMessage);
@@ -396,7 +396,7 @@ 
 
     /**
      * Sets the Message and returns message object.
-     * @return Gelf\Message Sets the Message and returns message object.
+     * @return \Gelf\Message Sets the Message and returns message object.
      */
     private function setMessage() {
         return $this->gelfMessage->setFullMessage($this->sExceptionMessage);
@@ -404,7 +404,7 @@ 
 
     /**
      * Sets the Level and returns message object.
-     * @return Gelf\Message Sets the Level and returns message object.
+     * @return \Gelf\Message Sets the Level and returns message object.
      */
     private function setLevel() {
         return $this->gelfMessage->setLevel($this->sExceptionLevel);
@@ -412,7 +412,7 @@ 
 
     /**
      * Sets the Facility and returns message object.
-     * @return Gelf\Message Sets the Facility and returns message object.
+     * @return \Gelf\Message Sets the Facility and returns message object.
      */
     private function setFacility() {
         return $this->gelfMessage->setFacility($this->sApplication);
@@ -420,7 +420,7 @@ 
 
     /**
      * Sets additional details from exception object and returns message object.
-     * @return Gelf\Message Sets the additional details from exception object and returns message object.
+     * @return \Gelf\Message Sets the additional details from exception object and returns message object.
      */
     private function setAdditionals() {
         $this->gelfMessage->setAdditional('ModuleName', $this->sModuleName);
@@ -435,7 +435,7 @@ 
     /**
      * Checks if function/method parameters are equal to function declaration and logs warning if not.
      * Gets all names of function/method parameters and its values and adds it to message object as additional.
-     * @return Gelf\Message Sets additional details to message object and returns the message object.
+     * @return \Gelf\Message Sets additional details to message object and returns the message object.
      */
     private function addParameters() {
         //get function/method parameters and parameters function was called

Spacing +13 added lines, -13 removed lines

@@ -163,8 +163,8 @@ 
             parent::__construct($this->sExceptionMessage, $this->iExceptionCode);
             //set additional exception details
             $this->sExceptionShortMessage = $this->getExceptionCode($_aExceptionDetails, get_called_class());
-            $this->sExceptionClass = $this->getTrace()[0]['class'];
-            $this->sExceptionFunction = $this->getTrace()[0]['function'];
+            $this->sExceptionClass = $this->getTrace()[ 0 ][ 'class' ];
+            $this->sExceptionFunction = $this->getTrace()[ 0 ][ 'function' ];
             $this->sModuleName = get_called_class();
             //log exception
             $sFunctionToCall = $this->sExceptionLevel;
@@ -182,12 +182,12 @@ 
      * @param array $_aExceptionDetails The exceptionDetails in form: "[int ExceptionCode, lfException::LOGLEVEL, string ExceptionMessage]".
      */
     private function validateExceptionDetails(Array $_aExceptionDetails) {
-        if (isset($_aExceptionDetails[0]) && is_numeric($_aExceptionDetails[0])) {
-            $this->iExceptionCode = $_aExceptionDetails[0];
-            if (isset($_aExceptionDetails[1]) && method_exists($this, $_aExceptionDetails[1])) {
-                $this->sExceptionLevel = $_aExceptionDetails[1];
-                if (isset($_aExceptionDetails[2]) && is_string($_aExceptionDetails[2])) {
-                    $this->sExceptionMessage = $_aExceptionDetails[2];
+        if (isset($_aExceptionDetails[ 0 ]) && is_numeric($_aExceptionDetails[ 0 ])) {
+            $this->iExceptionCode = $_aExceptionDetails[ 0 ];
+            if (isset($_aExceptionDetails[ 1 ]) && method_exists($this, $_aExceptionDetails[ 1 ])) {
+                $this->sExceptionLevel = $_aExceptionDetails[ 1 ];
+                if (isset($_aExceptionDetails[ 2 ]) && is_string($_aExceptionDetails[ 2 ])) {
+                    $this->sExceptionMessage = $_aExceptionDetails[ 2 ];
                 } else {
                     $this->sExceptionMessage = "__ERR_NOT_FOUND__";
                     $this->innerNotice("WrongConfig", "The exeption thrown, has no exceptionMessage.");
@@ -224,7 +224,7 @@ 
      */
     public function getNumberOfVariables(string $_sReplaceableString) {
         if (preg_match_all("~%(?:(\d+)[$])?[-+]?(?:[ 0]|['].)?(?:[-]?\d+)?(?:[.]\d+)?[%bcdeEufFgGosxX]~", $_sReplaceableString, $iNumberOfVariables) > 0) {
-            return count($iNumberOfVariables[1]);
+            return count($iNumberOfVariables[ 1 ]);
         }
         return 0;
     }
@@ -240,7 +240,7 @@ 
     public function getExceptionCode($_aExceptionDetails, $_sExceptionClass) {
         $oClass = new \ReflectionClass($_sExceptionClass);
         $sExceptionCode = array_search($_aExceptionDetails, $oClass->getConstants());
-        if($sExceptionCode === false){
+        if ($sExceptionCode === false) {
             $sExceptionCode = "UNKNOWN";
         }
         return $sExceptionCode;
@@ -441,7 +441,7 @@ 
         //get function/method parameters and parameters function was called
         $ref = new \ReflectionMethod($this->sExceptionClass, $this->sExceptionFunction);
         $aFunctionParameters = $ref->getParameters();
-        $aFunctionParametersSetted = $this->getTrace()[0]['args'];
+        $aFunctionParametersSetted = $this->getTrace()[ 0 ][ 'args' ];
         //log notice if number of parameters does not match
         if (count($aFunctionParameters) < count($aFunctionParametersSetted)) {
             $this->innerNotice("MissUse", "Function was called with '" . count($aFunctionParametersSetted) . "' parameters, but defined on function is only '" . count($aFunctionParameters) . "'");
@@ -449,10 +449,10 @@ 
         //add parameters and values to exception to log
         $i = 0;
         foreach ($aFunctionParametersSetted as $mParameterValue) {
-            if (!isset($aFunctionParameters[$i])) {
+            if (!isset($aFunctionParameters[ $i ])) {
                 $this->gelfMessage->setAdditional("Param_UNKNOWN" . $i, serialize($mParameterValue));
             } else {
-                $this->gelfMessage->setAdditional("Param_" . $aFunctionParameters[$i]->name, serialize($mParameterValue));
+                $this->gelfMessage->setAdditional("Param_" . $aFunctionParameters[ $i ]->name, serialize($mParameterValue));
             }
             $i++;
         }