gyselroth / balloon

src/app/Balloon.App.Elasticsearch/Job.php

Indentation +1 added lines, -1 removed lines

@@ -341,7 +341,7 @@
         }
 
         $this->logger->debug('elasticsarch blob document ['.$result['_id'].'] has no references left, remove completely', [
-         'category' => get_class($this),
+            'category' => get_class($this),
         ]);
 
         return $this->deleteBlob($result['_id']);

Spacing +2 added lines, -2 removed lines

@@ -271,7 +271,7 @@ 
     protected function addShare(Collection $collection): bool
     {
         $that = $this;
-        $collection->doRecursiveAction(function ($node) use ($that) {
+        $collection->doRecursiveAction(function($node) use ($that) {
             if ($node instanceof Collection) {
                 $that->addShare($node);
             } else {
@@ -292,7 +292,7 @@ 
     protected function deleteShare(Collection $collection): bool
     {
         $that = $this;
-        $collection->doRecursiveAction(function ($node) use ($that) {
+        $collection->doRecursiveAction(function($node) use ($that) {
             if ($node instanceof Collection) {
                 $that->deleteShare($node);
             } else {

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

Indentation +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
 
         $this->logger->info('new group ['.$result.'] created', [
             'category' => get_class($this),
-       ]);
+        ]);
 
         return true;
     }

src/lib/Migration/Delta/AddHashToHistory.php

Indentation +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
                 'foreignField' => '_id',
                 'localField' => 'history.storage._id',
                 'as' => 'blob',
-           ]],
+            ]],
         ]);
 
         foreach ($cursor as $object) {

src/lib/Filesystem/Node/AbstractNode.php

Spacing +1 added lines, -1 removed lines

@@ -629,7 +629,7 @@
             return true;
         }
 
-        return $this->doRecursiveAction(function ($node) use ($conflict, $recursion) {
+        return $this->doRecursiveAction(function($node) use ($conflict, $recursion) {
             $node->undelete($conflict, $recursion, false);
         }, NodeInterface::DELETED_ONLY);
     }

Doc Comments +1 added lines

@@ -1044,6 +1044,7 @@
 
     /**
      * Duplicate name with a uniqid within name.
+     * @param string $name
      */
     protected function getDuplicateName(?string $name = null, ?string $class = null): string
     {

src/lib/Filesystem/Node/File.php

Spacing +7 added lines, -7 removed lines

@@ -45,14 +45,14 @@
      * @param array
      **/
     protected $temp_files = [
-        '/^\._(.*)$/',     // OS/X resource forks
-        '/^.DS_Store$/',   // OS/X custom folder settings
+        '/^\._(.*)$/', // OS/X resource forks
+        '/^.DS_Store$/', // OS/X custom folder settings
         '/^desktop.ini$/', // Windows custom folder settings
-        '/^Thumbs.db$/',   // Windows thumbnail cache
-        '/^.(.*).swpx$/',  // ViM temporary files
-        '/^.(.*).swx$/',   // ViM temporary files
-        '/^.(.*).swp$/',   // ViM temporary files
-        '/^\.dat(.*)$/',   // Smultron seems to create these
+        '/^Thumbs.db$/', // Windows thumbnail cache
+        '/^.(.*).swpx$/', // ViM temporary files
+        '/^.(.*).swx$/', // ViM temporary files
+        '/^.(.*).swp$/', // ViM temporary files
+        '/^\.dat(.*)$/', // Smultron seems to create these
         '/^~lock.(.*)#$/', // Windows 7 lockfiles
     ];
 

Indentation +1 added lines, -1 removed lines

@@ -502,7 +502,7 @@
         $this->increaseVersion();
 
         $this->addVersion($attributes)
-             ->postPutFile();
+                ->postPutFile();
 
         return $this->version;
     }

src/lib/Filesystem/Node/Collection.php

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);
         }

Doc Comments +2 added lines, -1 removed lines

@@ -257,7 +257,7 @@ 
     /**
      * Get real id (reference).
      *
-     * @return ObjectId
+     * @return ObjectId|null
      */
     public function getRealId(): ?ObjectId
     {
@@ -639,6 +639,7 @@ 
 
     /**
      * Create new file as a child from this collection.
+     * @param string $name
      */
     public function addFile($name, ?ObjectId $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;
 use Balloon\Hook;
-use Balloon\Server\User;
 use Generator;
 use MongoDB\BSON\ObjectId;
 use MongoDB\BSON\Regex;

src/lib/Filesystem/Node/AttributeDecorator.php

Spacing +14 added lines, -14 removed lines

@@ -131,20 +131,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()) {
@@ -153,10 +153,10 @@ 
 
                 return $this->decorate($node->getParent(), ['id', 'name', '_links']);
             },
-            '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 null;
                 }
@@ -167,7 +167,7 @@ 
                     return null;
                 }
             },
