Completed
Pull Request — master (#150)
by Raffael
09:51
created
src/lib/Server/Group.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -265,6 +265,8 @@
 block discarded – undo
265 265
     /**
266 266
      * Get resolved member.
267 267
      *
268
+     * @param integer $offset
269
+     * @param integer $limit
268 270
      * @return Generator
269 271
      */
270 272
     public function getResolvedMembers(?int $offset = null, ?int $limit = null): ?Generator
Please login to merge, or discard this patch.
src/lib/Server/User.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -722,6 +722,8 @@
 block discarded – undo
722 722
     /**
723 723
      * Get resolved groups.
724 724
      *
725
+     * @param integer $offset
726
+     * @param integer $limit
725 727
      * @return Generator
726 728
      */
727 729
     public function getResolvedGroups(?int $offset = null, ?int $limit = null): ?Generator
Please login to merge, or discard this patch.
src/lib/Filesystem/Node/AbstractNode.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -629,7 +629,7 @@
 block discarded – undo
629 629
             return true;
630 630
         }
631 631
 
632
-        return $this->doRecursiveAction(function ($node) use ($conflict, $recursion) {
632
+        return $this->doRecursiveAction(function($node) use ($conflict, $recursion) {
633 633
             $node->undelete($conflict, $recursion, false);
634 634
         }, NodeInterface::DELETED_ONLY);
635 635
     }
Please login to merge, or discard this patch.
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -1044,6 +1044,7 @@
 block discarded – undo
1044 1044
 
1045 1045
     /**
1046 1046
      * Duplicate name with a uniqid within name.
1047
+     * @param string $name
1047 1048
      */
1048 1049
     protected function getDuplicateName(?string $name = null, ?string $class = null): string
1049 1050
     {
Please login to merge, or discard this patch.
src/lib/Filesystem/Node/File.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -45,14 +45,14 @@
 block discarded – undo
45 45
      * @param array
46 46
      **/
47 47
     protected $temp_files = [
48
-        '/^\._(.*)$/',     // OS/X resource forks
49
-        '/^.DS_Store$/',   // OS/X custom folder settings
48
+        '/^\._(.*)$/', // OS/X resource forks
49
+        '/^.DS_Store$/', // OS/X custom folder settings
50 50
         '/^desktop.ini$/', // Windows custom folder settings
51
-        '/^Thumbs.db$/',   // Windows thumbnail cache
52
-        '/^.(.*).swpx$/',  // ViM temporary files
53
-        '/^.(.*).swx$/',   // ViM temporary files
54
-        '/^.(.*).swp$/',   // ViM temporary files
55
-        '/^\.dat(.*)$/',   // Smultron seems to create these
51
+        '/^Thumbs.db$/', // Windows thumbnail cache
52
+        '/^.(.*).swpx$/', // ViM temporary files
53
+        '/^.(.*).swx$/', // ViM temporary files
54
+        '/^.(.*).swp$/', // ViM temporary files
55
+        '/^\.dat(.*)$/', // Smultron seems to create these
56 56
         '/^~lock.(.*)#$/', // Windows 7 lockfiles
57 57
     ];
58 58
 
Please login to merge, or discard this patch.
src/lib/Filesystem/Node/Collection.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
         $this->deleted = new UTCDateTime();
387 387
 
388 388
         if (!$this->isReference()) {
389
-            $this->doRecursiveAction(function ($node) use ($recursion) {
389
+            $this->doRecursiveAction(function($node) use ($recursion) {
390 390
                 $node->delete(false, $recursion, false);
391 391
             }, NodeInterface::DELETED_EXCLUDE);
392 392
         }
@@ -936,7 +936,7 @@  discard block
 block discarded – undo
936 936
     protected function _forceDelete(?string $recursion = null, bool $recursion_first = true): bool
937 937
     {
938 938
         if (!$this->isReference()) {
939
-            $this->doRecursiveAction(function ($node) use ($recursion) {
939
+            $this->doRecursiveAction(function($node) use ($recursion) {
940 940
                 $node->delete(true, $recursion, false);
941 941
             }, NodeInterface::DELETED_INCLUDE);
942 942
         }
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
     /**
258 258
      * Get real id (reference).
259 259
      *
260
-     * @return ObjectId
260
+     * @return ObjectId|null
261 261
      */
262 262
     public function getRealId(): ?ObjectId
263 263
     {
@@ -639,6 +639,7 @@  discard block
 block discarded – undo
639 639
 
640 640
     /**
641 641
      * Create new file as a child from this collection.
642
+     * @param string $name
642 643
      */
643 644
     public function addFile($name, ?ObjectId $session = null, array $attributes = [], int $conflict = NodeInterface::CONFLICT_NOACTION, bool $clone = false): File
644 645
     {
Please login to merge, or discard this patch.
src/lib/Bootstrap/Cli.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -171,7 +171,7 @@
 block discarded – undo
171 171
      */
172 172
     protected function setExceptionHandler(): self
173 173
     {
174
-        set_exception_handler(function ($e) {
174
+        set_exception_handler(function($e) {
175 175
             $this->logger->emergency('uncaught exception: '.$e->getMessage(), [
176 176
                 'category' => get_class($this),
177 177
                 'exception' => $e,
Please login to merge, or discard this patch.
src/lib/Bootstrap/Http.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 
86 86
         if ($this->auth->requireOne()) {
87 87
             if (!($this->auth->getIdentity()->getAdapter() instanceof AuthNone)) {
88
-                $this->auth->getIdentity()->getAttributeMap()->addMapper('binary', function ($value) {
88
+                $this->auth->getIdentity()->getAttributeMap()->addMapper('binary', function($value) {
89 89
                     return new Binary($value, Binary::TYPE_GENERIC);
90 90
                 });
91 91
 
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
      */
139 139
     protected function setExceptionHandler(): self
140 140
     {
141
-        set_exception_handler(function ($e) {
141
+        set_exception_handler(function($e) {
142 142
             $this->logger->emergency('uncaught exception: '.$e->getMessage(), [
143 143
                 'category' => get_class($this),
144 144
                 'exception' => $e,
Please login to merge, or discard this patch.
src/lib/Bootstrap/AbstractBootstrap.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
      */
43 43
     protected function setErrorHandler(): self
44 44
     {
45
-        set_error_handler(function ($severity, $message, $file, $line) {
45
+        set_error_handler(function($severity, $message, $file, $line) {
46 46
             $log = $message.' in '.$file.':'.$line;
47 47
 
48 48
             switch ($severity) {
Please login to merge, or discard this patch.
src/app/Balloon.App.Notification/Constructor/Http.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
             ->prependRoute(new Route('/api/v2/nodes|files|collections/subscription(/|\z)', v2\Subscription::class))
33 33
             ->prependRoute(new Route('/api/v2/nodes|files|collections/{id:#([0-9a-z]{24})#}/subscription(/|\z)', v2\Subscription::class));
34 34
 
35
-        $decorator->addDecorator('subscription', function ($node) use ($notifier, $server) {
35
+        $decorator->addDecorator('subscription', function($node) use ($notifier, $server) {
36 36
             $subscription = $notifier->getSubscription($node, $server->getIdentity());
37 37
             if ($subscription === null) {
38 38
                 return false;
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
             return true;
42 42
         });
43 43
 
44
-        $decorator->addDecorator('subscription_exclude_me', function ($node) use ($notifier, $server) {
44
+        $decorator->addDecorator('subscription_exclude_me', function($node) use ($notifier, $server) {
45 45
             $subscription = $notifier->getSubscription($node, $server->getIdentity());
46 46
             if ($subscription === null) {
47 47
                 return false;
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
             return $subscription['exclude_me'];
51 51
         });
52 52
 
53
-        $decorator->addDecorator('subscription_recursive', function ($node) use ($notifier, $server) {
53
+        $decorator->addDecorator('subscription_recursive', function($node) use ($notifier, $server) {
54 54
             if (!($node instanceof Collection)) {
55 55
                 return null;
56 56
             }
Please login to merge, or discard this patch.