gyselroth / balloon

src/app/Balloon.App.Convert/Constructor/Http.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             ->prependRoute(new Route('/api/v2/files/convert(/|\z)', Convert::class))
             ->prependRoute(new Route('/api/v2/files/{id:#([0-9a-z]{24})#}/convert(/|\z)', Convert::class));
 
-        $node_decorator->addDecorator('master', function ($node) use ($convert_decorator, $converter) {
+        $node_decorator->addDecorator('master', function($node) use ($convert_decorator, $converter) {
             $master = $node->getAppAttribute('Balloon\\App\\Convert', 'master');
             if (null === $master) {
                 return null;

src/app/Balloon.App.Convert/AttributeDecorator.php

Spacing +2 added lines, -2 removed lines

@@ -101,14 +101,14 @@
         return [
             'id' => (string) $slave['_id'],
             'format' => (string) $slave['format'],
-            'master' => function ($slave) use ($fs, $node_decorator) {
+            'master' => function($slave) use ($fs, $node_decorator) {
                 try {
                     return $node_decorator->decorate($fs->findNodeById($slave['master']), ['_links', 'id', 'name']);
                 } catch (\Exception $e) {
                     return null;
                 }
             },
-            'slave' => function ($slave) use ($fs, $node_decorator) {
+            'slave' => function($slave) use ($fs, $node_decorator) {
                 if (!isset($slave['slave'])) {
                     return null;
                 }

src/app/Balloon.App.ClamAv/Constructor/Http.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function __construct(AttributeDecorator $decorator)
     {
-        $decorator->addDecorator('malware_quarantine', function ($node) {
+        $decorator->addDecorator('malware_quarantine', function($node) {
             if ($node instanceof File) {
                 $quarantine = $node->getAppAttribute('Balloon\\App\\ClamAv', 'quarantine');
                 if (is_bool($quarantine)) {
@@ -32,7 +32,7 @@ 
                 }
             }
         });
-        $decorator->addDecorator('malware_scantime', function ($node) {
+        $decorator->addDecorator('malware_scantime', function($node) {
             if ($node instanceof File) {
                 $ts = $node->getAppAttribute('Balloon\\App\\ClamAv', 'scantime');
 
@@ -41,7 +41,7 @@ 
                 }
             }
         });
-        $decorator->addDecorator('malware_reason', function ($node) {
+        $decorator->addDecorator('malware_reason', function($node) {
             if ($node instanceof File) {
                 return $node->getAppAttribute('Balloon\\App\\ClamAv', 'reason');
             }

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

Spacing +2 added lines, -2 removed lines

@@ -270,7 +270,7 @@ 
         }
 
         $attributes = compact('namespace', 'optional');
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
 
         $id = $this->server->addGroup($name, $member, $attributes);
         $result = $this->decorator->decorate($this->server->getGroupById($id));
@@ -309,7 +309,7 @@ 
     public function patch(string $id, ?array $member = null, ?string $namespace = null, ?array $optional = null): Response
     {
         $attributes = compact('namespace', 'optional', 'name', 'member');
-        $attributes = array_filter($attributes, function ($attribute) {return !is_null($attribute); });
+        $attributes = array_filter($attributes, function($attribute) {return !is_null($attribute); });
 
         $group = $this->_getGroup($id, true);
         $group->setAttributes($attributes);

src/app/Balloon.App.Api/v1/AttributeDecorator/RoleDecorator.php

Spacing +6 added lines, -6 removed lines

@@ -105,13 +105,13 @@ 
         }
 
         return [
-            'created' => function ($role, $requested) use ($attributes) {
+            'created' => function($role, $requested) use ($attributes) {
                 return $attributes['created']->toDateTime()->format('c');
             },
-            'changed' => function ($role, $requested) use ($attributes) {
+            'changed' => function($role, $requested) use ($attributes) {
                 return $attributes['changed']->toDateTime()->format('c');
             },
-            'deleted' => function ($role, $requested) use ($attributes) {
+            'deleted' => function($role, $requested) use ($attributes) {
                 if (false === $attributes['deleted']) {
                     return null;
                 }
@@ -164,14 +164,14 @@ 
             'name' => (string) $attributes['username'],
             'namespace' => (string) $attributes['namespace'],
             'mail' => (string) $attributes['mail'],
-            'avatar' => function ($role, $requested) use ($attributes) {
+            'avatar' => function($role, $requested) use ($attributes) {
                 if ($attributes['avatar'] instanceof Binary) {
                     return base64_encode($attributes['avatar']->getData());
                 }
 
                 return null;
             },
-            'soft_quota' => function ($role, $requested) use ($attributes, $user) {
+            'soft_quota' => function($role, $requested) use ($attributes, $user) {
                 if ($user === null) {
                     return null;
                 }
@@ -182,7 +182,7 @@ 
 
                 return null;
             },
-            'hard_quota' => function ($role, $requested) use ($attributes, $user) {
+            'hard_quota' => function($role, $requested) use ($attributes, $user) {
                 if ($user === null) {
                     return null;
                 }

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
      * Add decorator.
      *
      *
-     * @return AttributeDecorator
+     * @return RoleDecorator
      */
     public function addDecorator(string $attribute, Closure $decorator): self
     {

src/app/Balloon.App.Api/Controller.php

Doc Comments +2 added lines, -2 removed lines

@@ -210,7 +210,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
      */
     protected function _getNode(
         ?string $id = null,
@@ -242,7 +242,7 @@ 
      * @param string $id
      * @param string $path
      * @param string $class   Force set node type
-     * @param bool   $deleted How to handle deleted node
+     * @param integer   $deleted How to handle deleted node
      */
     protected function _getNodes(
         $id = null,

src/app/Balloon.App.Api/v1/AttributeDecorator/NodeDecorator.php

Doc Comments +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
      * Add decorator.
      *
      *
-     * @return AttributeDecorator
+     * @return NodeDecorator
      */
     public function addDecorator(string $attribute, Closure $decorator): self
     {

Spacing +11 added lines, -11 removed lines

@@ -127,20 +127,20 @@ 
             'mime' => (string) $attributes['mime'],
             'readonly' => (bool) $attributes['readonly'],
             'directory' => $node instanceof Collection,
-            'meta' => function ($node) {
+            'meta' => function($node) {
                 return (object) $node->getMetaAttributes();
             },
-            'size' => function ($node) {
+            'size' => function($node) {
                 return $node->getSize();
             },
-            'path' => function ($node) {
+            'path' => function($node) {
                 try {
                     return $node->getPath();
                 } catch (\Exception $e) {
                     return null;
                 }
             },
-            'parent' => function ($node) {
+            'parent' => function($node) {
                 $parent = $node->getParent();
 
                 if (null === $parent || $parent->isRoot()) {
@@ -149,10 +149,10 @@ 
 
                 return (string) $parent->getId();
             },
-            'access' => function ($node) use ($acl) {
+            'access' => function($node) use ($acl) {
                 return $acl->getAclPrivilege($node);
             },
-            'share' => function ($node) {
+            'share' => function($node) {
                 if (!$node->isShared() && !$node->isSpecial()) {
                     return false;
                 }
@@ -163,7 +163,7 @@ 
                     return false;
                 }
             },
-            'shareowner' => function ($node) use ($server, $fs) {
+            'shareowner' => function($node) use ($server, $fs) {
                 if (!$node->isSpecial()) {
                     return null;
                 }
@@ -206,20 +206,20 @@ 
     protected function getTimeAttributes(NodeInterface $node, array $attributes): array
     {
         return [
-            'created' => function ($node) use ($attributes) {
+            'created' => function($node) use ($attributes) {
                 return $this->dateTimeToUnix($attributes['created']);
             },
-            'changed' => function ($node) use ($attributes) {
+            'changed' => function($node) use ($attributes) {
                 return $this->dateTimeToUnix($attributes['changed']);
             },
-            'deleted' => function ($node) use ($attributes) {
+            'deleted' => function($node) use ($attributes) {
                 if (false === $attributes['deleted']) {
                     return false;
                 }
 
                 return $this->dateTimeToUnix($attributes['deleted']);
             },
-            'destroy' => function ($node) use ($attributes) {
+            'destroy' => function($node) use ($attributes) {
                 if (null === $attributes['destroy']) {
                     return null;
                 }

src/app/Balloon.App.Notification/Hook/NewShareAdded.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon\App\Notification\Hook;
 
 use Balloon\App\Notification\Notifier;
-use Balloon\Async\Mail;
 use Balloon\Filesystem\Node\Collection;
 use Balloon\Filesystem\Node\NodeInterface;
 use Balloon\Hook\AbstractHook;

src/lib/Server.php

Doc Comments +1 added lines

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

Indentation +3 added lines, -3 removed lines

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