Merge pull request #29 from Manriel/patch-1

Wrong format of exception in log context fix

Author: danielme85

src/LogToDbCustomLoggingHandler.php

@@ -55,7 +55,7 @@ function __construct(array $config,
     protected function write(array $record): void
     {
         if (!empty($record)) {
-            if (!empty($record['context']['exception']) &&
+            if (!empty($record['context']['exception']) && is_object($record['context']['exception']) &&
                 get_class($record['context']['exception']) === DBLogException::class) {
                 //Do nothing if empty log record or an error Exception from itself.
             } else {
@@ -72,4 +72,4 @@ protected function write(array $record): void
             }
         }
     }
-}
+}

tests/LogToDbTest.php

@@ -245,6 +245,25 @@ public function testException()
         $this->expectException(DBLogException::class);
         throw new DBLogException('Dont log this');
     }
+    
+    /**
+     * Test exception when expected format is wrong.
+     *
+     * @group exception
+     */
+    public function testExceptionWrongFormat()
+    {
+        $e = [
+            'message' => 'Array instead exception',
+            'code'    => 0,
+            'file'    => __FILE__,
+            'line'    => __LINE__,
+            'trace'   => debug_backtrace(),
+        ];
+        Log::warning("Error", ['exception' => $e, 'more' => 'infohere']);
+        $log = LogToDB::model()->where('message', 'Error')->first();
+        $this->assertNotEmpty($log->context);
+    }
 
     /**
      *
@@ -564,4 +583,4 @@ public function testFinalCleanup()
 
         $this->artisan('migrate:rollback', ['--database' => 'mysql']);
     }
-}
+}