fixed Json field

src/BaseRow.php

@@ -3,7 +3,7 @@
 
 /**
  * Base class used by Row and RowCollection
- * 
+ *
  * @property mixed $id
  */
 abstract class BaseRow implements RowInterface

src/Entity.php

@@ -63,7 +63,7 @@ protected function init()
 
     /**
      * Register a new field type
-     * 
+     *
      * @param string      $name
      * @param string|null $type
      */

src/Fields/Json.php

@@ -11,7 +11,11 @@ class Json extends Field
      */
     public function dataToDatabase($data)
     {
-        return json_encode($data);
+        if (!is_string($data)) {
+            return json_encode($data);
+        }
+
+        return $data;
     }
 
     /**

src/Queries/LimitTrait.php

@@ -3,7 +3,7 @@
 
 /**
  * Common function to manage LIMIT clause
- * 
+ *
  * @property \SimpleCrud\Entity $entity
  */
 trait LimitTrait

src/Queries/Sqlite/CompiledOptionsTrait.php

@@ -6,7 +6,7 @@
 
 /**
  * Manages a database select count query in Mysql databases
- * 
+ *
  * @property \SimpleCrud\Entity $entity
  */
 trait CompiledOptionsTrait

src/Queries/WhereExtendedTrait.php

@@ -5,7 +5,7 @@
 
 /**
  * Common function to manage WHERE clause
- * 
+ *
  * @property \SimpleCrud\Entity $entity
  */
 trait WhereExtendedTrait

src/Queries/WhereTrait.php

@@ -3,7 +3,7 @@
 
 /**
  * Common function to manage WHERE clause
- * 
+ *
  * @property \SimpleCrud\Entity $entity
  */
 trait WhereTrait

src/Row.php

@@ -41,7 +41,7 @@ public function __get($name)
         switch ($this->entity->getRelation($name)) {
             case Entity::RELATION_HAS_ONE:
                 return $this->values[$name] = $this->select($name)->one();
-            
+
             case Entity::RELATION_HAS_MANY:
             case Entity::RELATION_HAS_BRIDGE:
                 return $this->values[$name] = $this->select($name)->all();
@@ -205,7 +205,8 @@ public function save($duplications = false, $externalRelations = false)
      *
      * @return $this
      */
-    protected function saveExternalRelations() {
+    protected function saveExternalRelations()
+    {
         $extData = array_diff_key($this->values, $this->entity->fields);
         $db = $this->entity->getDb();
 
@@ -225,7 +226,7 @@ protected function saveExternalRelations() {
             if ($this->entity->hasMany($entity)) {
                 $entity->update()
                     ->data([
-                        $this->entity->foreignKey => $this->id
+                        $this->entity->foreignKey => $this->id,
                     ])
                     ->by('id', $ids)
                     ->run();
@@ -244,7 +245,7 @@ protected function saveExternalRelations() {
                     $bridge->insert()
                         ->data([
                             $this->entity->foreignKey => $this->id,
-                            $entity->foreignKey => $id
+                            $entity->foreignKey => $id,
                         ])
                         ->run();
                 }

tests/RowTest.php

@@ -47,7 +47,7 @@ public function testPost()
     public function testCustomFunction()
     {
         $post = $this->db->post->create([
-            'title' => 'THIS IS THE TITLE'
+            'title' => 'THIS IS THE TITLE',
         ])->save();
 
         $this->assertSame('this is the title', $post->getTitleLowerCase());
@@ -58,7 +58,7 @@ public function testCustomFunction()
     }
 }
 
-class DefaultEntity extends Entity 
+class DefaultEntity extends Entity
 {
     protected function init()
     {
@@ -70,8 +70,8 @@ protected function init()
                 return $row->getTitleLowerCase();
             });
 
-        $this->collection->registerMethod('sumIds', function($collection) {
+        $this->collection->registerMethod('sumIds', function ($collection) {
             return array_sum($collection->id);
         });
     }
-}
+}