gyselroth / balloon

src/lib/Balloon/Bootstrap/AbstractBootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon

src/lib/Balloon/Http.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon

src/lib/Balloon/Filesystem/Node/Collection.php

Spacing +20 added lines, -20 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -72,7 +72,7 @@ 
      * @param  Filesystem $fs
      * @return void
      */
-    public function __construct(?BSONDocument $node, Filesystem $fs)
+    public function __construct(? BSONDocument $node, Filesystem $fs)
     {
         parent::__construct($node, $fs);
         
@@ -80,7 +80,7 @@ 
             $this->_id = null;
 
             if ($this->_user instanceof User) {
-                $this->owner  = $this->_user->getId();
+                $this->owner = $this->_user->getId();
             }
         }
             
@@ -97,7 +97,7 @@ 
      * @param  bool $recursion_first
      * @return INode
      */
-    public function copyTo(Collection $parent, int $conflict=INode::CONFLICT_NOACTION, ?string $recursion=null, bool $recursion_first=true): INode
+    public function copyTo(Collection $parent, int $conflict = INode::CONFLICT_NOACTION, ? string $recursion = null, bool $recursion_first = true) : INode
     {
         if ($recursion === null) {
             $recursion_first = true;
@@ -196,7 +196,7 @@ 
      * @param  array|string $attribute
      * @return array|string
      */
-    public function getAttribute($attribute=[])
+    public function getAttribute($attribute = [])
     {
         if (empty($attribute)) {
             $attribute = [
@@ -242,7 +242,7 @@ 
      * @param   array $filter
      * @return  Generator
      */
-    public function getChildNodes(int $deleted=INode::DELETED_EXCLUDE, array $filter=[]): Generator
+    public function getChildNodes(int $deleted = INode::DELETED_EXCLUDE, array $filter = []): Generator
     {
         if ($this->_user instanceof User) {
             $this->_user->findNewShares();
@@ -313,7 +313,7 @@ 
      * @param   array $filter
      * @return  Generator
      */
-    public function getChildren(int $deleted=INode::DELETED_EXCLUDE, array $filter=[]): array
+    public function getChildren(int $deleted = INode::DELETED_EXCLUDE, array $filter = []): array
     {
         return iterator_to_array($this->getChildNodes($deleted, $filter));
     }
@@ -350,7 +350,7 @@ 
      *
      * @return ObjectId
      */
-    public function getRealId(): ?ObjectId
+    public function getRealId(): ? ObjectId
     {
         if ($this->shared == true && $this->isReference()) {
             return $this->reference;
@@ -384,7 +384,7 @@ 
      * @param  array $filter
      * @return INode
      */
-    public function getChild($node, int $deleted=INode::DELETED_EXCLUDE, array $filter=[]): INode
+    public function getChild($node, int $deleted = INode::DELETED_EXCLUDE, array $filter = []): INode
     {
         //if $node is string load the object from the backend based on the current parent (the name
         //is unique per depth, so we can load the object)
@@ -453,7 +453,7 @@ 
      * @param   int $deleted
      * @return  bool
      */
-    protected function doRecursiveAction(string $method, array $params=[], int $deleted=INode::DELETED_EXCLUDE): bool
+    protected function doRecursiveAction(string $method, array $params = [], int $deleted = INode::DELETED_EXCLUDE): bool
     {
         if (!is_callable([$this, $method])) {
             throw new Exception("method $method is not callable in ".__CLASS__);
@@ -480,7 +480,7 @@ 
      * @param   bool $recursion_first
      * @return  bool
      */
-    public function delete(bool $force=false, ?string $recursion=null, bool $recursion_first=true): bool
+    public function delete(bool $force = false, ? string $recursion = null, bool $recursion_first = true) : bool
     {
         if (!$this->isAllowed('w') && !$this->isReference()) {
             throw new Exception\Forbidden('not allowed to delete node '.$this->name,
@@ -543,7 +543,7 @@ 
      * @param   bool $recursion_first
      * @return  bool
      */
-    protected function _forceDelete(?string $recursion=null, bool $recursion_first=true): bool
+    protected function _forceDelete(? string $recursion = null, bool $recursion_first = true) : bool
     {
         if (!$this->isReference()) {
             $this->doRecursiveAction('delete', [
@@ -595,7 +595,7 @@ 
      * @param   array $filter
      * @return  bool
      */
-    public function childExists($name, $deleted=INode::DELETED_EXCLUDE, array $filter=[]): bool
+    public function childExists($name, $deleted = INode::DELETED_EXCLUDE, array $filter = []): bool
     {
         $find = [
             'parent'  => $this->getRealId(),
@@ -729,7 +729,7 @@ 
         }
 
         $this->shared = false;
-        $this->acl   = null;
+        $this->acl = null;
         $action = [
             '$unset' => [
                 'shared' => $this->_id
@@ -789,7 +789,7 @@ 
      * @param   arary $files
      * @return  array
      */
-    public function getChildrenRecursive(?ObjectId $id=null, ?array &$shares=[], ?array &$files=[]): array
+    public function getChildrenRecursive(? ObjectId $id = null, ? array &$shares = [], ? array &$files = []) : array
     {
         $list = [];
         $result = $this->_db->storage->find([
@@ -833,7 +833,7 @@ 
      * @param   bool $clone
      * @return  Collection
      */
-    public function addDirectory($name, array $attributes=[], int $conflict=INode::CONFLICT_NOACTION, bool $clone=false): Collection
+    public function addDirectory($name, array $attributes = [], int $conflict = INode::CONFLICT_NOACTION, bool $clone = false): Collection
     {
         if (!$this->isAllowed('w')) {
             throw new Exception\Forbidden('not allowed to create new node here',
@@ -880,7 +880,7 @@ 
                 'shared'    => ($this->shared === true ? $this->getRealId() : $this->shared)
             ];
 
-            $save  = array_merge($meta, $attributes);
+            $save = array_merge($meta, $attributes);
 
             $result = $this->_db->storage->insertOne($save);
             $save['_id'] = $result->getInsertedId();
@@ -913,7 +913,7 @@ 
      * @param   bool $clone
      * @return  File
      */
-    public function addFile($name, $data=null, array $attributes=[], int $conflict=INode::CONFLICT_NOACTION, bool $clone=false): File
+    public function addFile($name, $data = null, array $attributes = [], int $conflict = INode::CONFLICT_NOACTION, bool $clone = false): File
     {
         if (!$this->isAllowed('w')) {
             throw new Exception\Forbidden('not allowed to create new node here',
@@ -963,7 +963,7 @@ 
                 'shared'    => ($this->shared === true ? $this->getRealId() : $this->shared),
             ];
 
-            $save  = array_merge($meta, $attributes);
+            $save = array_merge($meta, $attributes);
 
             $result = $this->_db->storage->insertOne($save);
             $save['_id'] = $result->getInsertedId();
@@ -1013,7 +1013,7 @@ 
      * @param   string $data
      * @return  File
      */
-    public function createFile($name, $data=null): String
+    public function createFile($name, $data = null): String
     {
         return $this->addFile($name, $data)->getETag();
     }

src/lib/Balloon/Plugin/LdapAutoShare.php

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -58,7 +58,7 @@ 
      * @param  Iterable $config
      * @return PluginInterface
      */
-    public function setOptions(?Iterable $config): PluginInterface
+    public function setOptions(? Iterable $config) : PluginInterface
     {
         if ($config === null) {
             return $this;
@@ -100,7 +100,7 @@ 
                 'category' => get_class($this),
             ]);
 
-            $result =  $this->_findShare($share);
+            $result = $this->_findShare($share);
             $shares = array_merge_recursive($shares, $result);
         }
         
@@ -121,7 +121,7 @@ 
                 'category' => get_class($this),
             ]);
             
-            $owner  = new User($owner, $this->logger, $this->fs, true);
+            $owner = new User($owner, $this->logger, $this->fs, true);
             $this->fs->setUser($owner);
             $root = $this->fs->getRoot();
  

src/lib/Balloon/Rest/v1/Node.php

Spacing +61 added lines, -61 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -176,12 +176,12 @@ 
      * @return  INode
      */
     protected function _getNode(
-        ?string $id=null,
-        ?string $path=null,
-        ?string $class=null,
-        bool $multiple=false,
-        bool $allow_root=false,
-        int $deleted=2): INode
+        ? string $id = null,
+        ? string $path = null,
+        ? string $class = null,
+        bool $multiple = false,
+        bool $allow_root = false,
+        int $deleted = 2) : INode
     {
         if ($class === null) {
             $class = join('', array_slice(explode('\\', get_class($this)), -1));
@@ -226,7 +226,7 @@ 
      * @param   int $deleted
      * @return  Response
      */
-    public function head(?string $id=null, ?string $p=null, int $deleted=0): Response
+    public function head(? string $id = null, ? string $p = null, int $deleted = 0) : Response
     {
         try {
             $result = $this->_getNode($id, $p, null, false, false, $deleted);
@@ -283,12 +283,12 @@ 
      * @return  void
      */
     public function postUndelete(
-        $id=null,
-        ?string $p=null,
-        bool $move=false,
-        ?string $destid=null,
-        ?string $destp=null,
-        int $conflict=0): Response
+        $id = null,
+        ? string $p = null,
+        bool $move = false,
+        ? string $destid = null,
+        ? string $destp = null,
+        int $conflict = 0) : Response
     {
         if ($move == true) {
             try {
@@ -380,7 +380,7 @@ 
      * @param   array $options
      * @return  Response
      */
-    public function postShareLink(?string $id=null, ?string $p=null, array $options=[]): Response
+    public function postShareLink(? string $id = null, ? string $p = null, array $options = []) : Response
     {
         $node = $this->_getNode($id, $p);
         $options = Helper::filter($options);
@@ -411,7 +411,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function deleteShareLink(?string $id=null, ?string $p=null): Response
+    public function deleteShareLink(? string $id = null, ? string $p = null) : Response
     {
         $node = $this->_getNode($id, $p);
         
@@ -454,7 +454,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function getShareLink(?string $id=null, ?string $p=null): Response
+    public function getShareLink(? string $id = null, ? string $p = null) : Response
     {
         $result = Helper::escape(
             $this->_getNode($id, $p)->getShareLink()
@@ -510,13 +510,13 @@ 
      * @return  void
      */
     public function get(
-        $id=null,
-        ?string $p=null,
-        int $offset=0,
-        int $length=0,
-        ?string $encode=null,
-        bool $download=false,
-        string $name='selected'): void
+        $id = null,
+        ? string $p = null,
+        int $offset = 0,
+        int $length = 0,
+        ? string $encode = null,
+        bool $download = false,
+        string $name = 'selected') : void
     {
         if (is_array($id)) {
             $this->_combine($id, $p, $name);
@@ -533,13 +533,13 @@ 
         }
 
         if ($download == true) {
-            header('Content-Disposition: attachment; filename*=UTF-8\'\'' .rawurlencode($name));
+            header('Content-Disposition: attachment; filename*=UTF-8\'\''.rawurlencode($name));
             header('Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0');
             header('Content-Type: application/octet-stream');
             header('Content-Length: '.$node->getSize());
             header('Content-Transfer-Encoding: binary');
         } else {
-            header('Content-Disposition: inline; filename*=UTF-8\'\'' .rawurlencode($name));
+            header('Content-Disposition: inline; filename*=UTF-8\'\''.rawurlencode($name));
         }
 
         if ($stream === null) {
@@ -608,7 +608,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function postReadonly($id=null, ?string $p=null, bool $readonly=true): Response
+    public function postReadonly($id = null, ? string $p = null, bool $readonly = true) : Response
     {
         if (is_array($id)) {
             $failures = [];
@@ -651,7 +651,7 @@ 
      * @param   string $name
      * @return  void
      */
-    protected function _combine($id=null, ?string $path=null, string $name='selected'): void
+    protected function _combine($id = null, ? string $path = null, string $name = 'selected') : void
     {
         $temp = $this->config->dir->temp.DIRECTORY_SEPARATOR.'zip';
         if (!file_exists($temp)) {
@@ -776,10 +776,10 @@ 
      * @param  array $attributes
      * @return Response
      */
-    public function getAttributes($id=null, ?string $p=null, array $attributes=[]): Response
+    public function getAttributes($id = null, ? string $p = null, array $attributes = []) : Response
     {
         if (is_array($id)) {
-            $nodes    = [];
+            $nodes = [];
             foreach ($this->fs->findNodes($id) as $node) {
                 $nodes[] = Helper::escape($node->getAttribute($attributes));
             }
@@ -905,7 +905,7 @@ 
      * @param  array $attributes
      * @return Response
      */
-    public function getParent(?string $id=null, ?string $p=null, array $attributes=[]): Response
+    public function getParent(? string $id = null, ? string $p = null, array $attributes = []) : Response
     {
         $result = Helper::escape(
             $this->_getNode($id, $p)
@@ -987,7 +987,7 @@ 
      * @param  array $attributes
      * @return Response
      */
-    public function getParents(?string $id=null, ?string $p=null, array $attributes=[], bool $self=false): Response
+    public function getParents(? string $id = null, ? string $p = null, array $attributes = [], bool $self = false) : Response
     {
         $request = $this->_getNode($id, $p);
         $parents = $request->getParents();
@@ -1035,7 +1035,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function postMetaAttributes(?string $id=null, ?string $p=null): Response
+    public function postMetaAttributes(? string $id = null, ? string $p = null) : Response
     {
         $this->_getNode($id, $p)->setMetaAttribute(Helper::filter($_POST));
         return (new Response())->setCode(204);
@@ -1068,7 +1068,7 @@ 
      * @param   string $name
      * @return  Response
      */
-    public function postName(string $name, ?string $id=null, ?string $p=null): Response
+    public function postName(string $name, ? string $id = null, ? string $p = null) : Response
     {
         $this->_getNode($id, $p)->setName($name);
         return (new Response())->setCode(204);
@@ -1106,11 +1106,11 @@ 
      * @return  Response
      */
     public function postClone(
-        $id=null,
-        ?string $p=null,
-        ?string $destid=null,
-        ?string $destp=null,
-        int $conflict=0): Response
+        $id = null,
+        ? string $p = null,
+        ? string $destid = null,
+        ? string $destp = null,
+        int $conflict = 0) : Response
     {
         try {
             $parent = $this->_getNode($destid, $destp, 'Collection', false, true);
@@ -1192,11 +1192,11 @@ 
      * @return  Response
      */
     public function postMove(
-        $id=null,
-        ?string $p=null,
-        ?string $destid=null,
-        ?string $destp=null,
-        int $conflict=0): Response
+        $id = null,
+        ? string $p = null,
+        ? string $destid = null,
+        ? string $destp = null,
+        int $conflict = 0) : Response
     {
         try {
             $parent = $this->_getNode($destid, $destp, 'Collection', false, true);
@@ -1276,11 +1276,11 @@ 
      * @return  Response
      */
     public function delete(
-        $id=null,
-        ?string $p=null,
-        bool $force=false,
-        bool $ignore_flag=false,
-        ?string $at=null): Response
+        $id = null,
+        ? string $p = null,
+        bool $force = false,
+        bool $ignore_flag = false,
+        ? string $at = null) : Response
     {
         $failures = [];
             
@@ -1373,7 +1373,7 @@ 
      * @param  array $attributes
      * @return Response
      */
-    public function getQuery(int $deleted=0, array $filter=[], array $attributes=[]): Response
+    public function getQuery(int $deleted = 0, array $filter = [], array $attributes = []): Response
     {
         $children = [];
         $nodes = $this->fs->findNodesWithCustomFilterUser($deleted, $filter);
@@ -1414,7 +1414,7 @@ 
      * @param  array $attributes
      * @return Response
      */
-    public function getTrash(array $attributes=[]): Response
+    public function getTrash(array $attributes = []): Response
     {
         $children = [];
         $nodes = $this->fs->findNodesWithCustomFilterUser(INode::DELETED_ONLY, ['deleted' => ['$type' => 9]]);
@@ -1489,7 +1489,7 @@ 
      * @param  int $deleted
      * @return Response
      */
-    public function getSearch(array $query, array $attributes=[], int $deleted=0): Response
+    public function getSearch(array $query, array $attributes = [], int $deleted = 0): Response
     {
         $children = [];
         $nodes = $this->fs->search($query, $deleted);
@@ -1599,11 +1599,11 @@ 
      * @return  Response
      */
     public function getDelta(
-        ?string $id=null,
-        ?string $p=null,
-        ?string $cursor=null,
-        int $limit=250,
-        array $attributes=[]): Response
+        ? string $id = null,
+        ? string $p = null,
+        ? string $cursor = null,
+        int $limit = 250,
+        array $attributes = []) : Response
     {
         if ($id !== null || $p !== null) {
             $node = $this->_getNode($id, $p);
@@ -1611,7 +1611,7 @@ 
             $node = null;
         }
         
-        $result= $this->fs->getDelta()->getDeltaFeed($cursor, $limit, $attributes, $node);
+        $result = $this->fs->getDelta()->getDeltaFeed($cursor, $limit, $attributes, $node);
 
         return (new Response())->setCode(200)->setBody($result);
     }
@@ -1719,7 +1719,7 @@ 
      * @param  int $limit
      * @return Response
      */
-    public function getEventLog(?string $id=null, ?string $p=null, int $skip=0, int $limit=100): Response
+    public function getEventLog(? string $id = null, ? string $p = null, int $skip = 0, int $limit = 100) : Response
     {
         if ($id !== null || $p !== null) {
             $node = $this->_getNode($id, $p);
@@ -1759,7 +1759,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function getLastCursor(?string $id=null, ?string $p=null): Response
+    public function getLastCursor(? string $id = null, ? string $p = null) : Response
     {
         if ($id !== null || $p !== null) {
             $node = $this->_getNode($id, $p);
@@ -1767,7 +1767,7 @@ 
             $node = null;
         }
 
-        $result= $this->fs->getDelta()->getLastCursor();
+        $result = $this->fs->getDelta()->getLastCursor();
         return (new Response())->setCode(200)->setBody($result);
     }
 }

src/lib/Balloon/Rest/v1/Collection.php

Spacing +15 added lines, -15 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -41,7 +41,7 @@ 
      * @param   string $p
      * @return  Response
      */
-    public function headChildren(?string $id=null, ?string $p=null): Response
+    public function headChildren(? string $id = null, ? string $p = null) : Response
     {
         $result   = $this->fs->getNode($id, $p, null, false, true);
         $children = $result->getSize();
@@ -97,11 +97,11 @@ 
      * @return Response
      */
     public function getChildren(
-        ?string $id=null,
-        ?string $p=null,
-        int $deleted=0,
-        Parameter\JSON $filter=NULL,
-        array $attributes=[]): Response
+        ? string $id = null,
+        ? string $p = null,
+        int $deleted = 0,
+        Parameter\JSON $filter = NULL,
+        array $attributes = []) : Response
     {
         $children = [];
         $nodes = $this->fs->getNode($id, $p, null, false, true)->getChildNodes($deleted, (array)$filter);
@@ -152,7 +152,7 @@ 
      * @param  string $p
      * @return Response
      */
-    public function getShare(?string $id=null, ?string $p=null): Response
+    public function getShare(? string $id = null, ? string $p = null) : Response
     {
         $result = $this->fs->getNode($id, $p)->getShare();
         return (new Response())->setCode(200)->setBody($result);
@@ -200,7 +200,7 @@ 
      * @param  array $acl
      * @return Response
      */
-    public function postShare(array $acl, ?string $id=null, ?string $p=null): Response
+    public function postShare(array $acl, ? string $id = null, ? string $p = null) : Response
     {
         $node    = $this->fs->getNode($id, $p);
         $result  = $node->share($acl);
@@ -236,7 +236,7 @@ 
      * @param  string $p
      * @return Response
      */
-    public function deleteShare(?string $id=null, ?string $p=null): Response
+    public function deleteShare(? string $id = null, ? string $p = null) : Response
     {
         $node = $this->fs->getNode($id, $p);
         $result = $node->unshare();
@@ -291,11 +291,11 @@ 
      * @return Response
      */
     public function post(
-        ?string $id=null,
-        ?string $p=null,
-        ?string $name=null,
-        array $attributes=[],
-        int $conflict=0): Response
+        ? string $id = null,
+        ? string $p = null,
+        ? string $name = null,
+        array $attributes = [],
+        int $conflict = 0) : Response
     {
         if ($p !== null && $name !== null) {
             throw new Exception\InvalidArgument('p and name can not be used at the same time');

src/lib/Balloon/Http/Router.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -196,7 +196,7 @@ 
      * @param  array $constructor
      * @return bool
      */
-    public function run(array $constructor=[]): bool
+    public function run(array $constructor = []): bool
     {
         $this->logger->info('execute requested route ['.$this->path.']', [
             'category' => get_class($this),
@@ -319,7 +319,7 @@ 
                 if (!empty($body)) {
                     $json_params = json_decode($body, true);
                 } else {
-                    $parts  = explode('&', $_SERVER['QUERY_STRING']);
+                    $parts = explode('&', $_SERVER['QUERY_STRING']);
                     if (!empty($parts)) {
                         $json_params = json_decode(urldecode($parts[0]), true);
                     }
@@ -341,13 +341,13 @@ 
                     $param_value = $request_params[$param->name];
                 } elseif (isset($json_params[$param->name])) {
                     $param_value = $json_params[$param->name];
-                } else if($optional === true) {
+                } else if ($optional === true) {
                     $param_value = $param->getDefaultValue();
                 } else {
                     $param_value = null;
                 }
 
-                switch($type) {
+                switch ($type) {
                     case 'bool':
                         $return[$param->name] = Helper::boolParam($param_value);
                     break;
@@ -358,7 +358,7 @@ 
                         $return[$param->name] = (array)$param_value;
                     break;
                     default:
-                        if(class_exists($type) && $param_value!==null) {
+                        if (class_exists($type) && $param_value !== null) {
                             $return[$param->name] = new $type($param, $param_value);
                         } else {
                             $return[$param->name] = $param_value;

src/lib/Balloon/Rest/v1/Parameter/JSON.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -23,17 +23,17 @@ 
      */
     public function __construct(\ReflectionParameter $param, $value)
     {
-        if(is_array($value)) {
+        if (is_array($value)) {
           $data = $value;
         } else {
-          if(!is_string($value)) {
-            throw new Exception\InvalidArgument('Parameter '.$param->name.' expects a json string. ' . gettype($value).' given.');
+          if (!is_string($value)) {
+            throw new Exception\InvalidArgument('Parameter '.$param->name.' expects a json string. '.gettype($value).' given.');
           }
 
           $data = json_decode($value);
 
-          if($data === null) {
-              throw new Exception\InvalidArgument('Parameter '.$param->name.' expects a valid json string. ' . json_last_error_msg());
+          if ($data === null) {
+              throw new Exception\InvalidArgument('Parameter '.$param->name.' expects a valid json string. '.json_last_error_msg());
           }
         }