gyselroth / balloon

src/lib/Balloon/Log/Adapter/Stdout.php

Doc Comments +2 added lines, -2 removed lines

@@ -61,8 +61,8 @@
      * Convert to bash color
      *
      * @param   string $string
-     * @param   string $forderground_color
-     * @param   string $backgrond_coilor
+     * @param   string $foreground_color
+     * @param   string $background_color
      * @return  string
      */
     public function getColoredString(string $string, string $foreground_color = null, string $background_color = null): string

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -71,15 +71,15 @@ 
 
         // Check if given foreground color found
         if (isset($this->foreground_colors[$foreground_color])) {
-            $colored_string .= "\033[" . $this->foreground_colors[$foreground_color] . "m";
+            $colored_string .= "\033[".$this->foreground_colors[$foreground_color]."m";
         }
         // Check if given background color found
         if (isset($this->background_colors[$background_color])) {
-            $colored_string .= "\033[" . $this->background_colors[$background_color] . "m";
+            $colored_string .= "\033[".$this->background_colors[$background_color]."m";
         }
 
         // Add string and end coloring
-        $colored_string .=  $string . "\033[0m";
+        $colored_string .= $string."\033[0m";
 
         return $colored_string;
     }

src/lib/Balloon/Log/Adapter/Syslog.php

Doc Comments -1 removed lines

@@ -40,7 +40,6 @@
     /**
      * Set options
      *
-     * @param   Iterable $options
      * @return  AdapterInterface
      */
     public function setOptions(?Iterable $config=null)

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -43,7 +43,7 @@ 
      * @param   Iterable $options
      * @return  AdapterInterface
      */
