Test Failed
Branch v2.5 (2efee3)
by Raffael
24:08
created
src/app/Balloon.App.Office/Session.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@
 block discarded – undo
137 137
      * Get Session.
138 138
      *
139 139
      *
140
-     * @return Session
140
+     * @return Session|null
141 141
      */
142 142
     public static function getByAccessToken(Server $server, ObjectId $session_id, string $access_token): self
143 143
     {
Please login to merge, or discard this patch.
src/lib/Migration/Delta/Md5BlobIgnoreNull.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@
 block discarded – undo
12 12
 namespace Balloon\Migration\Delta;
13 13
 
14 14
 use MongoDB\Database;
15
-use MongoDB\Driver\Exception\CommandException;
16 15
 use MongoDB\Driver\Exception\RuntimeException;
17 16
 
18 17
 class Md5BlobIgnoreNull implements DeltaInterface
Please login to merge, or discard this patch.
src/lib/Filesystem/Acl.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -236,8 +236,8 @@  discard block
 block discarded – undo
236 236
             $share = $node->getFilesystem()->findRawNode($node->getShareId());
237 237
         } catch (\Exception $e) {
238 238
             $this->logger->error('could not find share node ['.$node->getShareId().'] for reference ['.$node->getId().'], dead reference?', [
239
-                 'category' => get_class($this),
240
-                 'exception' => $e,
239
+                    'category' => get_class($this),
240
+                    'exception' => $e,
241 241
             ]);
242 242
 
243 243
             return self::PRIVILEGE_DENY;
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
 
246 246
         if ($share['deleted'] instanceof UTCDateTime || true !== $share['shared']) {
247 247
             $this->logger->error('share node ['.$share['_id'].'] has been deleted, dead reference?', [
248
-                 'category' => get_class($this),
248
+                    'category' => get_class($this),
249 249
             ]);
250 250
 
251 251
             return self::PRIVILEGE_DENY;
Please login to merge, or discard this patch.
src/app/Balloon.App.Notification/Notifier.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
 
349 349
             if ($node instanceof Collection && $recursive === true) {
350 350
                 $db = $this->db;
351
-                $node->doRecursiveAction(function ($child) use ($db, $subscription) {
351
+                $node->doRecursiveAction(function($child) use ($db, $subscription) {
352 352
                     $subscription['node'] = $child->getId();
353 353
                     $db->subscription->replaceOne(
354 354
                         [
@@ -374,7 +374,7 @@  discard block
 block discarded – undo
374 374
 
375 375
             if ($node instanceof Collection && $recursive === true) {
376 376
                 $db = $this->db;
377
-                $node->doRecursiveAction(function ($child) use ($db, $node_id, $user_id) {
377
+                $node->doRecursiveAction(function($child) use ($db, $node_id, $user_id) {
378 378
                     $db->subscription->deleteOne([
379 379
                         'user' => $user_id,
380 380
                         'node' => $node_id,
Please login to merge, or discard this patch.
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -93,6 +93,7 @@  discard block
 block discarded – undo
93 93
 
94 94
     /**
95 95
      * Send notification.
96
+     * @param User $sender
96 97
      */
97 98
     public function notify(Iterable $receiver, ?User $sender, MessageInterface $message): bool
98 99
     {
@@ -180,6 +181,7 @@  discard block
 block discarded – undo
180 181
 
181 182
     /**
182 183
      * Add notification.
184
+     * @param User $sender
183 185
      */
184 186
     public function postNotification(User $receiver, ?User $sender, MessageInterface $message): ObjectId
185 187
     {
@@ -206,6 +208,8 @@  discard block
 block discarded – undo
206 208
 
207 209
     /**
208 210
      * Get notifications.
211
+     * @param integer $offset
212
+     * @param integer $limit
209 213
      */
210 214
     public function getNotifications(User $user, ?int $offset = null, ?int $limit = null, ?int &$total = null): Iterable
211 215
     {
@@ -278,6 +282,7 @@  discard block
 block discarded – undo
278 282
 
279 283
     /**
280 284
      * Get subscription.
285
+     * @return NodeInterface
281 286
      */
282 287
     public function getSubscription(NodeInterface $node, User $user): ?array
283 288
     {
Please login to merge, or discard this patch.
src/app/Balloon.App.Api/v1/File.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
      *
118 118
      * @param string $id
119 119
      * @param string $p
120
-     * @param string $version
120
+     * @param integer $version
121 121
      */
122 122
     public function postRestore(int $version, ?string $id = null, ?string $p = null): Response
123 123
     {
@@ -406,6 +406,9 @@  discard block
 block discarded – undo
406 406
 
407 407
     /**
408 408
      * Get storage.
409
+     * @param string|null $id
410
+     * @param string|null $p
411
+     * @param string|null $collection
409 412
      */
410 413
     protected function getStorage($id, $p, $collection): StorageAdapterInterface
411 414
     {
Please login to merge, or discard this patch.
src/app/Balloon.App.Notification/AttributeDecorator.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
             'message' => (string) $message['body'],
99 99
             'subject' => (string) $message['subject'],
100 100
             'locale' => (string) $message['locale'],
101
-            'sender' => function ($message) use ($server, $role_decorator) {
101
+            'sender' => function($message) use ($server, $role_decorator) {
102 102
                 if (!isset($message['sender'])) {
103 103
                     return null;
104 104
                 }
@@ -109,10 +109,10 @@  discard block
 block discarded – undo
109 109
                     return null;
110 110
                 }
111 111
             },
112
-            'created' => function ($message) {
112
+            'created' => function($message) {
113 113
                 return (new DateTime())->setTimestamp($message['_id']->getTimestamp())->format('c');
114 114
             },
115
-            'node' => function ($message) use ($node_decorator, $fs) {
115
+            'node' => function($message) use ($node_decorator, $fs) {
116 116
                 if (isset($message['node'])) {
117 117
                     try {
118 118
                         return $node_decorator->decorate($fs->findNodeById($message['node']), ['id', 'name', '_links']);
Please login to merge, or discard this patch.
src/app/Balloon.App.Office/Api/v2/Documents.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -161,7 +161,7 @@
 block discarded – undo
161 161
         $tpl = new Template($type);
162 162
 
163 163
         $attributes = compact('readonly', 'meta');
164
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
164
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
165 165
 
166 166
         $stream = $tpl->get();
167 167
         $storage = $parent->getStorage();
Please login to merge, or discard this patch.
src/lib/Async/WorkerFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
      */
55 55
     protected function setErrorHandler(LoggerInterface $logger): self
56 56
     {
57
-        set_error_handler(function ($severity, $message, $file, $line) use ($logger) {
57
+        set_error_handler(function($severity, $message, $file, $line) use ($logger) {
58 58
             $log = $message.' in '.$file.':'.$line;
59 59
 
60 60
             switch ($severity) {
Please login to merge, or discard this patch.
src/lib/Async/SmbScanner.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@
 block discarded – undo
144 144
     protected function deleteNode(NodeInterface $node, Blackhole $dummy): bool
145 145
     {
146 146
         if ($node instanceof Collection) {
147
-            $node->doRecursiveAction(function (NodeInterface $node) use ($dummy) {
147
+            $node->doRecursiveAction(function(NodeInterface $node) use ($dummy) {
148 148
                 if ($node instanceof Collection) {
149 149
                     $node->setStorage($dummy);
150 150
                 }
Please login to merge, or discard this patch.