nextcloud / server

lib/private/Lock/MemcacheLockingProvider.php

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
 		} elseif ($existing === 'exclusive') {
 			return $existing;
 		} else {
-			return $existing . ' shared locks';
+			return $existing.' shared locks';
 		}
 	}
 }

Indentation +164 added lines, -164 removed lines

@@ -14,168 +14,168 @@
 use OCP\Lock\LockedException;
 
 class MemcacheLockingProvider extends AbstractLockingProvider {
-	/** @var array<string, array{time: int, ttl: int}> */
-	private array $oldTTLs = [];
-
-	public function __construct(
-		private IMemcache $memcache,
-		private ITimeFactory $timeFactory,
-		int $ttl = 3600,
-	) {
-		parent::__construct($ttl);
-	}
-
-	private function setTTL(string $path, ?int $ttl = null, mixed $compare = null): void {
-		if (is_null($ttl)) {
-			$ttl = $this->ttl;
-		}
-		if ($this->memcache instanceof IMemcacheTTL) {
-			if ($compare !== null) {
-				$this->memcache->compareSetTTL($path, $compare, $ttl);
-			} else {
-				$this->memcache->setTTL($path, $ttl);
-			}
-		}
-	}
-
-	private function getTTL(string $path): int {
-		if ($this->memcache instanceof IMemcacheTTL) {
-			$ttl = $this->memcache->getTTL($path);
-			return $ttl === false ? -1 : $ttl;
-		} else {
-			return -1;
-		}
-	}
-
-	public function isLocked(string $path, int $type): bool {
-		$lockValue = $this->memcache->get($path);
-		if ($type === self::LOCK_SHARED) {
-			return is_int($lockValue) && $lockValue > 0;
-		} elseif ($type === self::LOCK_EXCLUSIVE) {
-			return $lockValue === 'exclusive';
-		} else {
-			return false;
-		}
-	}
-
-	public function acquireLock(string $path, int $type, ?string $readablePath = null): void {
-		if ($type === self::LOCK_SHARED) {
-			// save the old TTL to for `restoreTTL`
-			$this->oldTTLs[$path] = [
-				'ttl' => $this->getTTL($path),
-				'time' => $this->timeFactory->getTime()
-			];
-			if (!$this->memcache->inc($path)) {
-				throw new LockedException($path, null, $this->getExistingLockForException($path), $readablePath);
-			}
-		} else {
-			// when getting exclusive locks, we know there are no old TTLs to restore
-			$this->memcache->add($path, 0);
-			// ttl is updated automatically when the `set` succeeds
-			if (!$this->memcache->cas($path, 0, 'exclusive')) {
-				throw new LockedException($path, null, $this->getExistingLockForException($path), $readablePath);
-			}
-			unset($this->oldTTLs[$path]);
-		}
-		$this->setTTL($path);
-		$this->markAcquire($path, $type);
-	}
-
-	public function releaseLock(string $path, int $type): void {
-		if ($type === self::LOCK_SHARED) {
-			$ownSharedLockCount = $this->getOwnSharedLockCount($path);
-			$newValue = 0;
-			if ($ownSharedLockCount === 0) { // if we are not holding the lock, don't try to release it
-				return;
-			}
-			if ($ownSharedLockCount === 1) {
-				$removed = $this->memcache->cad($path, 1); // if we're the only one having a shared lock we can remove it in one go
-				if (!$removed) { //someone else also has a shared lock, decrease only
-					$newValue = $this->memcache->dec($path);
-				}
-			} else {
-				// if we own more than one lock ourselves just decrease
-				$newValue = $this->memcache->dec($path);
-			}
-
-			if ($newValue > 0) {
-				$this->restoreTTL($path);
-			} else {
-				unset($this->oldTTLs[$path]);
-			}
-
-			// if we somehow release more locks then exists, reset the lock
-			if ($newValue < 0) {
-				$this->memcache->cad($path, $newValue);
-			}
-		} elseif ($type === self::LOCK_EXCLUSIVE) {
-			$this->memcache->cad($path, 'exclusive');
-		}
-		$this->markRelease($path, $type);
-	}
-
-	public function changeLock(string $path, int $targetType): void {
-		if ($targetType === self::LOCK_SHARED) {
-			if (!$this->memcache->cas($path, 'exclusive', 1)) {
-				throw new LockedException($path, null, $this->getExistingLockForException($path));
-			}
-		} elseif ($targetType === self::LOCK_EXCLUSIVE) {
-			// we can only change a shared lock to an exclusive if there's only a single owner of the shared lock
-			if (!$this->memcache->cas($path, 1, 'exclusive')) {
-				$this->restoreTTL($path);
-				throw new LockedException($path, null, $this->getExistingLockForException($path));
-			}
-			unset($this->oldTTLs[$path]);
-		}
-		$this->setTTL($path);
-		$this->markChange($path, $targetType);
-	}
-
-	/**
-	 * With shared locks, each time the lock is acquired, the ttl for the path is reset.
-	 *
-	 * Due to this "ttl extension" when a shared lock isn't freed correctly for any reason
-	 * the lock won't expire until no shared locks are required for the path for 1h.
-	 * This can lead to a client repeatedly trying to upload a file, and failing forever
-	 * because the lock never gets the opportunity to expire.
-	 *
-	 * To help the lock expire in this case, we lower the TTL back to what it was before we
-	 * took the shared lock *only* if nobody else got a shared lock after we did.
-	 *
-	 * This doesn't handle all cases where multiple requests are acquiring shared locks
-	 * but it should handle some of the more common ones and not hurt things further
-	 */
-	private function restoreTTL(string $path): void {
-		if (isset($this->oldTTLs[$path])) {
-			$saved = $this->oldTTLs[$path];
-			$elapsed = $this->timeFactory->getTime() - $saved['time'];
-
-			// old value to compare to when setting ttl in case someone else changes the lock in the middle of this function
-			$value = $this->memcache->get($path);
-
-			$currentTtl = $this->getTTL($path);
-
-			// what the old ttl would be given the time elapsed since we acquired the lock
-			// note that if this gets negative the key will be expired directly when we set the ttl
-			$remainingOldTtl = $saved['ttl'] - $elapsed;
-			// what the currently ttl would be if nobody else acquired a lock since we did (+1 to cover rounding errors)
-			$expectedTtl = $this->ttl - $elapsed + 1;
-
-			// check if another request has acquired a lock (and didn't release it yet)
-			if ($currentTtl <= $expectedTtl) {
-				$this->setTTL($path, $remainingOldTtl, $value);
-			}
-		}
-	}
-
-	private function getExistingLockForException(string $path): string {
-		$existing = $this->memcache->get($path);
-		if (!$existing) {
-			return 'none';
-		} elseif ($existing === 'exclusive') {
-			return $existing;
-		} else {
-			return $existing . ' shared locks';
-		}
-	}
+    /** @var array<string, array{time: int, ttl: int}> */
+    private array $oldTTLs = [];
+
+    public function __construct(
+        private IMemcache $memcache,
+        private ITimeFactory $timeFactory,
+        int $ttl = 3600,
+    ) {
+        parent::__construct($ttl);
+    }
+
+    private function setTTL(string $path, ?int $ttl = null, mixed $compare = null): void {
+        if (is_null($ttl)) {
+            $ttl = $this->ttl;
+        }
+        if ($this->memcache instanceof IMemcacheTTL) {
+            if ($compare !== null) {
+                $this->memcache->compareSetTTL($path, $compare, $ttl);
+            } else {
+                $this->memcache->setTTL($path, $ttl);
+            }
+        }
+    }
+
+    private function getTTL(string $path): int {
+        if ($this->memcache instanceof IMemcacheTTL) {
+            $ttl = $this->memcache->getTTL($path);
+            return $ttl === false ? -1 : $ttl;
+        } else {
+            return -1;
+        }
+    }
+
+    public function isLocked(string $path, int $type): bool {
+        $lockValue = $this->memcache->get($path);
+        if ($type === self::LOCK_SHARED) {
+            return is_int($lockValue) && $lockValue > 0;
+        } elseif ($type === self::LOCK_EXCLUSIVE) {
+            return $lockValue === 'exclusive';
+        } else {
+            return false;
+        }
+    }
+
+    public function acquireLock(string $path, int $type, ?string $readablePath = null): void {
+        if ($type === self::LOCK_SHARED) {
+            // save the old TTL to for `restoreTTL`
+            $this->oldTTLs[$path] = [
+                'ttl' => $this->getTTL($path),
+                'time' => $this->timeFactory->getTime()
+            ];
+            if (!$this->memcache->inc($path)) {
+                throw new LockedException($path, null, $this->getExistingLockForException($path), $readablePath);
+            }
+        } else {
+            // when getting exclusive locks, we know there are no old TTLs to restore
+            $this->memcache->add($path, 0);
+            // ttl is updated automatically when the `set` succeeds
+            if (!$this->memcache->cas($path, 0, 'exclusive')) {
+                throw new LockedException($path, null, $this->getExistingLockForException($path), $readablePath);
+            }
+            unset($this->oldTTLs[$path]);
+        }
+        $this->setTTL($path);
+        $this->markAcquire($path, $type);
+    }
+
+    public function releaseLock(string $path, int $type): void {
+        if ($type === self::LOCK_SHARED) {
+            $ownSharedLockCount = $this->getOwnSharedLockCount($path);
+            $newValue = 0;
+            if ($ownSharedLockCount === 0) { // if we are not holding the lock, don't try to release it
+                return;
+            }
+            if ($ownSharedLockCount === 1) {
+                $removed = $this->memcache->cad($path, 1); // if we're the only one having a shared lock we can remove it in one go
+                if (!$removed) { //someone else also has a shared lock, decrease only
+                    $newValue = $this->memcache->dec($path);
+                }
+            } else {
+                // if we own more than one lock ourselves just decrease
+                $newValue = $this->memcache->dec($path);
+            }
+
+            if ($newValue > 0) {
+                $this->restoreTTL($path);
+            } else {
+                unset($this->oldTTLs[$path]);
+            }
+
+            // if we somehow release more locks then exists, reset the lock
+            if ($newValue < 0) {
+                $this->memcache->cad($path, $newValue);
+            }
+        } elseif ($type === self::LOCK_EXCLUSIVE) {
+            $this->memcache->cad($path, 'exclusive');
+        }
+        $this->markRelease($path, $type);
+    }
+
+    public function changeLock(string $path, int $targetType): void {
+        if ($targetType === self::LOCK_SHARED) {
+            if (!$this->memcache->cas($path, 'exclusive', 1)) {
+                throw new LockedException($path, null, $this->getExistingLockForException($path));
+            }
+        } elseif ($targetType === self::LOCK_EXCLUSIVE) {
+            // we can only change a shared lock to an exclusive if there's only a single owner of the shared lock
+            if (!$this->memcache->cas($path, 1, 'exclusive')) {
+                $this->restoreTTL($path);
+                throw new LockedException($path, null, $this->getExistingLockForException($path));
+            }
+            unset($this->oldTTLs[$path]);
+        }
+        $this->setTTL($path);
+        $this->markChange($path, $targetType);
+    }
+
+    /**
+     * With shared locks, each time the lock is acquired, the ttl for the path is reset.
+     *
+     * Due to this "ttl extension" when a shared lock isn't freed correctly for any reason
+     * the lock won't expire until no shared locks are required for the path for 1h.
+     * This can lead to a client repeatedly trying to upload a file, and failing forever
+     * because the lock never gets the opportunity to expire.
+     *
+     * To help the lock expire in this case, we lower the TTL back to what it was before we
+     * took the shared lock *only* if nobody else got a shared lock after we did.
+     *
+     * This doesn't handle all cases where multiple requests are acquiring shared locks
+     * but it should handle some of the more common ones and not hurt things further
+     */
+    private function restoreTTL(string $path): void {
+        if (isset($this->oldTTLs[$path])) {
+            $saved = $this->oldTTLs[$path];
+            $elapsed = $this->timeFactory->getTime() - $saved['time'];
+
+            // old value to compare to when setting ttl in case someone else changes the lock in the middle of this function
+            $value = $this->memcache->get($path);
+
+            $currentTtl = $this->getTTL($path);
+
+            // what the old ttl would be given the time elapsed since we acquired the lock
+            // note that if this gets negative the key will be expired directly when we set the ttl
+            $remainingOldTtl = $saved['ttl'] - $elapsed;
+            // what the currently ttl would be if nobody else acquired a lock since we did (+1 to cover rounding errors)
+            $expectedTtl = $this->ttl - $elapsed + 1;
+
+            // check if another request has acquired a lock (and didn't release it yet)
+            if ($currentTtl <= $expectedTtl) {
+                $this->setTTL($path, $remainingOldTtl, $value);
+            }
+        }
+    }
+
+    private function getExistingLockForException(string $path): string {
+        $existing = $this->memcache->get($path);
+        if (!$existing) {
+            return 'none';
+        } elseif ($existing === 'exclusive') {
+            return $existing;
+        } else {
+            return $existing . ' shared locks';
+        }
+    }
 }