-    public function setOptions(?Iterable $config=null)
+    public function setOptions(? Iterable $config = null)
     {
         parent::setOptions($options);
 

src/lib/Balloon/Logger.php

Doc Comments +1 added lines, -1 removed lines

@@ -198,7 +198,7 @@
      * @param   string $date_format
      * @param   string $level
      * @param   array $context
-     * @return  void
+     * @return  string
      */
     protected function _format(string $message, string $format, string $date_format, string $level, array $context=[]): string
     {

Unused Use Statements -2 removed lines

@@ -14,9 +14,7 @@
 use \Psr\Log\AbstractLogger;
 use \Psr\Log\LogLevel;
 use \Psr\Log\LoggerInterface;
-use Balloon\Log\Adapter\AbstractAdapter;
 use Balloon\Log\Adapter\AdapterInterface;
-use Balloon\Config;
 
 class Logger extends AbstractLogger implements LoggerInterface
 {

Spacing +10 added lines, -10 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -57,7 +57,7 @@ 
      * @param   Iterable $config
      * @return  void
      */
-    public function __construct(?Iterable $config=null)
+    public function __construct(? Iterable $config = null)
     {
         $this->setOptions($config);
     }
@@ -69,7 +69,7 @@ 
      * @param  Iterable $config
      * @return Logger
      */
-    public function setOptions(?Iterable $config=null)
+    public function setOptions(? Iterable $config = null)
     {
         if ($config === null) {
             return $this;
@@ -93,7 +93,7 @@ 
      * @param  Iterable $config
      * @return AdapterInterface
      */
-    public function addAdapter(string $name, string $class, ?Iterable $config=null): AdapterInterface
+    public function addAdapter(string $name, string $class, ? Iterable $config = null) : AdapterInterface
     {
         if (isset($this->adapter[$name])) {
             throw new Exception\InvalidArgument('log adapter '.$name.' is already registered');
@@ -130,7 +130,7 @@ 
      * @param  array $adapters
      * @return array
      */
-    public function getAdapters(array $adapters=[]): array
+    public function getAdapters(array $adapters = []): array
     {
         if (empty($adapter)) {
             return $this->adapter;
@@ -157,7 +157,7 @@ 
      * @param   array $context
      * @return  bool
      */
-    public function log($level, $message, array $context=[]): bool
+    public function log($level, $message, array $context = []): bool
     {
         if (!array_key_exists($level, self::PRIORITIES)) {
             throw new Exception\InvalidArgument('log level '.$level.' is unkown');
@@ -200,9 +200,9 @@ 
      * @param   array $context
      * @return  void
      */
-    protected function _format(string $message, string $format, string $date_format, string $level, array $context=[]): string
+    protected function _format(string $message, string $format, string $date_format, string $level, array $context = []): string
     {
-        $parsed = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function ($match) use ($message, $level, $date_format, $context) {
+        $parsed = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function($match) use ($message, $level, $date_format, $context) {
             $key = '';
             $context = array_merge($this->context, $context);
                     
@@ -225,9 +225,9 @@ 
                     $replace = [];
                     foreach ($context as $key => $val) {
                         if (!is_array($val) && (!is_object($val) || method_exists($val, '__toString'))) {
-                            $replace['{' . $key . '}'] = $val;
+                            $replace['{'.$key.'}'] = $val;
                         } else {
-                            $replace['{' . $key . '}'] = json_encode($val);
+                            $replace['{'.$key.'}'] = json_encode($val);
                         }
                     }
 

src/lib/Balloon/Plugin.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
      * Run plugin method
      *
      * @param   string $method
-     * @param   array $context
+     * @param   User[] $context
      * @return  bool
      */
     public function run(string $method, array $context=[]): bool

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon;
 
 use Balloon\Plugin\PluginInterface;
-use Balloon\Filesystem;
 use Balloon\Plugin\Exception;
 use \Psr\Log\LoggerInterface as Logger;
 

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -53,7 +53,7 @@ 
      * @param   object|array $config
      * @return  bool
      */
-    public function registerPlugin($name, $config=null): bool
+    public function registerPlugin($name, $config = null): bool
     {
         if ($name instanceof Config) {
             foreach ($name as $id => $config) {
@@ -112,7 +112,7 @@ 
      * @param   array $context
      * @return  bool
      */
-    public function run(string $method, array $context=[]): bool
+    public function run(string $method, array $context = []): bool
     {
         $this->logger->debug('execute plugins hooks for ['.$method.']', [
             'category' => get_class($this),

src/lib/Balloon/Plugin/CleanTrash.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon\Plugin;
 
 use Balloon\Filesystem;
-use Balloon\Exception;
 use \MongoDB\BSON\UTCDateTime;
 
 class CleanTrash extends AbstractPlugin

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -31,7 +31,7 @@ 
      * @param  Iterable $config
      * @return PluginInterface
      */
-    public function setOptions(?Iterable $config): PluginInterface
+    public function setOptions(? Iterable $config) : PluginInterface
     {
         if ($config === null) {
             return $this;
@@ -57,7 +57,7 @@ 
      */
     public function cli(Filesystem $fs): void
     {
-        $lt  = time() - $this->max_age;
+        $lt = time() - $this->max_age;
 
         $result = $fs->findNodesWithCustomFilter(['deleted' => ['$lt' => new UTCDateTime($lt)]]);
         $this->logger->info('found ['.count($result).'] nodes for cleanup, force remove them from trash', [

src/lib/Balloon/Plugin/Delta.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Balloon\Filesystem\Node\Collection;
 use Balloon\Filesystem\Node\File;
 use Balloon\Filesystem\Node\INode;
-use Balloon\Filesystem;
 use \MongoDB\BSON\ObjectID;
 
 class Delta extends AbstractPlugin

Indentation +7 added lines, -7 removed lines

@@ -33,11 +33,11 @@ 
     ];
 
 
-   /**
-    * Init
-    *
-    * @return void
-    */
+    /**
+     * Init
+     *
+     * @return void
+     */
     public function init(): void
     {
         if (php_sapi_name() === 'cli') {
@@ -437,8 +437,8 @@ 
         ];
  
         $node->getFilesystem()->getDatabase()->delta->updateMany([
-             'node' => [
-                 '$in' => $toset,
+                'node' => [
+                    '$in' => $toset,
             ],
         ], $action);
 

Spacing +11 added lines, -11 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -125,7 +125,7 @@ 
      * @return  void
      */
     public function postCopyCollection(Collection $node, Collection $parent,
-        Collection $new_node, int $conflict, ?string $recursion, bool $recursion_first): void
+        Collection $new_node, int $conflict, ? string $recursion, bool $recursion_first) : void
     {
         if ($recursion_first === false) {
             return;
@@ -162,7 +162,7 @@ 
      * @return  void
      */
     public function postCopyFile(File $node, Collection $parent, File $new_node,
-        int $conflict, ?string $recursion, bool $recursion_first): void
+        int $conflict, ? string $recursion, bool $recursion_first) : void
     {
         if ($recursion_first === false) {
             return;
@@ -229,14 +229,14 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function postDeleteCollection(Collection $node, bool $force, ?string $recursion, bool $recursion_first): void
+    public function postDeleteCollection(Collection $node, bool $force, ? string $recursion, bool $recursion_first) : void
     {
         if ($recursion_first === false) {
             return;
         }
    
         $attributes = $node->getAttribute(['parent', 'name']);
-        $attributes['node']      = $node->getId();
+        $attributes['node'] = $node->getId();
            
 
         if ($node->isReference()) {
@@ -257,7 +257,7 @@ 
             $attributes['share'] = $node->getShareId();
         }
 
-        $attributes['client']= $this->client;
+        $attributes['client'] = $this->client;
         $attributes['owner'] = $this->getEventOwner($node);
         $attributes['force'] = $force;
         $node->getFilesystem()->getDelta()->add($attributes);
@@ -292,14 +292,14 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function postDeleteFile(File $node, bool $force, ?string $recursion, bool $recursion_first): void
+    public function postDeleteFile(File $node, bool $force, ? string $recursion, bool $recursion_first) : void
     {
         if ($recursion_first === false) {
             return;
         }
     
         $attributes = $node->getAttribute(['parent', 'name']);
-        $attributes['node']      = $node->getId();
+        $attributes['node'] = $node->getId();
         
         if ($force === true) {
             $attributes['operation'] = 'forceDeleteFile';
@@ -311,7 +311,7 @@ 
             $attributes['share'] = $node->getShareId();
         }
 
-        $attributes['client']= $this->client;
+        $attributes['client'] = $this->client;
         $attributes['owner'] = $this->getEventOwner($node);
         $attributes['force'] = $force;
         $node->getFilesystem()->getDelta()->add($attributes);
@@ -330,7 +330,7 @@ 
      * @param  bool $recursion_first
      * @return void
      */
-    public function postSaveNodeAttributes(INode $node, array $attributes, array $remove, ?string $recursion, bool $recursion_first): void
+    public function postSaveNodeAttributes(INode $node, array $attributes, array $remove, ? string $recursion, bool $recursion_first) : void
     {
         if ($recursion_first === false) {
             return;
@@ -413,7 +413,7 @@ 
             }
         }
         
-        $log['client']= $this->client;
+        $log['client'] = $this->client;
 
         if (isset($log['operation'])) {
             $node->getFilesystem()->getDelta()->add($log);

src/lib/Balloon/Plugin/MailNotification.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Balloon\Plugin;
 
 use Balloon\User;
-use Balloon\Queue\JobInterface;
 use Balloon\Filesystem\Node\INode;
 use Balloon\Resource;
 use Balloon\Filesystem\Node\Collection;

Indentation +1 added lines, -1 removed lines

@@ -152,7 +152,7 @@
                 $mail  = new Message();
                 $mail->setBody($body);
                 $mail->setFrom($this->notifications['new_share']['sender']['address'],
-                  $this->notifications['new_share']['sender']['name']);
+                    $this->notifications['new_share']['sender']['name']);
                 $mail->setSubject($subject);
                 
                 foreach ($receiver as $rec) {

Spacing +8 added lines, -8 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -52,7 +52,7 @@ 
      * @param  Iterable $config
      * @return PluginInterface
      */
-    public function setOptions(?Iterable $config): PluginInterface
+    public function setOptions(? Iterable $config) : PluginInterface
     {
         if ($config === null) {
             return $this;
@@ -107,7 +107,7 @@ 
      * @param  bool $recursion_first
      * @return void
      */
-    public function postSaveNodeAttributes(INode $node, array $attributes, array $remove, ?string $recursion, bool $recursion_first): void
+    public function postSaveNodeAttributes(INode $node, array $attributes, array $remove, ? string $recursion, bool $recursion_first) : void
     {
         if (!($node instanceof Collection)) {
             return;
@@ -123,7 +123,7 @@ 
         $acl = $node->getShare();
         $resource = new Resource($fs->getUser(), $this->logger, $fs);
         if ($node->isShared() && is_array($acl) && (isset($raw['acl']) && $raw['acl'] !== $node->getAcl() || !isset($raw['acl']))) {
-            $receiver=[];
+            $receiver = [];
             foreach ($acl as $rule) {
                 if ($rule['type'] == 'user') {
                     $user = new User($rule['name'], $this->logger, $fs, true, false);
@@ -142,14 +142,14 @@ 
             }
 
             if (!empty($receiver)) {
-                $body = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function ($match) use ($node) {
+                $body = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function($match) use ($node) {
                     return $node->getAttribute($match[2]);
                 }, $this->notifications['new_share']['body']);
-                $subject = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function ($match) use ($node) {
+                $subject = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function($match) use ($node) {
                     return $node->getAttribute($match[2]);
                 }, $this->notifications['new_share']['subject']);
 
-                $mail  = new Message();
+                $mail = new Message();
                 $mail->setBody($body);
                 $mail->setFrom($this->notifications['new_share']['sender']['address'],
                   $this->notifications['new_share']['sender']['name']);
@@ -171,7 +171,7 @@ 
      * @param   User $user
      * @return  string
      */
-    protected function checkNotify(INode $node, User $user): ?string
+    protected function checkNotify(INode $node, User $user): ? string
     {
         if ($user->hasShare($node)) {
             $this->logger->debug('skip mail notifcation for share ['.$node->getId().'] user ['.$user->getId().'] already owns it', [

src/lib/Balloon/Plugin/PluginInterface.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 namespace Balloon\Plugin;
 
-use Balloon\Exception;
 use \Psr\Log\LoggerInterface as Logger;
 use Balloon\Filesystem;
 use Balloon\Filesystem\Node\Collection;

Indentation +2 added lines, -2 removed lines

@@ -278,7 +278,7 @@ 
      * @return  void
      */
     public function preCopyFile(File $node, Collection $parent,
-       int $conflict, ?string $recursion, bool $recursion_first): void;
+        int $conflict, ?string $recursion, bool $recursion_first): void;
 
 
     /**
@@ -295,7 +295,7 @@ 
      * @return  void
      */
     public function postCopyFile(File $node, Collection $parent, File $new_node,
-       int $conflict, ?string $recursion, bool $recursion_first): void;
+        int $conflict, ?string $recursion, bool $recursion_first): void;
 
 
     /**

Spacing +17 added lines, -17 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Balloon
@@ -36,7 +36,7 @@ 
      * @param  Iterable $config
      * @return PluginInterface
      */
-    public function setOptions(?Iterable $config): PluginInterface;
+    public function setOptions(? Iterable $config) : PluginInterface;
    
  
     /**
@@ -54,7 +54,7 @@ 
      * @param   array $params
      * @return  void
      */
-    public function __call(string $method, array $params=[]): void;
+    public function __call(string $method, array $params = []): void;
 
 
     /**
@@ -110,7 +110,7 @@ 
      * @param   mixed $content
      * @return  void
      */
-    public function preHttpResponse(\Balloon\Http\Response $response, int &$code, &$content): void;
+    public function preHttpResponse(\Balloon\Http\Response $response, int&$code, &$content): void;
     
     
     /**
@@ -189,7 +189,7 @@ 
      * @param   bool $clone
      * @return  void
      */
-    public function preCreateCollection(Collection $parent, string &$name, array &$attributes, bool $clone): void;
+    public function preCreateCollection(Collection $parent, string&$name, array &$attributes, bool $clone): void;
     
 
     /**
@@ -216,7 +216,7 @@ 
      * @param   bool $clone
      * @return  void
      */
-    public function preCreateFile(Collection $parent, string &$name, array &$attributes, bool $clone): void;
+    public function preCreateFile(Collection $parent, string&$name, array &$attributes, bool $clone): void;
 
 
     /**
@@ -245,7 +245,7 @@ 
      * @return  void
      */
     public function preCopyCollection(Collection $node, Collection $parent,
-        int $conflict, ?string $recursion, bool $recursion_first): void;
+        int $conflict, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -262,7 +262,7 @@ 
      * @return  void
      */
     public function postCopyCollection(Collection $node, Collection $parent,
-        Collection $new_node, int $conflict, ?string $recursion, bool $recursion_first): void;
+        Collection $new_node, int $conflict, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -278,7 +278,7 @@ 
      * @return  void
      */
     public function preCopyFile(File $node, Collection $parent,
-       int $conflict, ?string $recursion, bool $recursion_first): void;
+       int $conflict, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -295,7 +295,7 @@ 
      * @return  void
      */
     public function postCopyFile(File $node, Collection $parent, File $new_node,
-       int $conflict, ?string $recursion, bool $recursion_first): void;
+       int $conflict, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -309,7 +309,7 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function preDeleteCollection(Collection $node, bool $force, ?string $recursion, bool $recursion_first): void;
+    public function preDeleteCollection(Collection $node, bool $force, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -323,7 +323,7 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function postDeleteCollection(Collection $node, bool $force, ?string $recursion, bool $recursion_first): void;
+    public function postDeleteCollection(Collection $node, bool $force, ? string $recursion, bool $recursion_first) : void;
 
     
     /**
@@ -337,7 +337,7 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function preDeleteFile(File $node, bool $force, ?string $recursion, bool $recursion_first): void;
+    public function preDeleteFile(File $node, bool $force, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -351,7 +351,7 @@ 
      * @param   bool $recursion_first
      * @return  void
      */
-    public function postDeleteFile(File $node, bool $force, ?string $recursion, bool $recursion_first): void;
+    public function postDeleteFile(File $node, bool $force, ? string $recursion, bool $recursion_first) : void;
 
 
     /**
@@ -367,7 +367,7 @@ 
      * @return  void
      */
     public function preSaveNodeAttributes(INode $node, array &$save_attributes,
-        array &$remove_attributes, ?string $recursion, bool $recursion_first): void;
+        array &$remove_attributes, ? string $recursion, bool $recursion_first) : void;
 
     
     /**
@@ -383,7 +383,7 @@ 
      * @return void
      */
     public function postSaveNodeAttributes(INode $node, array $save_attributes,
-        array $remove_attributes, ?string $recursion, bool $recursion_first): void;
+        array $remove_attributes, ? string $recursion, bool $recursion_first) : void;
    
  
     /**
@@ -397,7 +397,7 @@ 
      * @param   bool $autocreate
      * @return  void
      */
-    public function preInstanceUser(\Balloon\User $user, string &$username, ?array &$attributes, bool $autocreate): void;
+    public function preInstanceUser(\Balloon\User $user, string&$username, ? array &$attributes, bool $autocreate) : void;
     
 
     /**

src/lib/Balloon/Plugin/Preview.php

Unused Use Statements -2 removed lines

@@ -11,8 +11,6 @@
 
 namespace Balloon\Plugin;
 
-use Balloon\Filesystem;
-use Balloon\Exception;
 use Balloon\Filesystem\Node\File;
 use Balloon\Queue\Preview as PreviewJob;
 

Spacing +1 added lines, -1 removed lines

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