gyselroth / balloon

src/lib/Filesystem.php

Doc Comments +3 added lines, -4 removed lines

@@ -181,7 +181,7 @@ 
     /**
      * Find raw node.
      *
-     * @param ObjectId $id
+     * @param ObjectId|null $id
      *
      * @return array
      */
@@ -302,7 +302,7 @@ 
      *
      * @param array  $id
      * @param string $class   Force check node type
-     * @param bool   $deleted
+     * @param integer   $deleted
      *
      * @return Generator
      */
@@ -431,7 +431,7 @@ 
      * @param string $class      Force set node type
      * @param bool   $multiple   Allow $id to be an array
      * @param bool   $allow_root Allow instance of root collection
-     * @param bool   $deleted    How to handle deleted node
+     * @param integer   $deleted    How to handle deleted node
      *
      * @return NodeInterface
      */
@@ -493,7 +493,6 @@ 
      * @param array $filter
      * @param int   $offset
      * @param int   $limit
-     * @param int   $total
      *
      * @return Generator
      */

src/lib/Filesystem/Node/Collection.php

Doc Comments +2 added lines, -4 removed lines

@@ -269,7 +269,7 @@ 
     /**
      * Get real id (reference).
      *
-     * @return ObjectId
+     * @return ObjectId|null
      */
     public function getRealId(): ?ObjectId
     {
@@ -298,7 +298,6 @@ 
     /**
      * Fetch children items of this collection.
      *
-     * @param string $node
      * @param int    $deleted
      * @param array  $filter
      * @param mixed  $name
@@ -860,7 +859,7 @@ 
      * @param string $name
      * @param string $data
      *
-     * @return File
+     * @return string
      */
     public function createFile($name, $data = null): String
     {
@@ -885,7 +884,6 @@ 
      *
      * @param callable $callable
      * @param int      $deleted
-     * @param bool     $ignore_exception
      *
      * @return bool
      */

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use Balloon\Filesystem\Acl\Exception\Forbidden as ForbiddenException;
 use Balloon\Filesystem\Exception;
 use Balloon\Hook;
-use Balloon\Server\User;
 use Generator;
 use MongoDB\BSON\ObjectId;
 use MongoDB\BSON\Regex;

Spacing +2 added lines, -2 removed lines

@@ -386,7 +386,7 @@ 
         $this->deleted = new UTCDateTime();
 
         if (!$this->isReference()) {
-            $this->doRecursiveAction(function ($node) use ($recursion) {
+            $this->doRecursiveAction(function($node) use ($recursion) {
                 $node->delete(false, $recursion, false);
             }, NodeInterface::DELETED_EXCLUDE);
         }
@@ -936,7 +936,7 @@ 
     protected function _forceDelete(?string $recursion = null, bool $recursion_first = true): bool
     {
         if (!$this->isReference()) {
-            $this->doRecursiveAction(function ($node) use ($recursion) {
+            $this->doRecursiveAction(function($node) use ($recursion) {
                 $node->delete(true, $recursion, false);
             }, NodeInterface::DELETED_INCLUDE);
         }

src/lib/Server.php

Doc Comments +1 added lines

@@ -209,6 +209,7 @@
     /**
      * Filesystem factory.
      *
+     * @param User $user
      * @return Filesystem
      */
     public function getFilesystem(?User $user = null): Filesystem

Indentation +3 added lines, -3 removed lines

@@ -531,7 +531,7 @@ 
 
         $this->identity = $user;
         $user->updateIdentity($identity)
-             ->updateShares();
+                ->updateShares();
         $this->hook->run('postServerIdentity', [$user]);
 
         return true;
@@ -662,7 +662,7 @@ 
     public function getGroupByName(string $name): Group
     {
         $group = $this->db->group->findOne([
-           'name' => $name,
+            'name' => $name,
         ]);
 
         if (null === $group) {
@@ -682,7 +682,7 @@ 
     public function getGroupById(ObjectId $id): Group
     {
         $group = $this->db->group->findOne([
-           '_id' => $id,
+            '_id' => $id,
         ]);
 
         if (null === $group) {

src/lib/Server/Group.php

Doc Comments +2 added lines

@@ -265,6 +265,8 @@
     /**
      * Get resolved member.
      *
+     * @param integer $offset
+     * @param integer $limit
      * @return Generator
      */
     public function getResolvedMembers(?int $offset = null, ?int $limit = null): ?Generator

src/app/Balloon.App.Notification/Api/v2/Notifications.php

Indentation +4 added lines, -4 removed lines

@@ -221,10 +221,10 @@
     {
         $mail = new Message();
         $mail->setBody($body)
-          ->setFrom($this->user->getAttributes()['mail'], $this->user->getAttributes()['username'])
-          ->setSubject($subject)
-          ->setTo($this->user->getAttributes()['mail'], 'Undisclosed Recipients')
-          ->setBcc($receiver);
+            ->setFrom($this->user->getAttributes()['mail'], $this->user->getAttributes()['username'])
+            ->setSubject($subject)
+            ->setTo($this->user->getAttributes()['mail'], 'Undisclosed Recipients')
+            ->setBcc($receiver);
         $this->async->addJob(Mail::class, $mail->toString());
 
         return (new Response())->setCode(202);

src/app/Balloon.App.Sharelink/Migration/Delta/SharelinkIntoApp.php

Indentation +2 added lines, -2 removed lines

@@ -49,8 +49,8 @@
             $this->db->storage->updateOne(
                 ['_id' => $object['_id']],
                 [
-                  '$unset' => ['sharelink' => 1],
-                  '$set' => ['app.Balloon\App\Sharelink' => $object['sharelink']],
+                    '$unset' => ['sharelink' => 1],
+                    '$set' => ['app.Balloon\App\Sharelink' => $object['sharelink']],
                 ]
             );
         }

src/app/Balloon.App.Api/v2/Nodes.php

Indentation +44 added lines, -44 removed lines

@@ -279,50 +279,50 @@ 
         }
 
         return $response
-          ->setOutputFormat(null)
-          ->setBody(function () use ($node, $encode, $offset, $limit) {
-              $stream = $node->get();
-              $name = $node->getName();
-
-              if (null === $stream) {
-                  return;
-              }
-
-              if (0 !== $offset) {
-                  if (fseek($stream, $offset) === -1) {
-                      throw new Exception\Conflict(
+            ->setOutputFormat(null)
+            ->setBody(function () use ($node, $encode, $offset, $limit) {
+                $stream = $node->get();
+                $name = $node->getName();
+
+                if (null === $stream) {
+                    return;
+                }
+
+                if (0 !== $offset) {
+                    if (fseek($stream, $offset) === -1) {
+                        throw new Exception\Conflict(
                         'invalid offset requested',
                         Exception\Conflict::INVALID_OFFSET
                     );
-                  }
-              }
-
-              $read = 0;
-              if ('base64' === $encode) {
-                  header('Content-Encoding: base64');
-                  while (!feof($stream)) {
-                      if (0 !== $limit && $read + 8192 > $limit) {
-                          echo base64_encode(fread($stream, $limit - $read));
-                          exit();
-                      }
-
-                      echo base64_encode(fread($stream, 8192));
-                      $read += 8192;
-                  }
-              } else {
-                  while (!feof($stream)) {
-                      if (0 !== $limit && $read + 8192 > $limit) {
-                          echo fread($stream, $limit - $read);
-                          exit();
-                      }
-
-                      echo fread($stream, 8192);
-                      $read += 8192;
-                  }
-
-                  exit();
-              }
-          });
+                    }
+                }
+
+                $read = 0;
+                if ('base64' === $encode) {
+                    header('Content-Encoding: base64');
+                    while (!feof($stream)) {
+                        if (0 !== $limit && $read + 8192 > $limit) {
+                            echo base64_encode(fread($stream, $limit - $read));
+                            exit();
+                        }
+
+                        echo base64_encode(fread($stream, 8192));
+                        $read += 8192;
+                    }
+                } else {
+                    while (!feof($stream)) {
+                        if (0 !== $limit && $read + 8192 > $limit) {
+                            echo fread($stream, $limit - $read);
+                            exit();
+                        }
+
+                        echo fread($stream, 8192);
+                        $read += 8192;
+                    }
+
+                    exit();
+                }
+            });
     }
 
     /**
@@ -1073,9 +1073,9 @@ 
                 $node->zip($archive);
             } catch (\Exception $e) {
                 $this->logger->debug('failed zip node in multi node request ['.$node->getId().']', [
-                   'category' => get_class($this),
-                   'exception' => $e,
-               ]);
+                    'category' => get_class($this),
+                    'exception' => $e,
+                ]);
             }
         }
 

Spacing +8 added lines, -8 removed lines

@@ -187,7 +187,7 @@ 
             }
         }
 
-        return $this->bulk($id, $p, function ($node) use ($parent, $conflict, $move) {
+        return $this->bulk($id, $p, function($node) use ($parent, $conflict, $move) {
             if (true === $move) {
                 $node = $node->setParent($parent, $conflict);
             }
@@ -280,7 +280,7 @@ 
 
         return $response
           ->setOutputFormat(null)
-          ->setBody(function () use ($node, $encode, $offset, $limit) {
+          ->setBody(function() use ($node, $encode, $offset, $limit) {
               $stream = $node->get();
               $name = $node->getName();
 
@@ -429,7 +429,7 @@ 
             return (new Response())->setCode(200)->setBody($result);
         }
 
-        return $this->bulk($id, $p, function ($node) use ($attributes) {
+        return $this->bulk($id, $p, function($node) use ($attributes) {
             return [
                 'code' => 200,
                 'data' => $this->node_decorator->decorate($node, $attributes),
@@ -543,9 +543,9 @@ 
     public function patch(?string $name = null, ?array $meta = null, ?bool $readonly = null, ?array $filter = null, ?string $id = null, ?string $p = null): Response
     {
         $attributes = compact('name', 'meta', 'readonly', 'filter');
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
 
-        return $this->bulk($id, $p, function ($node) use ($attributes) {
+        return $this->bulk($id, $p, function($node) use ($attributes) {
             foreach ($attributes as $attribute => $value) {
                 switch ($attribute) {
                     case 'name':
@@ -627,7 +627,7 @@ 
             );
         }
 
-        return $this->bulk($id, $p, function ($node) use ($parent, $conflict) {
+        return $this->bulk($id, $p, function($node) use ($parent, $conflict) {
             $result = $node->copyTo($parent, $conflict);
 
             return [
@@ -691,7 +691,7 @@ 
             );
         }
 
-        return $this->bulk($id, $p, function ($node) use ($parent, $conflict) {
+        return $this->bulk($id, $p, function($node) use ($parent, $conflict) {
             $result = $node->setParent($parent, $conflict);
 
             return [
@@ -745,7 +745,7 @@ 
             $at = $this->_verifyAttributes(['destroy' => $at])['destroy'];
         }
 
-        return $this->bulk($id, $p, function ($node) use ($force, $ignore_flag, $at) {
+        return $this->bulk($id, $p, function($node) use ($force, $ignore_flag, $at) {
             if (null === $at) {
                 $node->delete($force && $node->isDeleted() || $force && $ignore_flag);
             } else {

src/app/Balloon.App.ClamAv/Scanner.php

Doc Comments +1 added lines, -2 removed lines

@@ -73,7 +73,6 @@ 
     /**
      * Constructor.
      *
-     * @param Database        $db
      * @param LoggerInterface $logger
      */
     public function __construct(SocketFactory $factory, LoggerInterface $logger, ?Iterable $config = null)
@@ -131,7 +130,7 @@ 
      *
      * @param File $file
      *
-     * @return array
+     * @return File
      */
     public function scan(File $file): array
     {

src/app/Balloon.App.Cli/Console/Jobs.php

Doc Comments -1 removed lines

@@ -50,7 +50,6 @@
     /**
      * Constructor.
      *
-     * @param App             $app
      * @param Async           $async
      * @param LoggerInterface $logger
      * @param GetOpt          $getopt