lib/public/Collaboration/Collaborators/SearchResultType.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 	 * @since 13.0.0
 	 */
 	protected function getValidatedType($type) {
-		$type = trim((string)$type);
+		$type = trim((string) $type);
 
 		if ($type === '') {
 			throw new \InvalidArgumentException('Type must not be empty');

Indentation +40 added lines, -40 removed lines

@@ -12,44 +12,44 @@
  * @since 13.0.0
  */
 class SearchResultType {
-	/** @var string */
-	protected $label;
-
-	/**
-	 * SearchResultType constructor.
-	 *
-	 * @param string $label
-	 * @since 13.0.0
-	 */
-	public function __construct($label) {
-		$this->label = $this->getValidatedType($label);
-	}
-
-	/**
-	 * @return string
-	 * @since 13.0.0
-	 */
-	public function getLabel() {
-		return $this->label;
-	}
-
-	/**
-	 * @param $type
-	 * @return string
-	 * @throws \InvalidArgumentException
-	 * @since 13.0.0
-	 */
-	protected function getValidatedType($type) {
-		$type = trim((string)$type);
-
-		if ($type === '') {
-			throw new \InvalidArgumentException('Type must not be empty');
-		}
-
-		if ($type === 'exact') {
-			throw new \InvalidArgumentException('Provided type is a reserved word');
-		}
-
-		return $type;
-	}
+    /** @var string */
+    protected $label;
+
+    /**
+     * SearchResultType constructor.
+     *
+     * @param string $label
+     * @since 13.0.0
+     */
+    public function __construct($label) {
+        $this->label = $this->getValidatedType($label);
+    }
+
+    /**
+     * @return string
+     * @since 13.0.0
+     */
+    public function getLabel() {
+        return $this->label;
+    }
+
+    /**
+     * @param $type
+     * @return string
+     * @throws \InvalidArgumentException
+     * @since 13.0.0
+     */
+    protected function getValidatedType($type) {
+        $type = trim((string)$type);
+
+        if ($type === '') {
+            throw new \InvalidArgumentException('Type must not be empty');
+        }
+
+        if ($type === 'exact') {
+            throw new \InvalidArgumentException('Provided type is a reserved word');
+        }
+
+        return $type;
+    }
 }

lib/private/IntegrityCheck/Helpers/FileAccessHelper.php

Indentation +49 added lines, -49 removed lines

@@ -34,57 +34,57 @@
  * @package OC\IntegrityCheck\Helpers
  */
 class FileAccessHelper {
-	/**
-	 * Wrapper around file_get_contents($filename, $data)
-	 *
-	 * @param string $filename
-	 * @return string|false
-	 */
-	public function file_get_contents(string $filename) {
-		return file_get_contents($filename);
-	}
+    /**
+     * Wrapper around file_get_contents($filename, $data)
+     *
+     * @param string $filename
+     * @return string|false
+     */
+    public function file_get_contents(string $filename) {
+        return file_get_contents($filename);
+    }
 
-	/**
-	 * Wrapper around file_exists($filename)
-	 *
-	 * @param string $filename
-	 * @return bool
-	 */
-	public function file_exists(string $filename): bool {
-		return file_exists($filename);
-	}
+    /**
+     * Wrapper around file_exists($filename)
+     *
+     * @param string $filename
+     * @return bool
+     */
+    public function file_exists(string $filename): bool {
+        return file_exists($filename);
+    }
 
-	/**
-	 * Wrapper around file_put_contents($filename, $data)
-	 *
-	 * @param string $filename
-	 * @param string $data
-	 * @return int
-	 * @throws \Exception
-	 */
-	public function file_put_contents(string $filename, string $data): int {
-		$bytesWritten = @file_put_contents($filename, $data);
-		if ($bytesWritten === false || $bytesWritten !== \strlen($data)) {
-			throw new \Exception('Failed to write into ' . $filename);
-		}
-		return $bytesWritten;
-	}
+    /**
+     * Wrapper around file_put_contents($filename, $data)
+     *
+     * @param string $filename
+     * @param string $data
+     * @return int
+     * @throws \Exception
+     */
+    public function file_put_contents(string $filename, string $data): int {
+        $bytesWritten = @file_put_contents($filename, $data);
+        if ($bytesWritten === false || $bytesWritten !== \strlen($data)) {
+            throw new \Exception('Failed to write into ' . $filename);
+        }
+        return $bytesWritten;
+    }
 
-	/**
-	 * @param string $path
-	 * @return bool
-	 */
-	public function is_writable(string $path): bool {
-		return is_writable($path);
-	}
+    /**
+     * @param string $path
+     * @return bool
+     */
+    public function is_writable(string $path): bool {
+        return is_writable($path);
+    }
 
-	/**
-	 * @param string $path
-	 * @throws \Exception
-	 */
-	public function assertDirectoryExists(string $path) {
-		if (!is_dir($path)) {
-			throw new \Exception('Directory ' . $path . ' does not exist.');
-		}
-	}
+    /**
+     * @param string $path
+     * @throws \Exception
+     */
+    public function assertDirectoryExists(string $path) {
+        if (!is_dir($path)) {
+            throw new \Exception('Directory ' . $path . ' does not exist.');
+        }
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
 	public function file_put_contents(string $filename, string $data): int {
 		$bytesWritten = @file_put_contents($filename, $data);
 		if ($bytesWritten === false || $bytesWritten !== \strlen($data)) {
-			throw new \Exception('Failed to write into ' . $filename);
+			throw new \Exception('Failed to write into '.$filename);
 		}
 		return $bytesWritten;
 	}
@@ -84,7 +84,7 @@ 
 	 */
 	public function assertDirectoryExists(string $path) {
 		if (!is_dir($path)) {
-			throw new \Exception('Directory ' . $path . ' does not exist.');
+			throw new \Exception('Directory '.$path.' does not exist.');
 		}
 	}
 }

lib/private/Security/FeaturePolicy/FeaturePolicy.php

Indentation +36 added lines, -36 removed lines

@@ -27,51 +27,51 @@
 namespace OC\Security\FeaturePolicy;
 
 class FeaturePolicy extends \OCP\AppFramework\Http\FeaturePolicy {
-	public function getAutoplayDomains(): array {
-		return $this->autoplayDomains;
-	}
+    public function getAutoplayDomains(): array {
+        return $this->autoplayDomains;
+    }
 
-	public function setAutoplayDomains(array $autoplayDomains): void {
-		$this->autoplayDomains = $autoplayDomains;
-	}
+    public function setAutoplayDomains(array $autoplayDomains): void {
+        $this->autoplayDomains = $autoplayDomains;
+    }
 
-	public function getCameraDomains(): array {
-		return $this->cameraDomains;
-	}
+    public function getCameraDomains(): array {
+        return $this->cameraDomains;
+    }
 
-	public function setCameraDomains(array $cameraDomains): void {
-		$this->cameraDomains = $cameraDomains;
-	}
+    public function setCameraDomains(array $cameraDomains): void {
+        $this->cameraDomains = $cameraDomains;
+    }
 
-	public function getFullscreenDomains(): array {
-		return $this->fullscreenDomains;
-	}
+    public function getFullscreenDomains(): array {
+        return $this->fullscreenDomains;
+    }
 
-	public function setFullscreenDomains(array $fullscreenDomains): void {
-		$this->fullscreenDomains = $fullscreenDomains;
-	}
+    public function setFullscreenDomains(array $fullscreenDomains): void {
+        $this->fullscreenDomains = $fullscreenDomains;
+    }
 
-	public function getGeolocationDomains(): array {
-		return $this->geolocationDomains;
-	}
+    public function getGeolocationDomains(): array {
+        return $this->geolocationDomains;
+    }
 
-	public function setGeolocationDomains(array $geolocationDomains): void {
-		$this->geolocationDomains = $geolocationDomains;
-	}
+    public function setGeolocationDomains(array $geolocationDomains): void {
+        $this->geolocationDomains = $geolocationDomains;
+    }
 
-	public function getMicrophoneDomains(): array {
-		return $this->microphoneDomains;
-	}
+    public function getMicrophoneDomains(): array {
+        return $this->microphoneDomains;
+    }
 
-	public function setMicrophoneDomains(array $microphoneDomains): void {
-		$this->microphoneDomains = $microphoneDomains;
-	}
+    public function setMicrophoneDomains(array $microphoneDomains): void {
+        $this->microphoneDomains = $microphoneDomains;
+    }
 
-	public function getPaymentDomains(): array {
-		return $this->paymentDomains;
-	}
+    public function getPaymentDomains(): array {
+        return $this->paymentDomains;
+    }
 
-	public function setPaymentDomains(array $paymentDomains): void {
-		$this->paymentDomains = $paymentDomains;
-	}
+    public function setPaymentDomains(array $paymentDomains): void {
+        $this->paymentDomains = $paymentDomains;
+    }
 }

lib/private/Security/CSRF/CsrfToken.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 	public function getEncryptedValue(): string {
 		if ($this->encryptedValue === '') {
 			$sharedSecret = random_bytes(\strlen($this->value));
-			$this->encryptedValue = base64_encode($this->value ^ $sharedSecret) . ':' . base64_encode($sharedSecret);
+			$this->encryptedValue = base64_encode($this->value ^ $sharedSecret).':'.base64_encode($sharedSecret);
 		}
 
 		return $this->encryptedValue;

Indentation +32 added lines, -32 removed lines

@@ -17,40 +17,40 @@
  * @package OC\Security\CSRF
  */
 class CsrfToken {
-	private string $encryptedValue = '';
+    private string $encryptedValue = '';
 
-	/**
-	 * @param string $value Value of the token. Can be encrypted or not encrypted.
-	 */
-	public function __construct(
-		private string $value,
-	) {
-	}
+    /**
+     * @param string $value Value of the token. Can be encrypted or not encrypted.
+     */
+    public function __construct(
+        private string $value,
+    ) {
+    }
 
-	/**
-	 * Encrypted value of the token. This is used to mitigate BREACH alike
-	 * vulnerabilities. For display measures do use this functionality.
-	 */
-	public function getEncryptedValue(): string {
-		if ($this->encryptedValue === '') {
-			$sharedSecret = random_bytes(\strlen($this->value));
-			$this->encryptedValue = base64_encode($this->value ^ $sharedSecret) . ':' . base64_encode($sharedSecret);
-		}
+    /**
+     * Encrypted value of the token. This is used to mitigate BREACH alike
+     * vulnerabilities. For display measures do use this functionality.
+     */
+    public function getEncryptedValue(): string {
+        if ($this->encryptedValue === '') {
+            $sharedSecret = random_bytes(\strlen($this->value));
+            $this->encryptedValue = base64_encode($this->value ^ $sharedSecret) . ':' . base64_encode($sharedSecret);
+        }
 
-		return $this->encryptedValue;
-	}
+        return $this->encryptedValue;
+    }
 
-	/**
-	 * The unencrypted value of the token. Used for decrypting an already
-	 * encrypted token.
-	 */
-	public function getDecryptedValue(): string {
-		$token = explode(':', $this->value);
-		if (\count($token) !== 2) {
-			return '';
-		}
-		$obfuscatedToken = $token[0];
-		$secret = $token[1];
-		return base64_decode($obfuscatedToken) ^ base64_decode($secret);
-	}
+    /**
+     * The unencrypted value of the token. Used for decrypting an already
+     * encrypted token.
+     */
+    public function getDecryptedValue(): string {
+        $token = explode(':', $this->value);
+        if (\count($token) !== 2) {
+            return '';
+        }
+        $obfuscatedToken = $token[0];
+        $secret = $token[1];
+        return base64_decode($obfuscatedToken) ^ base64_decode($secret);
+    }
 }

lib/private/Command/QueueBus.php

Indentation +40 added lines, -40 removed lines

@@ -27,48 +27,48 @@
 use OCP\Command\ICommand;
 
 class QueueBus implements IBus {
-	/**
-	 * @var ICommand[]|callable[]
-	 */
-	private $queue = [];
+    /**
+     * @var ICommand[]|callable[]
+     */
+    private $queue = [];
 
-	/**
-	 * Schedule a command to be fired
-	 *
-	 * @param \OCP\Command\ICommand | callable $command
-	 */
-	public function push($command) {
-		$this->queue[] = $command;
-	}
+    /**
+     * Schedule a command to be fired
+     *
+     * @param \OCP\Command\ICommand | callable $command
+     */
+    public function push($command) {
+        $this->queue[] = $command;
+    }
 
-	/**
-	 * Require all commands using a trait to be run synchronous
-	 *
-	 * @param string $trait
-	 */
-	public function requireSync($trait) {
-	}
+    /**
+     * Require all commands using a trait to be run synchronous
+     *
+     * @param string $trait
+     */
+    public function requireSync($trait) {
+    }
 
-	/**
-	 * @param \OCP\Command\ICommand | callable $command
-	 */
-	private function runCommand($command) {
-		if ($command instanceof ICommand) {
-			// ensure the command can be serialized
-			$serialized = serialize($command);
-			if (strlen($serialized) > 4000) {
-				throw new \InvalidArgumentException('Trying to push a command which serialized form can not be stored in the database (>4000 character)');
-			}
-			$unserialized = unserialize($serialized);
-			$unserialized->handle();
-		} else {
-			$command();
-		}
-	}
+    /**
+     * @param \OCP\Command\ICommand | callable $command
+     */
+    private function runCommand($command) {
+        if ($command instanceof ICommand) {
+            // ensure the command can be serialized
+            $serialized = serialize($command);
+            if (strlen($serialized) > 4000) {
+                throw new \InvalidArgumentException('Trying to push a command which serialized form can not be stored in the database (>4000 character)');
+            }
+            $unserialized = unserialize($serialized);
+            $unserialized->handle();
+        } else {
+            $command();
+        }
+    }
 
-	public function run() {
-		while ($command = array_shift($this->queue)) {
-			$this->runCommand($command);
-		}
-	}
+    public function run() {
+        while ($command = array_shift($this->queue)) {
+            $this->runCommand($command);
+        }
+    }
 }

apps/user_ldap/ajax/setConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 \OC_JSON::checkAppEnabled('user_ldap');
 \OC_JSON::callCheck();
 
-$prefix = (string)$_POST['ldap_serverconfig_chooser'];
+$prefix = (string) $_POST['ldap_serverconfig_chooser'];
 
 // Checkboxes are not submitted, when they are unchecked. Set them manually.
 // only legacy checkboxes (Advanced and Expert tab) need to be handled here,

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
 // only legacy checkboxes (Advanced and Expert tab) need to be handled here,
 // the Wizard-like tabs handle it on their own
 $chkboxes = ['ldap_configuration_active', 'ldap_override_main_server',
-	'ldap_turn_off_cert_check'];
+    'ldap_turn_off_cert_check'];
 foreach ($chkboxes as $boxid) {
-	if (!isset($_POST[$boxid])) {
-		$_POST[$boxid] = 0;
-	}
+    if (!isset($_POST[$boxid])) {
+        $_POST[$boxid] = 0;
+    }
 }
 
 $ldapWrapper = new LDAP();

apps/dav/lib/Command/ListCalendars.php

Spacing +3 added lines, -3 removed lines

@@ -77,7 +77,7 @@ 
 			}
 
 			$readOnly = false;
-			$readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
+			$readOnlyIndex = '{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}read-only';
 			if (isset($calendar[$readOnlyIndex])) {
 				$readOnly = $calendar[$readOnlyIndex];
 			}
@@ -85,8 +85,8 @@ 
 			$calendarTableData[] = [
 				$calendar['uri'],
 				$calendar['{DAV:}displayname'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
+				$calendar['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}owner-principal'],
+				$calendar['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD.'}owner-displayname'],
 				$readOnly ? ' x ' : ' ✓ ',
 			];
 		}

Indentation +49 added lines, -49 removed lines

@@ -15,61 +15,61 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ListCalendars extends Command {
-	public function __construct(
-		protected IUserManager $userManager,
-		private CalDavBackend $caldav,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        private CalDavBackend $caldav,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('dav:list-calendars')
-			->setDescription('List all calendars of a user')
-			->addArgument('uid',
-				InputArgument::REQUIRED,
-				'User for whom all calendars will be listed');
-	}
+    protected function configure(): void {
+        $this
+            ->setName('dav:list-calendars')
+            ->setDescription('List all calendars of a user')
+            ->addArgument('uid',
+                InputArgument::REQUIRED,
+                'User for whom all calendars will be listed');
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $input->getArgument('uid');
-		if (!$this->userManager->userExists($user)) {
-			throw new \InvalidArgumentException("User <$user> is unknown.");
-		}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $user = $input->getArgument('uid');
+        if (!$this->userManager->userExists($user)) {
+            throw new \InvalidArgumentException("User <$user> is unknown.");
+        }
 
-		$calendars = $this->caldav->getCalendarsForUser("principals/users/$user");
+        $calendars = $this->caldav->getCalendarsForUser("principals/users/$user");
 
-		$calendarTableData = [];
-		foreach ($calendars as $calendar) {
-			// skip birthday calendar
-			if ($calendar['uri'] === BirthdayService::BIRTHDAY_CALENDAR_URI) {
-				continue;
-			}
+        $calendarTableData = [];
+        foreach ($calendars as $calendar) {
+            // skip birthday calendar
+            if ($calendar['uri'] === BirthdayService::BIRTHDAY_CALENDAR_URI) {
+                continue;
+            }
 
-			$readOnly = false;
-			$readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
-			if (isset($calendar[$readOnlyIndex])) {
-				$readOnly = $calendar[$readOnlyIndex];
-			}
+            $readOnly = false;
+            $readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
+            if (isset($calendar[$readOnlyIndex])) {
+                $readOnly = $calendar[$readOnlyIndex];
+            }
 
-			$calendarTableData[] = [
-				$calendar['uri'],
-				$calendar['{DAV:}displayname'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
-				$readOnly ? ' x ' : ' ✓ ',
-			];
-		}
+            $calendarTableData[] = [
+                $calendar['uri'],
+                $calendar['{DAV:}displayname'],
+                $calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
+                $calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
+                $readOnly ? ' x ' : ' ✓ ',
+            ];
+        }
 
-		if (count($calendarTableData) > 0) {
-			$table = new Table($output);
-			$table->setHeaders(['URI', 'Displayname', 'Owner principal', 'Owner displayname', 'Writable'])
-				->setRows($calendarTableData);
+        if (count($calendarTableData) > 0) {
+            $table = new Table($output);
+            $table->setHeaders(['URI', 'Displayname', 'Owner principal', 'Owner displayname', 'Writable'])
+                ->setRows($calendarTableData);
 
-			$table->render();
-		} else {
-			$output->writeln("<info>User <$user> has no calendars</info>");
-		}
-		return self::SUCCESS;
-	}
+            $table->render();
+        } else {
+            $output->writeln("<info>User <$user> has no calendars</info>");
+        }
+        return self::SUCCESS;
+    }
 }

apps/files_external/lib/Lib/Backend/SFTP_Key.php

Indentation +15 added lines, -15 removed lines

@@ -30,19 +30,19 @@
 use OCP\IL10N;
 
 class SFTP_Key extends Backend {
-	public function __construct(IL10N $l, RSA $legacyAuth, SFTP $sftpBackend) {
-		$this
-			->setIdentifier('\OC\Files\Storage\SFTP_Key')
-			->setStorageClass('\OCA\Files_External\Lib\Storage\SFTP')
-			->setText($l->t('SFTP with secret key login'))
-			->addParameters([
-				new DefinitionParameter('host', $l->t('Host')),
-				(new DefinitionParameter('root', $l->t('Remote subfolder')))
-					->setFlag(DefinitionParameter::FLAG_OPTIONAL),
-			])
-			->addAuthScheme(AuthMechanism::SCHEME_PUBLICKEY)
-			->setLegacyAuthMechanism($legacyAuth)
-			->deprecateTo($sftpBackend)
-		;
-	}
+    public function __construct(IL10N $l, RSA $legacyAuth, SFTP $sftpBackend) {
+        $this
+            ->setIdentifier('\OC\Files\Storage\SFTP_Key')
+            ->setStorageClass('\OCA\Files_External\Lib\Storage\SFTP')
+            ->setText($l->t('SFTP with secret key login'))
+            ->addParameters([
+                new DefinitionParameter('host', $l->t('Host')),
+                (new DefinitionParameter('root', $l->t('Remote subfolder')))
+                    ->setFlag(DefinitionParameter::FLAG_OPTIONAL),
+            ])
+            ->addAuthScheme(AuthMechanism::SCHEME_PUBLICKEY)
+            ->setLegacyAuthMechanism($legacyAuth)
+            ->deprecateTo($sftpBackend)
+        ;
+    }
 }