-            'sharename' => function ($node) {
+            'sharename' => function($node) {
                 if (!$node->isShared()) {
                     return null;
                 }
@@ -178,7 +178,7 @@ 
                     return null;
                 }
             },
-            'shareowner' => function ($node) use ($server, $fs, $decorator) {
+            'shareowner' => function($node) use ($server, $fs, $decorator) {
                 if (!$node->isSpecial()) {
                     return null;
                 }
@@ -192,7 +192,7 @@ 
                     return null;
                 }
             },
-            'owner' => function ($node) use ($server, $decorator) {
+            'owner' => function($node) use ($server, $decorator) {
                 try {
                     return $decorator->decorate(
                         $server->getUserById($node->getOwner()),
@@ -216,20 +216,20 @@ 
     protected function getTimeAttributes(NodeInterface $node, array $attributes): array
     {
         return [
-            'created' => function ($node) use ($attributes) {
+            'created' => function($node) use ($attributes) {
                 return $attributes['created']->toDateTime()->format('c');
             },
-            'changed' => function ($node) use ($attributes) {
+            'changed' => function($node) use ($attributes) {
                 return $attributes['changed']->toDateTime()->format('c');
             },
-            'deleted' => function ($node) use ($attributes) {
+            'deleted' => function($node) use ($attributes) {
                 if (false === $attributes['deleted']) {
                     return null;
                 }
 
                 return $attributes['deleted']->toDateTime()->format('c');
             },
-            'destroy' => function ($node) use ($attributes) {
+            'destroy' => function($node) use ($attributes) {
                 if (null === $attributes['destroy']) {
                     return null;
                 }
@@ -262,7 +262,7 @@ 
         return [
             'shared' => $node->isShared(),
             'reference' => $node->isReference(),
-            'filter' => function ($node) use ($attributes) {
+            'filter' => function($node) use ($attributes) {
                 if (null === $attributes['filter']) {
                     return null;
                 }

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     /**
      * Init.
      *
-     * @param Decorator $role_decorator
+     * @param RoleAttributeDecorator $role_decorator
      */
     public function __construct(Server $server, Acl $acl, RoleAttributeDecorator $role_decorator)
     {

src/lib/Bootstrap/Cli.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
      */
     protected function setExceptionHandler(): self
     {
-        set_exception_handler(function ($e) {
+        set_exception_handler(function($e) {
             $this->logger->emergency('uncaught exception: '.$e->getMessage(), [
                 'category' => get_class($this),
                 'exception' => $e,

src/lib/Bootstrap/Http.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 
         if ($this->auth->requireOne()) {
             if (!($this->auth->getIdentity()->getAdapter() instanceof AuthNone)) {
-                $this->auth->getIdentity()->getAttributeMap()->addMapper('binary', function ($value) {
+                $this->auth->getIdentity()->getAttributeMap()->addMapper('binary', function($value) {
                     return new Binary($value, Binary::TYPE_GENERIC);
                 });
 
@@ -138,7 +138,7 @@ 
      */
     protected function setExceptionHandler(): self
     {
-        set_exception_handler(function ($e) {
+        set_exception_handler(function($e) {
             $this->logger->emergency('uncaught exception: '.$e->getMessage(), [
                 'category' => get_class($this),
                 'exception' => $e,