Completed
Push — master ( dd1654...09c137 )
by Russell
02:14
created
tests/JSONTextBasicTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      */
29 29
     public function __construct()
30 30
     {
31
-        foreach($this->fixtures as $name => $path) {
31
+        foreach ($this->fixtures as $name => $path) {
32 32
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
33 33
         }
34 34
     }
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
  */
8 8
 
9 9
 use JSONText\Fields;
10
-use JSONText\Exceptions;
11 10
 
12 11
 class JSONTextTest extends SapphireTest
13 12
 {
Please login to merge, or discard this patch.
tests/JSONTextQueryTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
      */
53 53
     public function __construct()
54 54
     {
55
-        foreach($this->fixtures as $name => $path) {
55
+        foreach ($this->fixtures as $name => $path) {
56 56
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
57 57
         }
58 58
     }
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
         $field->setReturnType('array');
279 279
         $field->setValue($this->getFixture('object'));
280 280
         $this->assertEquals(
281
-            [['Subaru' => 'Impreza'],['Kawasaki' => 'KR1S250']],
281
+            [['Subaru' => 'Impreza'], ['Kawasaki' => 'KR1S250']],
282 282
             $field->query('#>', '{"japanese":"fast"}')
283 283
         );
284 284
 
Please login to merge, or discard this patch.
tests/JSONTextSetValueTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      */
29 29
     public function __construct()
30 30
     {
31
-        foreach($this->fixtures as $name => $path) {
31
+        foreach ($this->fixtures as $name => $path) {
32 32
             $this->fixtures[$name] = MODULE_DIR . '/' . $path;
33 33
         }
34 34
     }
Please login to merge, or discard this patch.
code/models/fieldtypes/JSONText.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
             return $this->returnAsType([]);
298 298
         }
299 299
 
300
-        $count = count($data) -1;
300
+        $count = count($data)-1;
301 301
         $key = array_keys($data)[$count];
302 302
         $val = array_values($data)[$count];
303 303
 
@@ -403,7 +403,7 @@  discard block
 block discarded – undo
403 403
                     return $result;
404 404
                 }
405 405
             }
406
-        } else if($expressionParamIsValid) {
406
+        } else if ($expressionParamIsValid) {
407 407
             $backendDBApiInst = \Injector::inst()->createWithArgs(
408 408
                 '\JSONText\Backends\\' . $dbBackend, [
409 409
                 $expression,
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -249,7 +249,7 @@
 block discarded – undo
249 249
     }
250 250
 
251 251
     /**
252
-     * @param mixed $value
252
+     * @param string $value
253 253
      * @return array
254 254
      * @throws \JSONText\Exceptions\JSONTextException
255 255
      */
Please login to merge, or discard this patch.
tests/JSONTextIntegrationTest.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -6,8 +6,6 @@
 block discarded – undo
6 6
  * @author Russell Michell <[email protected]>
7 7
  */
8 8
 
9
-use JSONText\Exceptions;
10
-
11 9
 class JSONTextIntegrationTest extends SapphireTest
12 10
 {
13 11
     
Please login to merge, or discard this patch.