Fix issue where context array was only json encoded in cases where it contained an exception.

Moved json encoding to the model layer and should now run on any user or system supplied array as should be expected behavior.

Author: danielme85

src/LogToDB.php

@@ -197,7 +197,7 @@ public function getConfig(string $config)
      * Parse the exception class
      *
      * @param mixed $context
-     * @return mixed
+     * @return array
      */
     private function parseIfException($context)
     {
@@ -237,11 +237,7 @@ private function parseIfException($context)
             }
         }
 
-        if (!empty($context)) {
-            return json_encode($context);
-        }
-
-        return null;
+        return $context;
     }
 
 }

src/Models/LogToDbCreateObject.php

@@ -76,6 +76,16 @@ public function setDatetimeAttribute(object $value)
         $this->attributes['datetime'] = $value->format(config('logtodb.datetime_format'));
     }
 
+    /**
+     * Context Mutator
+     *
+     * @param array $value
+     */
+    public function setContextAttribute($value)
+    {
+        $this->attributes['context'] = $this->jsonEncodeIfNotEmpty($value);
+    }
+
     /**
      * Extra Mutator
      *

tests/LogToDbTest.php

@@ -182,6 +182,18 @@ public function testMessageLoggedEvent()
         Log::debug("This is to trigger a log event.");
     }
 
+    /**
+     * @group context
+     */
+    public function testContext()
+    {
+        Log::error("Im trying to add some context", ['whatDis?' => 'dis some context, should always be array']);
+        $log = LogToDB::model()->where('message', '=' , 'Im trying to add some context')->first();
+        $this->assertNotEmpty($log);
+        $this->assertStringContainsString("Im trying to add some context", $log->message);
+        $this->assertIsArray($log->context);
+    }
+
     /**
      * Check to see if processors are adding extra content.
      *