@@ -16,7 +16,6 @@ |
||
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; |
@@ -386,7 +386,7 @@ discard block |
||
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 |
||
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 | } |
@@ -255,7 +255,7 @@ discard block |
||
255 | 255 | /** |
256 | 256 | * Get real id (reference). |
257 | 257 | * |
258 | - * @return ObjectId |
|
258 | + * @return ObjectId|null |
|
259 | 259 | */ |
260 | 260 | public function getRealId(): ?ObjectId |
261 | 261 | { |
@@ -752,7 +752,7 @@ discard block |
||
752 | 752 | * @param string $name |
753 | 753 | * @param string $data |
754 | 754 | * |
755 | - * @return File |
|
755 | + * @return string |
|
756 | 756 | */ |
757 | 757 | public function createFile($name, $data = null): String |
758 | 758 | { |
@@ -265,6 +265,8 @@ |
||
265 | 265 | /** |
266 | 266 | * Get resolved member. |
267 | 267 | * |
268 | + * @param integer $offset |
|
269 | + * @param integer $limit |
|
268 | 270 | * @return Generator |
269 | 271 | */ |
270 | 272 | public function getResolvedMembers(?int $offset = null, ?int $limit = null): ?Generator |
@@ -531,7 +531,7 @@ discard block |
||
531 | 531 | |
532 | 532 | $this->identity = $user; |
533 | 533 | $user->updateIdentity($identity) |
534 | - ->updateShares(); |
|
534 | + ->updateShares(); |
|
535 | 535 | $this->hook->run('postServerIdentity', [$user]); |
536 | 536 | |
537 | 537 | return true; |
@@ -662,7 +662,7 @@ discard block |
||
662 | 662 | public function getGroupByName(string $name): Group |
663 | 663 | { |
664 | 664 | $group = $this->db->group->findOne([ |
665 | - 'name' => $name, |
|
665 | + 'name' => $name, |
|
666 | 666 | ]); |
667 | 667 | |
668 | 668 | if (null === $group) { |
@@ -682,7 +682,7 @@ discard block |
||
682 | 682 | public function getGroupById(ObjectId $id): Group |
683 | 683 | { |
684 | 684 | $group = $this->db->group->findOne([ |
685 | - '_id' => $id, |
|
685 | + '_id' => $id, |
|
686 | 686 | ]); |
687 | 687 | |
688 | 688 | if (null === $group) { |
@@ -195,6 +195,7 @@ |
||
195 | 195 | |
196 | 196 | /** |
197 | 197 | * Filesystem factory. |
198 | + * @param User $user |
|
198 | 199 | */ |
199 | 200 | public function getFilesystem(?User $user = null): Filesystem |
200 | 201 | { |
@@ -221,10 +221,10 @@ |
||
221 | 221 | { |
222 | 222 | $mail = new Message(); |
223 | 223 | $mail->setBody($body) |
224 | - ->setFrom($this->user->getAttributes()['mail'], $this->user->getAttributes()['username']) |
|
225 | - ->setSubject($subject) |
|
226 | - ->setTo($this->user->getAttributes()['mail'], 'Undisclosed Recipients') |
|
227 | - ->setBcc($receiver); |
|
224 | + ->setFrom($this->user->getAttributes()['mail'], $this->user->getAttributes()['username']) |
|
225 | + ->setSubject($subject) |
|
226 | + ->setTo($this->user->getAttributes()['mail'], 'Undisclosed Recipients') |
|
227 | + ->setBcc($receiver); |
|
228 | 228 | $this->async->addJob(Mail::class, $mail->toString()); |
229 | 229 | |
230 | 230 | return (new Response())->setCode(202); |
@@ -49,8 +49,8 @@ |
||
49 | 49 | $this->db->storage->updateOne( |
50 | 50 | ['_id' => $object['_id']], |
51 | 51 | [ |
52 | - '$unset' => ['sharelink' => 1], |
|
53 | - '$set' => ['app.Balloon\App\Sharelink' => $object['sharelink']], |
|
52 | + '$unset' => ['sharelink' => 1], |
|
53 | + '$set' => ['app.Balloon\App\Sharelink' => $object['sharelink']], |
|
54 | 54 | ] |
55 | 55 | ); |
56 | 56 | } |
@@ -341,7 +341,7 @@ |
||
341 | 341 | } |
342 | 342 | |
343 | 343 | $this->logger->debug('elasticsarch blob document ['.$result['_id'].'] has no references left, remove completely', [ |
344 | - 'category' => get_class($this), |
|
344 | + 'category' => get_class($this), |
|
345 | 345 | ]); |
346 | 346 | |
347 | 347 | return $this->deleteBlob($result['_id']); |
@@ -271,7 +271,7 @@ discard block |
||
271 | 271 | protected function addShare(Collection $collection): bool |
272 | 272 | { |
273 | 273 | $that = $this; |
274 | - $collection->doRecursiveAction(function ($node) use ($that) { |
|
274 | + $collection->doRecursiveAction(function($node) use ($that) { |
|
275 | 275 | if ($node instanceof Collection) { |
276 | 276 | $that->addShare($node); |
277 | 277 | } else { |
@@ -292,7 +292,7 @@ discard block |
||
292 | 292 | protected function deleteShare(Collection $collection): bool |
293 | 293 | { |
294 | 294 | $that = $this; |
295 | - $collection->doRecursiveAction(function ($node) use ($that) { |
|
295 | + $collection->doRecursiveAction(function($node) use ($that) { |
|
296 | 296 | if ($node instanceof Collection) { |
297 | 297 | $that->deleteShare($node); |
298 | 298 | } else { |
@@ -75,7 +75,7 @@ |
||
75 | 75 | |
76 | 76 | $this->logger->info('new group ['.$result.'] created', [ |
77 | 77 | 'category' => get_class($this), |
78 | - ]); |
|
78 | + ]); |
|
79 | 79 | |
80 | 80 | return true; |
81 | 81 | } |
@@ -49,7 +49,7 @@ |
||
49 | 49 | 'foreignField' => '_id', |
50 | 50 | 'localField' => 'history.storage._id', |
51 | 51 | 'as' => 'blob', |
52 | - ]], |
|
52 | + ]], |
|
53 | 53 | ]); |
54 | 54 | |
55 | 55 | foreach ($cursor as $object) { |
@@ -629,7 +629,7 @@ |
||
629 | 629 | return true; |
630 | 630 | } |
631 | 631 | |
632 | - return $this->doRecursiveAction(function ($node) use ($conflict, $recursion) { |
|
632 | + return $this->doRecursiveAction(function($node) use ($conflict, $recursion) { |
|
633 | 633 | $node->undelete($conflict, $recursion, false); |
634 | 634 | }, NodeInterface::DELETED_ONLY); |
635 | 635 | } |
@@ -1044,6 +1044,7 @@ |
||
1044 | 1044 | |
1045 | 1045 | /** |
1046 | 1046 | * Duplicate name with a uniqid within name. |
1047 | + * @param string $name |
|
1047 | 1048 | */ |
1048 | 1049 | protected function getDuplicateName(?string $name = null, ?string $class = null): string |
1049 | 1050 | { |