understand / understand-laravel5

src/Understand/UnderstandLaravel5/ExceptionEncoder.php

Doc Comments +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param $relativePath
+     * @param string|null $relativePath
      * @param $line
      * @param int $linesAround
      * @return array|void
@@ -217,7 +217,7 @@ 
      * Return stack trace file
      *
      * @param array $trace
-     * @return mixed
+     * @return string|null
      */
     protected function getStackTraceFile(array $trace)
     {

Braces +10 added lines, -20 removed lines

@@ -120,8 +120,7 @@ 
             if (strpos($trace['file'], $this->projectRoot . 'vendor' . DIRECTORY_SEPARATOR) === 0)
             {
                 unset($stackTrace[$index]);
-            }
-            else
+            } else
             {
                 $vendorExcluded = true;
             }
@@ -215,10 +214,8 @@ 
             }
 
             return $codeLines;
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
     }
 
@@ -294,32 +291,25 @@ 
             if (is_array($arg))
             {
                 $params[] = 'array(' . count($arg) . ')';
-            }
-            else if (is_object($arg))
+            } else if (is_object($arg))
             {
                 $params[] = get_class($arg);
-            }
-            else if (is_string($arg))
+            } else if (is_string($arg))
             {
                 $params[] = 'string(' . $arg . ')';
-            }
-            else if (is_int($arg))
+            } else if (is_int($arg))
             {
                 $params[] = 'int(' . $arg . ')';
-            }
-            else if (is_float($arg))
+            } else if (is_float($arg))
             {
                 $params[] = 'float(' . $arg . ')';
-            }
-            else if (is_bool($arg))
+            } else if (is_bool($arg))
             {
                 $params[] = 'bool(' . ($arg ? 'true' : 'false') . ')';
-            }
-            else if ($arg instanceof \__PHP_Incomplete_Class)
+            } else if ($arg instanceof \__PHP_Incomplete_Class)
             {
                 $params[] = 'object(__PHP_Incomplete_Class)'; 
-            }
-            else
+            } else
             {
                 $params[] = gettype($arg);
             }

src/Understand/UnderstandLaravel5/FieldProvider.php

Doc Comments +3 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
     protected $app;
 
     /**
-     * @return void
+     * @return Application
      */
     public function __construct()
     {
@@ -108,7 +108,7 @@ 
     /**
      * Set session store
      *
-     * @param type $service
+     * @param SessionStore $service
      */
     public function setSessionStore(SessionStore $service)
     {
@@ -157,7 +157,6 @@ 
      * Register a custom HTML macro.
      *
      * @param string $name
-     * @param  mixed  $macro
      * @return void
      */
     public function extend($name, $provider)
@@ -168,7 +167,7 @@ 
     /**
      * Set token provider
      *
-     * @param UniqueProcessIdentifier $tokenProvider
+     * @param TokenProvider $tokenProvider
      */
     public function setTokenProvider(TokenProvider $tokenProvider)
     {

Braces +14 added lines, -28 removed lines

@@ -266,8 +266,7 @@ 
         {
             // Laravel
             return Application::VERSION;
-        }
-        else
+        } else
         {
             $re = '/Lumen \((.*)\) \(.*\)/m';
 
@@ -327,17 +326,14 @@ 
                 if ($value instanceof \DateTimeInterface)
                 {
                     $binding = $value->format('Y-m-d H:i:s');
-                }
-                elseif (is_bool($value))
+                } elseif (is_bool($value))
                 {
                     $binding = (int) $value;
-                }
-                else
+                } else
                 {
                     $binding = (string)$value;
                 }
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $binding = '[handler error]';
             }
@@ -414,8 +410,7 @@ 
             try
             {
                 $queryString[$key] = $this->parseRequestFieldValue($key, $value);
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $queryString[$key] = '[handler error]';
             }
@@ -448,12 +443,10 @@ 
         if (method_exists($this->request, 'json') && method_exists($this->request, 'isJson') && $this->request->isJson())
         {
             $source = $this->request->json();
-        }
-        else if ($this->request->request instanceof \IteratorAggregate)
+        } else if ($this->request->request instanceof \IteratorAggregate)
         {
             $source = $this->request->request;
-        }
-        else
+        } else
         {
             return;
         }
@@ -470,8 +463,7 @@ 
             try
             {
                 $postData[$key] = $this->parseRequestFieldValue($key, $value);
-            }
-            catch (\Exception $e)
+            } catch (\Exception $e)
             {
                 $postData[$key] = '[handler error]';
             }
@@ -640,10 +632,8 @@ 
             {
                 return $userId;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
 
         try
@@ -652,10 +642,8 @@ 
             {
                 return $user->id;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
 
         try
@@ -664,10 +652,8 @@ 
             {
                 return $user->id;
             }
-        }
-        catch (\Throwable $e)
-        {}
-        catch (\Exception $e)
+        } catch (\Throwable $e)
+        {} catch (\Exception $e)
         {}
     }
 

