gyselroth / balloon

src/app/Balloon.App.Api/v1/File.php

Doc Comments +5 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
      *
      * @param string $id
      * @param string $p
-     * @param string $version
+     * @param integer $version
      */
     public function postRestore(int $version, ?string $id = null, ?string $p = null): Response
     {
@@ -406,6 +406,9 @@ 
 
     /**
      * Get Parent.
+     * @param string|null $id
+     * @param string|null $p
+     * @param string|null $collection
      */
     protected function getParent($id, $p, $collection): Collection
     {
@@ -427,7 +430,7 @@ 
     /**
      * Add or update file.
      *
-     * @param ObjecId $session
+     * @param SessionInterface $session
      * @param string  $id
      * @param string  $p
      * @param string  $collection

src/app/Balloon.App.Notification/Notifier.php

Doc Comments +4 added lines

@@ -129,6 +129,7 @@ 
 
     /**
      * Send notification.
+     * @param User $sender
      */
     public function notify(iterable $receiver, ?User $sender, MessageInterface $message): bool
     {
@@ -212,6 +213,7 @@ 
 
     /**
      * Add notification.
+     * @param User $sender
      */
     public function postNotification(User $receiver, ?User $sender, MessageInterface $message): ObjectId
     {
@@ -238,6 +240,8 @@ 
 
     /**
      * Get notifications.
+     * @param integer $offset
+     * @param integer $limit
      */
     public function getNotifications(User $user, array $query = [], ?int $offset = null, ?int $limit = null, ?int &$total = null): iterable
     {

Spacing +1 added lines, -1 removed lines

@@ -420,7 +420,7 @@
 
             if ($node instanceof Collection && $recursive === true) {
                 $db = $this->db;
-                $node->doRecursiveAction(function ($child) use ($db, $node_id, $user_id) {
+                $node->doRecursiveAction(function($child) use ($db, $node_id, $user_id) {
                     $db->subscription->deleteOne([
                         'user' => $user_id,
                         'node' => $node_id,

src/app/Balloon.App.Preview/Preview.php

Doc Comments +1 added lines

@@ -65,6 +65,7 @@
 
     /**
      * Set preview.
+     * @param resource $stream
      */
     public function setPreview(File $file, $stream): ObjectId
     {

src/app/Balloon.App.Wopi/HostManager.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon\App\Wopi;
 
 use GuzzleHttp\ClientInterface as GuzzleHttpClientInterface;
-use InvalidArgumentException;
 use phpseclib\Crypt\RSA;
 use phpseclib\Math\BigInteger;
 use Psr\Log\LoggerInterface;

src/lib/Filesystem/Node/Collection.php

Doc Comments +1 added lines

@@ -590,6 +590,7 @@
 
     /**
      * Create new file as a child from this collection.
+     * @param string $name
      */
     public function addFile($name, ?SessionInterface $session = null, array $attributes = [], int $conflict = NodeInterface::CONFLICT_NOACTION, bool $clone = false): File
     {

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Balloon\Filesystem\Exception;
 use Balloon\Filesystem\Storage\Adapter\AdapterInterface as StorageAdapterInterface;
 use Balloon\Hook;
-use Balloon\Server\User;
 use Balloon\Session\Factory as SessionFactory;
 use Balloon\Session\SessionInterface;
 use Generator;

src/lib/Migration/Delta/CreateUniqueUserMailIndexAllowNull.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon\Migration\Delta;
 
 use MongoDB\Database;
-use MongoDB\Driver\Exception\CommandException;
 use MongoDB\Driver\Exception\RuntimeException;
 
 class CreateUniqueUserMailIndexAllowNull implements DeltaInterface

src/lib/Session/Factory.php

Doc Comments +1 added lines

@@ -89,6 +89,7 @@
 
     /**
      * Update.
+     * @param resource $stream
      */
     public function update(User $user, SessionInterface $resource, Collection $parent, $stream): bool
     {

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

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
 
         if ($index === $chunks) {
             $attributes = compact('changed', 'created', 'readonly', 'meta', 'acl');
-            $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+            $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
             $attributes = $this->_verifyAttributes($attributes);
 
             return $this->_put($session, $id, $collection, $name, $attributes, $conflict);
@@ -129,7 +129,7 @@ 
         $parent = $this->getParent($id, $collection);
         $session = $this->session_factory->add($this->server->getIdentity(), $parent, $input);
         $attributes = compact('changed', 'created', 'readonly', 'meta', 'acl');
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
         $attributes = $this->_verifyAttributes($attributes);
 
         return $this->_put($session, $id, $collection, $name, $attributes, $conflict);

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

Spacing +7 added lines, -7 removed lines

@@ -108,7 +108,7 @@ 
             }
         }
 
-        return $this->bulk($id, function ($node) use ($parent, $conflict, $move) {
+        return $this->bulk($id, function($node) use ($parent, $conflict, $move) {
             if (true === $move) {
                 $node = $node->setParent($parent, $conflict);
             }
@@ -175,7 +175,7 @@ 
             return (new Response())->setCode(200)->setBody($result);
         }
 
-        return $this->bulk($id, function ($node) use ($attributes) {
+        return $this->bulk($id, function($node) use ($attributes) {
             return [
                 'code' => 200,
                 'data' => $this->node_decorator->decorate($node, $attributes),
@@ -211,11 +211,11 @@ 
     public function patch(string $id, ?string $name = null, ?array $meta = null, ?bool $readonly = null, ?array $filter = null, $lock = null): Response
     {
         $attributes = compact('name', 'meta', 'readonly', 'filter', 'lock');
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
 
         $lock = $_SERVER['HTTP_LOCK_TOKEN'] ?? null;
 
-        return $this->bulk($id, function ($node) use ($attributes, $lock) {
+        return $this->bulk($id, function($node) use ($attributes, $lock) {
             foreach ($attributes as $attribute => $value) {
                 switch ($attribute) {
                     case 'name':
@@ -268,7 +268,7 @@ 
             throw new Exception\NotFound('destination collection was not found or is not a collection', Exception\NotFound::DESTINATION_NOT_FOUND);
         }
 
-        return $this->bulk($id, function ($node) use ($parent, $conflict) {
+        return $this->bulk($id, function($node) use ($parent, $conflict) {
             $result = $node->copyTo($parent, $conflict);
 
             return [
@@ -292,7 +292,7 @@ 
             throw new Exception\NotFound('destination collection was not found or is not a collection', Exception\NotFound::DESTINATION_NOT_FOUND);
         }
 
-        return $this->bulk($id, function ($node) use ($parent, $conflict) {
+        return $this->bulk($id, function($node) use ($parent, $conflict) {
             $result = $node->setParent($parent, $conflict);
 
             return [
@@ -317,7 +317,7 @@ 
             $at = $this->_verifyAttributes(['destroy' => $at])['destroy'];
         }
 
-        return $this->bulk($id, function ($node) use ($force, $ignore_flag, $at) {
+        return $this->bulk($id, function($node) use ($force, $ignore_flag, $at) {
             if (null === $at) {
                 $node->delete($force && $node->isDeleted() || $force && $ignore_flag);
             } else {