chore: Fix code style

Signed-off-by: Ferdinand Thiessen <[email protected]>

Author: susnux

lib/DAV/CorePlugin.php

@@ -590,7 +590,7 @@ public function httpMove(RequestInterface $request, ResponseInterface $response)
         $moveInfo = $this->server->getCopyAndMoveInfo($request);
 
         // MOVE does only allow "infinity" every other header value is considered invalid
-        if ($moveInfo['depth'] !== 'infinity') {
+        if ('infinity' !== $moveInfo['depth']) {
             throw new BadRequest('The HTTP Depth header must only contain "infinity" for MOVE');
         }
 

lib/DAV/ICopyTarget.php

@@ -28,12 +28,12 @@ interface ICopyTarget extends ICollection
      * is that the copy was successful.
      * If you return false, sabre/dav will handle the copy itself.
      *
-     * @param string $targetName new local file/collection name
-     * @param string $sourcePath Full path to source node
-     * @param INode  $sourceNode Source node itself
-     * @param string|int $depth How many level of children to copy.
-     *                          The value can be 'infinity' or a positiv number including zero.
-     *                          Zero means to only copy a shallow collection with props but without children.
+     * @param string     $targetName new local file/collection name
+     * @param string     $sourcePath Full path to source node
+     * @param INode      $sourceNode Source node itself
+     * @param string|int $depth      How many level of children to copy.
+     *                               The value can be 'infinity' or a positiv number including zero.
+     *                               Zero means to only copy a shallow collection with props but without children.
      *
      * @return bool
      */

lib/DAV/Server.php

@@ -728,8 +728,8 @@ public function getCopyAndMoveInfo(RequestInterface $request)
 
         // Depth of inifinty is valid for MOVE and COPY. If it is not set the RFC requires to act like it was 'infinity'.
         $depth = strtolower($request->getHeader('Depth') ?? 'infinity');
-        if ($depth !== 'infinity' && is_numeric($depth)) {
-            $depth = (int)$depth;
+        if ('infinity' !== $depth && is_numeric($depth)) {
+            $depth = (int) $depth;
             if ($depth < 0) {
                 throw new Exception\BadRequest('The HTTP Depth header may only be "infinity", 0 or a positiv number');
             }

lib/DAV/Tree.php

@@ -135,11 +135,11 @@ public function nodeExists($path)
     /**
      * Copies a file from path to another.
      *
-     * @param string $sourcePath      The source location
-     * @param string $destinationPath The full destination path
-     * @param int|string $depth How much levle of children to copy.
-     *                          The value can be 'infinity' or a positiv integer, including zero.
-     *                          Zero means only copy the collection without children but with its properties.
+     * @param string     $sourcePath      The source location
+     * @param string     $destinationPath The full destination path
+     * @param int|string $depth           How much levle of children to copy.
+     *                                    The value can be 'infinity' or a positiv integer, including zero.
+     *                                    Zero means only copy the collection without children but with its properties.
      */
     public function copy($sourcePath, $destinationPath, $depth = 'infinity')
     {
@@ -310,8 +310,8 @@ public function getMultipleNodes($paths)
     /**
      * copyNode.
      *
-     * @param string $destinationName
-     * @param int|string $depth How many children of the node to copy
+     * @param string     $destinationName
+     * @param int|string $depth           How many children of the node to copy
      */
     protected function copyNode(INode $source, ICollection $destinationParent, ?string $destinationName = null, $depth = 'infinity')
     {
@@ -338,9 +338,9 @@ protected function copyNode(INode $source, ICollection $destinationParent, ?stri
             $destination = $destinationParent->getChild($destinationName);
 
             // Copy children if depth is not zero
-            if ($depth !== 0) {
+            if (0 !== $depth) {
                 // Adjust next depth for children (keep 'infinity' or decrease)
-                $depth = $depth === 'infinity' ? 'infinity' : $depth - 1;
+                $depth = 'infinity' === $depth ? 'infinity' : $depth - 1;
                 $destination = $destinationParent->getChild($destinationName);
                 foreach ($source->getChildren() as $child) {
                     $this->copyNode($child, $destination, null, $depth);

tests/Sabre/DAV/CorePluginTest.php

@@ -16,18 +16,21 @@ public function testGetInfo()
         self::assertEquals('core', $corePlugin->getPluginInfo()['name']);
     }
 
-    public function moveInvalidDepthHeaderProvider() {
+    public function moveInvalidDepthHeaderProvider()
+    {
         return [
             [0],
             [1],
         ];
     }
 
     /**
-     * MOVE does only allow "infinity" every other header value is considered invalid
+     * MOVE does only allow "infinity" every other header value is considered invalid.
+     *
      * @dataProvider moveInvalidDepthHeaderProvider
      */
-    public function testMoveWithInvalidDepth($depthHeader) {
+    public function testMoveWithInvalidDepth($depthHeader)
+    {
         $request = new HTTP\Request('MOVE', '/path/');
         $response = new HTTP\Response();
 
@@ -45,9 +48,10 @@ public function testMoveWithInvalidDepth($depthHeader) {
     }
 
     /**
-     * MOVE does only allow "infinity" every other header value is considered invalid
+     * MOVE does only allow "infinity" every other header value is considered invalid.
      */
-    public function testMoveSupportsDepth() {
+    public function testMoveSupportsDepth()
+    {
         $request = new HTTP\Request('MOVE', '/path/');
         $response = new HTTP\Response();
 

tests/Sabre/DAV/Mock/PropertiesCollection.php

@@ -49,6 +49,7 @@ public function propPatch(PropPatch $proppatch)
                     foreach ($updateProperties as $k => $v) {
                         $this->properties[$k] = $v;
                     }
+
                     return true;
                 case 'updatepropsarray':
                     $r = [];
@@ -81,7 +82,7 @@ public function propPatch(PropPatch $proppatch)
      */
     public function getProperties($requestedProperties)
     {
-        if (count($requestedProperties) === 0) {
+        if (0 === count($requestedProperties)) {
             return $this->properties;
         }
 
@@ -97,7 +98,7 @@ public function getProperties($requestedProperties)
     }
 
     /**
-     * Creates a new subdirectory. (Override to ensure props are preserved)
+     * Creates a new subdirectory. (Override to ensure props are preserved).
      *
      * @param string $name
      */