Passed
Push — master ( b4c5b9...e77e2d )
by Raffael
04:04
created
src/lib/Filesystem/Delta.php 1 patch
Doc Comments   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -70,9 +70,7 @@  discard block
 block discarded – undo
70 70
     /**
71 71
      * Add delta event.
72 72
      *
73
-     * @param string        $name
74 73
      * @param NodeInterface $node
75
-     * @param array         $options
76 74
      *
77 75
      * @return ObjectId
78 76
      */
@@ -359,7 +357,8 @@  discard block
 block discarded – undo
359 357
     /**
360 358
      * Get delta feed filter.
361 359
      *
362
-     * @param array $cursor
360
+     * @param integer[] $cursor
361
+     * @param NodeInterface|null $node
363 362
      *
364 363
      * @return array
365 364
      */
Please login to merge, or discard this patch.
src/lib/Hook/AutoCreateUser.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
         }
91 91
 
92 92
         $this->logger->info('found first time username ['.$identity->getIdentifier().'], auto-create user', [
93
-             'category' => get_class($this),
93
+                'category' => get_class($this),
94 94
         ]);
95 95
 
96 96
         $attributes = [];
Please login to merge, or discard this patch.
src/app/Balloon.App.Api/Controller.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
      * @param string $class      Force set node type
215 215
      * @param bool   $multiple   Allow $id to be an array
216 216
      * @param bool   $allow_root Allow instance of root collection
217
-     * @param bool   $deleted    How to handle deleted node
217
+     * @param integer   $deleted    How to handle deleted node
218 218
      *
219 219
      * @return NodeInterface
220 220
      */
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
      * @param string $id
249 249
      * @param string $path
250 250
      * @param string $class   Force set node type
251
-     * @param bool   $deleted How to handle deleted node
251
+     * @param integer   $deleted How to handle deleted node
252 252
      *
253 253
      * @return Generator
254 254
      */
Please login to merge, or discard this patch.
src/app/Balloon.App.Api/v2/Groups.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -302,7 +302,6 @@
 block discarded – undo
302 302
      * HTTP/1.1 200 OK
303 303
      *
304 304
      * @param string $id
305
-     * @param array  $attributes
306 305
      *
307 306
      * @return Response
308 307
      */
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
         }
271 271
 
272 272
         $attributes = compact('namespace', 'optional');
273
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
273
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
274 274
 
275 275
         $id = $this->server->addGroup($name, $member, $attributes);
276 276
         $result = $this->decorator->decorate($this->server->getGroupById($id));
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
     public function patch(string $id, ?array $member = null, ?string $namespace = null, ?array $optional = null): Response
310 310
     {
311 311
         $attributes = compact('namespace', 'optional', 'name', 'member');
312
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
312
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
313 313
 
314 314
         $group = $this->_getGroup($id, true);
315 315
         $group->setAttributes($attributes);
Please login to merge, or discard this patch.
src/app/Balloon.App.Api/v2/Users.php 2 patches
Doc Comments   -3 removed lines patch added patch discarded remove patch
@@ -158,8 +158,6 @@  discard block
 block discarded – undo
158 158
      *     "name": "peter.meier"
159 159
      * }
160 160
      *
161
-     * @param string $id
162
-     * @param string $uname
163 161
      *
164 162
      * @return Response
165 163
      */
@@ -429,7 +427,6 @@  discard block
 block discarded – undo
429 427
      *
430 428
      * @param string $uname
431 429
      * @param string $id
432
-     * @param array  $attributes
433 430
      *
434 431
      * @return Response
435 432
      */
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -388,7 +388,7 @@  discard block
 block discarded – undo
388 388
         }
389 389
 
390 390
         $attributes = compact('password', 'soft_quota', 'hard_quota', 'avatar', 'mail', 'admin', 'namespace', 'locale', 'optional');
391
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
391
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
392 392
 
393 393
         if (isset($attributes['avatar'])) {
394 394
             $attributes['avatar'] = new Binary(base64_decode($attributes['avatar']), Binary::TYPE_GENERIC);
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
     public function patch(?string $id = null, ?string $uname = null, ?string $username = null, ?string $password = null, ?int $soft_quota = null, ?int $hard_quota = null, ?string $avatar = null, ?string $mail = null, ?bool $admin = false, ?string $namespace = null, ?string $locale = null, ?array $optional = null): Response
437 437
     {
438 438
         $attributes = compact('username', 'password', 'soft_quota', 'hard_quota', 'avatar', 'mail', 'admin', 'namespace', 'locale', 'optional');
439
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
439
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
440 440
 
441 441
         if (isset($attributes['avatar'])) {
442 442
             $attributes['avatar'] = new Binary(base64_decode($attributes['avatar']), Binary::TYPE_GENERIC);
Please login to merge, or discard this patch.
src/app/Balloon.App.Elasticsearch/Api/v2/Search.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -107,7 +107,7 @@
 block discarded – undo
107 107
      * @param array $query
108 108
      * @param array $attributes
109 109
      * @param int   $deleted
110
-     * @param mixed $limit
110
+     * @param integer $limit
111 111
      *
112 112
      * @return Response
113 113
      */
Please login to merge, or discard this patch.
src/app/Balloon.App.Sharelink/Constructor/Http.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,6 @@
 block discarded – undo
50 50
      * @param Router             $router
51 51
      * @param Hook               $hook
52 52
      * @param Share              $sharelink
53
-     * @param AttributeDecorator $decorator
54 53
      * @param LoggerInterface    $logger
55 54
      */
56 55
     public function __construct(Router $router, Hook $hook, Share $sharelink, NodeAttributeDecorator $node_decorator_v2, NodeAttributeDecoratorv1 $node_decorator_v1, LoggerInterface $logger)
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
             }
73 73
         });