src/Understand/UnderstandLaravel5/Logger.php

Doc Comments -2 removed lines

@@ -28,7 +28,6 @@ 
     /**
      * @param FieldProvider $fieldProvider
      * @param BaseHandler $handler
-     * @param bool $silent
      */
     public function __construct(FieldProvider $fieldProvider, BaseHandler $handler)
     {
@@ -144,7 +143,6 @@ 
     /**
      * Send data to storage
      *
-     * @param array $requestData
      * @return mixed
      */
     protected function send(array $event)

Braces +2 added lines, -4 removed lines

@@ -152,12 +152,10 @@
         try
         {
             return $this->handler->handle($event);
-        }
-        catch (\Throwable $e)
+        } catch (\Throwable $e)
         {
             return false;
-        }
-        catch (\Exception $ex)
+        } catch (\Exception $ex)
         {
             return false;
         }

src/Understand/UnderstandLaravel5/ExceptionLogger.php

Braces +1 added lines, -2 removed lines

@@ -90,8 +90,7 @@
             ];
 
             $log = $this->encoder->setCurrentStackTrace($log);
-        }
-        else
+        } else
         {
             $log = (array)$message;
             $log = $this->encoder->setCurrentStackTrace($log);

src/Understand/UnderstandLaravel5/EventLogger.php

Braces +1 added lines, -2 removed lines

@@ -45,8 +45,7 @@
             $log = [
                 'message' => $message
             ];
-        }
-        else
+        } else
         {
             $log = (array)$message;
         }

src/Understand/UnderstandLaravel5/DataCollector.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     }
 
     /**
-     * @param $key
+     * @param string $key
      * @return mixed
      */
     public function getByKey($key)

src/Understand/UnderstandLaravel5/UnderstandLaravel5ServiceProvider.php

Braces +4 added lines, -8 removed lines

@@ -237,8 +237,7 @@ 
             {
                 $this->handleEvent($level, $message, $context);
             });
-        }
-        else
+        } else
         {
             // starting from L5.4 MessageLogged event class was introduced
             // https://github.com/laravel/framework/commit/57c82d095c356a0fe0f9381536afec768cdcc072
@@ -266,8 +265,7 @@ 
                     $this->app['understand.tokenProvider']->generate();
                     $this->app['understand.dataCollector']->reset();
                 });
-            }
-            else
+            } else
             {
                 $this->app['events']->listen('Illuminate\Queue\Events\JobProcessing', function()
                 {
@@ -297,8 +295,7 @@ 
                     'time' => $time,
                 ]);
             });
-        }
-        else
+        } else
         {
             // https://laravel.com/api/5.3/Illuminate/Database/Events/QueryExecuted.html
             $this->app['events']->listen('Illuminate\Database\Events\QueryExecuted', function($event)
@@ -340,8 +337,7 @@ 
             unset($context['exception']);
 
             $this->app['understand.exceptionLogger']->logError($level, $exception, $context);
-        }
-        else
+        } else
         {
             $this->app['understand.exceptionLogger']->logError($level, $message, $context);
         }

Doc Comments +2 added lines, -2 removed lines

@@ -122,8 +122,8 @@
     /**
      * Detect Laravel version
      * 
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLaravelVersion(array $versions)
     {

src/Understand/UnderstandLaravel5/UnderstandLumenServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -129,8 +129,8 @@
     /**
      * Detect Lumen version
      *
-     * @param array $versions
-     * @return type
+     * @param string[] $versions
+     * @return boolean
      */
     protected function detectLumenVersion(array $versions)
     {

Braces +3 added lines, -6 removed lines

@@ -161,8 +161,7 @@ 
             {
                 $this->handleEvent($log['level'], $log['message'], $log['context']);
             });
-        }
-        else
+        } else
         {
             // starting from Lumen 5.6 MessageLogged event class was introduced
             $this->app['events']->listen('Illuminate\Log\Events\MessageLogged', function ($log)
@@ -211,8 +210,7 @@ 
                     'time' => $time,
                 ]);
             });
-        }
-        else
+        } else
         {
             // https://laravel.com/api/5.3/Illuminate/Database/Events/QueryExecuted.html
             $this->app['events']->listen('Illuminate\Database\Events\QueryExecuted', function($event)
@@ -253,8 +251,7 @@ 
             unset($context['exception']);
 
             $this->app['understand.exceptionLogger']->logError($level, $exception, $context);
-        }
-        else
+        } else
         {
             $this->app['understand.exceptionLogger']->logError($level, $message, $context);
         }

src/Understand/UnderstandLaravel5/UnderstandServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
     }
 
     /**
-     * @param $level
+     * @param string $level
      * @param $message
      * @param $context
      * @return bool
@@ -136,7 +136,7 @@ 
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {