Completed
Pull Request — master (#2)
by Emily
02:14
created
src/Service/ConditionalPropertyAccessor.php 1 patch
Spacing   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -35,8 +35,7 @@  discard block
 block discarded – undo
35 35
         }
36 36
         else
37 37
         {
38
-            throw new PropertyNotWritableException
39
-            (
38
+            throw new PropertyNotWritableException(
40 39
                 get_class($this->object),
41 40
                 $property->name
42 41
             );
@@ -50,16 +49,14 @@  discard block
 block discarded – undo
50 49
     {
51 50
         if (!$this->reflect->properties->containsKey($property))
52 51
         {
53
-            throw new CannotReadPropertyException
54
-            (
52
+            throw new CannotReadPropertyException(
55 53
                 get_class($this->object),
56 54
                 $property
57 55
             );
58 56
         }
59 57
         elseif (!$this->reflect->properties[$property]->readable)
60 58
         {
61
-            throw new PropertyNotReadableException
62
-            (
59
+            throw new PropertyNotReadableException(
63 60
                 get_class($this->object),
64 61
                 $property
65 62
             );
Please login to merge, or discard this patch.
src/Service/PropertyAccessor.php 1 patch
Spacing   +11 added lines, -22 removed lines patch added patch discarded remove patch
@@ -65,8 +65,7 @@  discard block
 block discarded – undo
65 65
         {
66 66
             if (!isset($args[$i]))
67 67
             {
68
-                throw new MissingRequiredParameterException
69
-                (
68
+                throw new MissingRequiredParameterException(
70 69
                     get_class($this->object),
71 70
                     $property->name
72 71
                 );
@@ -144,14 +143,12 @@  discard block
 block discarded – undo
144 143
     {
145 144
         if (!$this->reflect->properties->containsKey($property))
146 145
         {
147
-            throw new CannotWritePropertyException
148
-            (
146
+            throw new CannotWritePropertyException(
149 147
                 get_class($this->object), $property
150 148
             );
151 149
         }
152 150
 
153
-        $this->setAnyValue
154
-        (
151
+        $this->setAnyValue(
155 152
             $this->reflect->properties[$property],
156 153
             $value
157 154
         );
@@ -198,8 +195,7 @@  discard block
 block discarded – undo
198 195
      * @param ReflectionProperty $property The property to set
199 196
      * @param mixed $value The value to set
200 197
      */
201
-    private function setNonNullValue
202
-    (
198
+    private function setNonNullValue(
203 199
         ReflectionProperty $property,
204 200
         $value
205 201
     )
@@ -248,8 +244,7 @@  discard block
 block discarded – undo
248 244
      * @param callable $cast Method to cast a value to the scalar data
249 245
      *     type
250 246
      */
251
-    private function setScalarValue
252
-    (
247
+    private function setScalarValue(
253 248
         ReflectionProperty $property,
254 249
         $value,
255 250
         string $name,
@@ -264,8 +259,7 @@  discard block
 block discarded – undo
264 259
         }
265 260
         elseif (is_object($value) && method_exists([$value, $method]))
266 261
         {
267
-            $this->setScalarValue
268
-            (
262
+            $this->setScalarValue(
269 263
                 $property,
270 264
                 $value->$method(),
271 265
                 $method,
@@ -284,8 +278,7 @@  discard block
 block discarded – undo
284 278
      * @param ReflectionProperty $property The property to set
285 279
      * @param mixed $value The value to set
286 280
      */
287
-    private function setObjectValue
288
-    (
281
+    private function setObjectValue(
289 282
         ReflectionProperty $property,
290 283
         $value
291 284
     )
@@ -296,8 +289,7 @@  discard block
 block discarded – undo
296 289
         }
297 290
         else
298 291
         {
299
-            $this->throwError
300
-            (
292
+            $this->throwError(
301 293
                 $property,
302 294
                 $property->type->classname,
303 295
                 $value
@@ -311,8 +303,7 @@  discard block
 block discarded – undo
311 303
      * @param ReflectionProperty $property The property to set
312 304
      * @param mixed The value to set
313 305
      */
314
-    private function setCollectionValue
315
-    (
306
+    private function setCollectionValue(
316 307
         ReflectionProperty $property,
317 308
         $value
318 309
     )
@@ -334,15 +325,13 @@  discard block
 block discarded – undo
334 325
      * @param string $expected The expected datatype
335 326
      * @param string $value The value being set
336 327
      */
337
-    private function throwError
338
-    (
328
+    private function throwError(
339 329
         ReflectionProperty $property,
340 330
         string $expected,
341 331
         $value
342 332
     )
343 333
     {
344
-        throw new IllegalPropertyTypeException
345
-        (
334
+        throw new IllegalPropertyTypeException(
346 335
             get_class($this->object),
347 336
             $property->name,
348 337
             $expected,
Please login to merge, or discard this patch.
src/Model/Collection/ArrayList.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -45,10 +45,9 @@
 block discarded – undo
45 45
     /**
46 46
      * {@inheritDoc}
47 47
      */
48
-    public function splice
49
-    (
48
+    public function splice(
50 49
         int $offset,
51
-        ?int $length = null,
50
+        ? int $length = null,
52 51
         array $replacement = []
53 52
     )
54 53
     {
Please login to merge, or discard this patch.
src/Model/Collection/ListInterface.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -33,10 +33,9 @@
 block discarded – undo
33 33
      * @param int $length The number of elements to remove
34 34
      * @param array $items Items to add
35 35
      */
36
-    public function splice
37
-    (
36
+    public function splice(
38 37
         int $offset,
39
-        ?int$length = null,
38
+        ? int$length = null,
40 39
         array $items = []
41 40
     );
42 41
 
Please login to merge, or discard this patch.
src/Model/Collection/HashMap.php 1 patch
Spacing   +3 added lines, -5 removed lines patch added patch discarded remove patch
@@ -41,8 +41,7 @@  discard block
 block discarded – undo
41 41
         return static::$defaultHashProducer;
42 42
     }
43 43
 
44
-    public static function setDefaultHashProducer
45
-    (
44
+    public static function setDefaultHashProducer(
46 45
         HashProducerInterface $hashProducer
47 46
     )
48 47
     : HashProducerInterface
@@ -60,9 +59,8 @@  discard block
 block discarded – undo
60 59
      */
61 60
     protected $hashProducer;
62 61
 
63
-    public function __construct
64
-    (
65
-        ?HashProducerInterface $hashProducer = null
62
+    public function __construct(
63
+        ? HashProducerInterface $hashProducer = null
66 64
     )
67 65
     {
68 66
         $this->hashProducer =
Please login to merge, or discard this patch.
src/Model/Collection/FixedList.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -45,10 +45,9 @@
 block discarded – undo
45 45
     /**
46 46
      * {@inheritDoc}
47 47
      */
48
-    public function splice
49
-    (
48
+    public function splice(
50 49
         int $offset,
51
-        ?int $length = null,
50
+        ? int $length = null,
52 51
         array $replacement = []
53 52
     )
54 53
     {
Please login to merge, or discard this patch.
src/Service/ReflectionCompositeProvider.php 1 patch
Spacing   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -39,8 +39,7 @@  discard block
 block discarded – undo
39 39
         return static::$default;
40 40
     }
41 41
 
42
-    public static function setDefault
43
-    (
42
+    public static function setDefault(
44 43
         ReflectionCompositeProviderInterface $default
45 44
     )
46 45
     {
@@ -60,8 +59,7 @@  discard block
 block discarded – undo
60 59
         {
61 60
             $this->cache[$classname] =
62 61
                 (
63
-                    ReflectionCompositeFactory::fromClassName
64
-                    (
62
+                    ReflectionCompositeFactory::fromClassName(
65 63
                         $classname
66 64
                     )
67 65
                 )
Please login to merge, or discard this patch.
src/Factory/Reflection/ReflectionCompositeFactory.php 1 patch
Spacing   +9 added lines, -18 removed lines patch added patch discarded remove patch
@@ -78,8 +78,7 @@  discard block
 block discarded – undo
78 78
      * @param PHPNativeReflectionClass $reflect
79 79
      * @param ReflectionCompositeProviderInterface $provider
80 80
      */
81
-    public function __construct
82
-    (
81
+    public function __construct(
83 82
         PHPNativeReflectionClass $reflect,
84 83
         ReflectionCompositeProviderInterface $provider
85 84
     )
@@ -117,13 +116,11 @@  discard block
 block discarded – undo
117 116
         $file = (new ReflectionFileFactory($fileName))->build();
118 117
         $this->accessor->setRawValue('file', $file);
119 118
 
120
-        $this->accessor->setRawValue
121
-        (
119
+        $this->accessor->setRawValue(
122 120
             'classname',
123 121
             $this->reflector->name
124 122
         );
125
-        $this->accessor->setRawValue
126
-        (
123
+        $this->accessor->setRawValue(
127 124
             'namespace',
128 125
             $file->namespaces[$this->reflector->getNamespaceName()]
129 126
         );
@@ -146,8 +143,7 @@  discard block
 block discarded – undo
146 143
             $size = count($this->reflector->{'get' . $name}());
147 144
             foreach ($prefixes as $prefix)
148 145
             {
149
-                $this->accessor->setRawValue
150
-                (
146
+                $this->accessor->setRawValue(
151 147
                     $prefix . $name,
152 148
                     new FixedList($size)
153 149
                 );
@@ -177,8 +173,7 @@  discard block
 block discarded – undo
177 173
      * @param bool $checkFile
178 174
      * @param string $singular
179 175
      */
180
-    protected function addItems
181
-    (
176
+    protected function addItems(
182 177
         string $name,
183 178
         bool $checkFile,
184 179
         string $signular
@@ -209,8 +204,7 @@  discard block
 block discarded – undo
209 204
                     new $factory($item),
210 205
                     $item
211 206
                 );
212
-                $this->accessor->rawAddToValue
213
-                (
207
+                $this->accessor->rawAddToValue(
214 208
                     'local' . ucfirst($name),
215 209
                     $item
216 210
                 );
@@ -227,8 +221,7 @@  discard block
 block discarded – undo
227 221
      * @param PHPNativeReflectionClass $reflect
228 222
      * @param string $method
229 223
      */
230
-    protected function addInheritance
231
-    (
224
+    protected function addInheritance(
232 225
         string $group,
233 226
         PHPNativeReflectionClass $reflect,
234 227
         string $method = 'rawAddToValue'
@@ -248,15 +241,13 @@  discard block
 block discarded – undo
248 241
      * @param ReflectionPropertyFactory $factory
249 242
      * @return ReflectionProperty
250 243
      */
251
-    protected function buildProperty
252
-    (
244
+    protected function buildProperty(
253 245
         ReflectionPropertyFactory $factory,
254 246
         PHPNativeReflectionProperty $reflect
255 247
     )
256 248
     : ReflectionProperty
257 249
     {
258
-        return $factory->build
259
-        (
250
+        return $factory->build(
260 251
             $this->object,
261 252
             $this->reflector
262 253
                 ->getDefaultProperties()[$reflect->getName()]
Please login to merge, or discard this patch.