74 74
 
75
-        $node_decorator_v2->addDecorator('sharelink_token', function ($node) use ($sharelink) {
75
+        $node_decorator_v2->addDecorator('sharelink_token', function($node) use ($sharelink) {
76 76
             $attributes = $sharelink->getSharelink($node);
77 77
 
78 78
             if (isset($attributes['token'])) {
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
             }
81 81
         });
82 82
 
83
-        $node_decorator_v2->addDecorator('sharelink_expire', function ($node) use ($sharelink) {
83
+        $node_decorator_v2->addDecorator('sharelink_expire', function($node) use ($sharelink) {
84 84
             $attributes = $sharelink->getSharelink($node);
85 85
 
86 86
             if (isset($attributes['expiration'])) {
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
             }
91 91
         });
92 92
 
93
-        $node_decorator_v1->addDecorator('sharelink', function ($node) use ($sharelink) {
93
+        $node_decorator_v1->addDecorator('sharelink', function($node) use ($sharelink) {
94 94
             return isset($sharelink->getSharelink($node)['token']);
95 95
         });
96 96
 
Please login to merge, or discard this patch.
src/lib/Filesystem.php 1 patch
Doc Comments   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
     /**
182 182
      * Find raw node.
183 183
      *
184
-     * @param ObjectId $id
184
+     * @param ObjectId|null $id
185 185
      *
186 186
      * @return array
187 187
      */
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
      *
303 303
      * @param array  $id
304 304
      * @param string $class   Force check node type
305
-     * @param bool   $deleted
305
+     * @param integer   $deleted
306 306
      *
307 307
      * @return Generator
308 308
      */
@@ -431,7 +431,7 @@  discard block
 block discarded – undo
431 431
      * @param string $class      Force set node type
432 432
      * @param bool   $multiple   Allow $id to be an array
433 433
      * @param bool   $allow_root Allow instance of root collection
434
-     * @param bool   $deleted    How to handle deleted node
434
+     * @param integer   $deleted    How to handle deleted node
435 435
      *
436 436
      * @return NodeInterface
437 437
      */
@@ -493,7 +493,6 @@  discard block
 block discarded – undo
493 493
      * @param array $filter
494 494
      * @param int   $offset
495 495
      * @param int   $limit
496
-     * @param int   $total
497 496
      *
498 497
      * @return Generator
499 498
      */
Please login to merge, or discard this patch.
src/lib/Filesystem/Node/Collection.php 3 patches
Doc Comments   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
     /**
270 270
      * Get real id (reference).
271 271
      *
272
-     * @return ObjectId
272
+     * @return ObjectId|null
273 273
      */
274 274
     public function getRealId(): ?ObjectId
275 275
     {
@@ -298,7 +298,6 @@  discard block
 block discarded – undo
298 298
     /**
299 299
      * Fetch children items of this collection.
300 300
      *
301
-     * @param string $node
302 301
      * @param int    $deleted
303 302
      * @param array  $filter
304 303
      * @param mixed  $name
@@ -860,7 +859,7 @@  discard block
 block discarded – undo
860 859
      * @param string $name
861 860
      * @param string $data
862 861
      *
863
-     * @return File
862
+     * @return string
864 863
      */
865 864
     public function createFile($name, $data = null): String
866 865
     {
@@ -885,7 +884,6 @@  discard block
 block discarded – undo
885 884
      *
886 885
      * @param callable $callable
887 886
      * @param int      $deleted
888
-     * @param bool     $ignore_exception
889 887
      *
890 888
      * @return bool
891 889
      */
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,6 @@
 block discarded – undo
16 16
 use Balloon\Filesystem\Acl\Exception\Forbidden as ForbiddenException;
17 17
 use Balloon\Filesystem\Exception;
18 18
 use Balloon\Hook;
19
-use Balloon\Server\User;
20 19
 use Generator;
21 20
 use MongoDB\BSON\ObjectId;
22 21
 use MongoDB\BSON\Regex;
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
         $this->deleted = new UTCDateTime();
387 387
 
388 388
         if (!$this->isReference()) {
389
-            $this->doRecursiveAction(function ($node) use ($recursion) {
389
+            $this->doRecursiveAction(function($node) use ($recursion) {
390 390
                 $node->delete(false, $recursion, false);
391 391
             }, NodeInterface::DELETED_EXCLUDE);
392 392
         }
@@ -936,7 +936,7 @@  discard block
 block discarded – undo
936 936
     protected function _forceDelete(?string $recursion = null, bool $recursion_first = true): bool
937 937
     {
938 938
         if (!$this->isReference()) {
939
-            $this->doRecursiveAction(function ($node) use ($recursion) {
939
+            $this->doRecursiveAction(function($node) use ($recursion) {
940 940
                 $node->delete(true, $recursion, false);
941 941
             }, NodeInterface::DELETED_INCLUDE);
942 942
         }
Please login to merge, or discard this patch.