phptek / silverstripe-jsontext

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;
         }
     }

Unused Use Statements -1 removed lines

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

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/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;
         }
     }

tests/JSONTextIntegrationTest.php

Unused Use Statements -2 removed lines

@@ -6,8 +6,6 @@
  * @author Russell Michell <russ@theruss.com>
  */
 
-use JSONText\Exceptions;
-
 class JSONTextIntegrationTest extends SapphireTest
 {
     

tests/JSONTextTest.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/models/fieldtypes/JSONText.php

Doc Comments +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
     }
 
     /**
-     * @param mixed $value
+     * @param string $value
      * @return array
      * @throws \JSONText\Exceptions\JSONTextDataException
      */

Spacing +2 added lines, -2 removed lines

@@ -288,7 +288,7 @@ 
             return $this->returnAsType([]);
         }
 
-        $count = count($data) -1;
+        $count = count($data)-1;
         $key = array_keys($data)[$count];
         $val = array_values($data)[$count];
 
@@ -389,7 +389,7 @@ 
                     return $result;
                 }
             }
-        } else if($expressionParamIsValid) {
+        } else if ($expressionParamIsValid) {
             $dbBackendInst = $this->createBackendInst($expression);
             if ($result = $dbBackendInst->matchOnExpr()) {
                 return $result;

code/extensions/JSONTextExtension.php

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      * 
      * @return null
      */
-	public function onBeforeWrite()
+    public function onBeforeWrite()
     {
         parent::onBeforeWrite();
         

tests/JSONTextExtensionTest.php

Indentation +3 added lines, -3 removed lines

@@ -18,9 +18,9 @@
      * 
      * @var array
      */
-	protected $extraDataObjects = [
-		'JSONTextTestPage',
-	];
+    protected $extraDataObjects = [
+        'JSONTextTestPage',
+    ];
     
     /**
      * Is an exception thrown when no POSTed vars are available for