phptek / silverstripe-jsontext

code/models/fieldtypes/JSONText.php

Doc Comments +3 added lines, -3 removed lines

@@ -126,7 +126,7 @@ 
      * Tell all class methods to return data as JSON , an array or an array of SilverStripe DBField subtypes.
      * 
      * @param string $type
-     * @return \JSONText
+     * @return JSONText
      * @throws \JSONText\Exceptions\JSONTextException
      */
     public function setReturnType($type)
@@ -201,7 +201,7 @@ 
      * Convert an array to JSON via json_encode().
      * 
      * @param array $value
-     * @return mixed null|string
+     * @return string null|string
      */
     public function toJson(array $value)
     {
@@ -242,7 +242,7 @@ 
     }
 
     /**
-     * @param mixed $value
+     * @param string $value
      * @return array
      * @throws \JSONText\Exceptions\JSONTextException
      */

Spacing +2 added lines, -2 removed lines

@@ -297,7 +297,7 @@ 
             return $this->returnAsType([]);
         }
 
-        $count = count($data) -1;
+        $count = count($data)-1;
         $key = array_keys($data)[$count];
         $val = array_values($data)[$count];
 
@@ -403,7 +403,7 @@ 
                     return $result;
                 }
             }
-        } else if($expressionParamIsValid) {
+        } else if ($expressionParamIsValid) {
             $backendDBApiInst = \Injector::inst()->createWithArgs(
                 '\JSONText\Backends\\' . $dbBackend, [
                 $expression,

tests/JSONTextBasicTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function __construct()
     {
-        foreach($this->fixtures as $name => $path) {
+        foreach ($this->fixtures as $name => $path) {
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
         }
     }

tests/JSONTextQueryTest.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
      */
     public function __construct()
     {
-        foreach($this->fixtures as $name => $path) {
+        foreach ($this->fixtures as $name => $path) {
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
         }
     }
@@ -278,7 +278,7 @@ 
         $field->setReturnType('array');
         $field->setValue($this->getFixture('object'));
         $this->assertEquals(
-            [['Subaru' => 'Impreza'],['Kawasaki' => 'KR1S250']],
+            [['Subaru' => 'Impreza'], ['Kawasaki' => 'KR1S250']],
             $field->query('#>', '{"japanese":"fast"}')
         );
 

tests/JSONTextTest.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
  */
 
 use JSONText\Fields;
-use JSONText\Exceptions;
 
 class JSONTextTest extends SapphireTest
 {

tests/JSONTextSetValueTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function __construct()
     {
-        foreach($this->fixtures as $name => $path) {
+        foreach ($this->fixtures as $name => $path) {
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
         }
     }

code/backends/PostgresJSONBackend.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
 
         $keys = array_keys($operandAsArray);
         $vals = array_values($operandAsArray);
-        if (count($keys) >1 || count($vals) >1) {
+        if (count($keys) > 1 || count($vals) > 1) {
             $msg = 'Sorry. I can\'t handle complex operands.';
             throw new JSONTextException($msg);
         }