Completed
Push — master ( 9d18b4...301d75 )
by Russell
03:01
created
code/models/fieldtypes/JSONText.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
      * Tell all class methods to return data as JSON , an array or an array of SilverStripe DBField subtypes.
127 127
      * 
128 128
      * @param string $type
129
-     * @return \JSONText
129
+     * @return JSONText
130 130
      * @throws \JSONText\Exceptions\JSONTextException
131 131
      */
132 132
     public function setReturnType($type)
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
      * Convert an array to JSON via json_encode().
202 202
      * 
203 203
      * @param array $value
204
-     * @return mixed null|string
204
+     * @return string null|string
205 205
      */
206 206
     public function toJson(array $value)
207 207
     {
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
     }
243 243
 
244 244
     /**
245
-     * @param mixed $value
245
+     * @param string $value
246 246
      * @return array
247 247
      * @throws \JSONText\Exceptions\JSONTextException
248 248
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -290,7 +290,7 @@
 block discarded – undo
290 290
             return $this->returnAsType([]);
291 291
         }
292 292
 
293
-        $count = count($data) -1;
293
+        $count = count($data)-1;
294 294
         $key = array_keys($data)[$count];
295 295
         $val = array_values($data)[$count];
296 296
 
Please login to merge, or discard this patch.
code/models/backends/PostgresJSONBackend.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
 
80 80
         $keys = array_keys($operandAsArray);
81 81
         $vals = array_values($operandAsArray);
82
-        if (count($keys) >1 || count($vals) >1) {
82
+        if (count($keys) > 1 || count($vals) > 1) {
83 83
             $msg = 'Sorry. I can\'t handle complex operands.';
84 84
             throw new JSONTextException($msg);
85 85
         }
Please login to merge, or discard this patch.
tests/JSONTextBasicTest.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.
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.