nextcloud / server

lib/public/AppFramework/Http/EmptyFeaturePolicy.php

Spacing +6 added lines, -6 removed lines

@@ -139,42 +139,42 @@
 		if (empty($this->autoplayDomains)) {
 			$policy .= "autoplay 'none';";
 		} else {
-			$policy .= 'autoplay ' . implode(' ', $this->autoplayDomains);
+			$policy .= 'autoplay '.implode(' ', $this->autoplayDomains);
 			$policy .= ';';
 		}
 
 		if (empty($this->cameraDomains)) {
 			$policy .= "camera 'none';";
 		} else {
-			$policy .= 'camera ' . implode(' ', $this->cameraDomains);
+			$policy .= 'camera '.implode(' ', $this->cameraDomains);
 			$policy .= ';';
 		}
 
 		if (empty($this->fullscreenDomains)) {
 			$policy .= "fullscreen 'none';";
 		} else {
-			$policy .= 'fullscreen ' . implode(' ', $this->fullscreenDomains);
+			$policy .= 'fullscreen '.implode(' ', $this->fullscreenDomains);
 			$policy .= ';';
 		}
 
 		if (empty($this->geolocationDomains)) {
 			$policy .= "geolocation 'none';";
 		} else {
-			$policy .= 'geolocation ' . implode(' ', $this->geolocationDomains);
+			$policy .= 'geolocation '.implode(' ', $this->geolocationDomains);
 			$policy .= ';';
 		}
 
 		if (empty($this->microphoneDomains)) {
 			$policy .= "microphone 'none';";
 		} else {
-			$policy .= 'microphone ' . implode(' ', $this->microphoneDomains);
+			$policy .= 'microphone '.implode(' ', $this->microphoneDomains);
 			$policy .= ';';
 		}
 
 		if (empty($this->paymentDomains)) {
 			$policy .= "payment 'none';";
 		} else {
-			$policy .= 'payment ' . implode(' ', $this->paymentDomains);
+			$policy .= 'payment '.implode(' ', $this->paymentDomains);
 			$policy .= ';';
 		}
 

Indentation +143 added lines, -143 removed lines

@@ -36,147 +36,147 @@
  * @since 17.0.0
  */
 class EmptyFeaturePolicy {
-	/** @var string[] of allowed domains to autoplay media */
-	protected $autoplayDomains = null;
-
-	/** @var string[] of allowed domains that can access the camera */
-	protected $cameraDomains = null;
-
-	/** @var string[] of allowed domains that can use fullscreen */
-	protected $fullscreenDomains = null;
-
-	/** @var string[] of allowed domains that can use the geolocation of the device */
-	protected $geolocationDomains = null;
-
-	/** @var string[] of allowed domains that can use the microphone */
-	protected $microphoneDomains = null;
-
-	/** @var string[] of allowed domains that can use the payment API */
-	protected $paymentDomains = null;
-
-	/**
-	 * Allows to use autoplay from a specific domain. Use * to allow from all domains.
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedAutoplayDomain(string $domain): self {
-		$this->autoplayDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Allows to use the camera on a specific domain. Use * to allow from all domains
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedCameraDomain(string $domain): self {
-		$this->cameraDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Allows the full screen functionality to be used on a specific domain. Use * to allow from all domains
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedFullScreenDomain(string $domain): self {
-		$this->fullscreenDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Allows to use the geolocation on a specific domain. Use * to allow from all domains
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedGeoLocationDomain(string $domain): self {
-		$this->geolocationDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Allows to use the microphone on a specific domain. Use * to allow from all domains
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedMicrophoneDomain(string $domain): self {
-		$this->microphoneDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Allows to use the payment API on a specific domain. Use * to allow from all domains
-	 *
-	 * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
-	 * @return $this
-	 * @since 17.0.0
-	 */
-	public function addAllowedPaymentDomain(string $domain): self {
-		$this->paymentDomains[] = $domain;
-		return $this;
-	}
-
-	/**
-	 * Get the generated Feature-Policy as a string
-	 *
-	 * @return string
-	 * @since 17.0.0
-	 */
-	public function buildPolicy(): string {
-		$policy = '';
-
-		if (empty($this->autoplayDomains)) {
-			$policy .= "autoplay 'none';";
-		} else {
-			$policy .= 'autoplay ' . implode(' ', $this->autoplayDomains);
-			$policy .= ';';
-		}
-
-		if (empty($this->cameraDomains)) {
-			$policy .= "camera 'none';";
-		} else {
-			$policy .= 'camera ' . implode(' ', $this->cameraDomains);
-			$policy .= ';';
-		}
-
-		if (empty($this->fullscreenDomains)) {
-			$policy .= "fullscreen 'none';";
-		} else {
-			$policy .= 'fullscreen ' . implode(' ', $this->fullscreenDomains);
-			$policy .= ';';
-		}
-
-		if (empty($this->geolocationDomains)) {
-			$policy .= "geolocation 'none';";
-		} else {
-			$policy .= 'geolocation ' . implode(' ', $this->geolocationDomains);
-			$policy .= ';';
-		}
-
-		if (empty($this->microphoneDomains)) {
-			$policy .= "microphone 'none';";
-		} else {
-			$policy .= 'microphone ' . implode(' ', $this->microphoneDomains);
-			$policy .= ';';
-		}
-
-		if (empty($this->paymentDomains)) {
-			$policy .= "payment 'none';";
-		} else {
-			$policy .= 'payment ' . implode(' ', $this->paymentDomains);
-			$policy .= ';';
-		}
-
-		return rtrim($policy, ';');
-	}
+    /** @var string[] of allowed domains to autoplay media */
+    protected $autoplayDomains = null;
+
+    /** @var string[] of allowed domains that can access the camera */
+    protected $cameraDomains = null;
+
+    /** @var string[] of allowed domains that can use fullscreen */
+    protected $fullscreenDomains = null;
+
+    /** @var string[] of allowed domains that can use the geolocation of the device */
+    protected $geolocationDomains = null;
+
+    /** @var string[] of allowed domains that can use the microphone */
+    protected $microphoneDomains = null;
+
+    /** @var string[] of allowed domains that can use the payment API */
+    protected $paymentDomains = null;
+
+    /**
+     * Allows to use autoplay from a specific domain. Use * to allow from all domains.
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedAutoplayDomain(string $domain): self {
+        $this->autoplayDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Allows to use the camera on a specific domain. Use * to allow from all domains
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedCameraDomain(string $domain): self {
+        $this->cameraDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Allows the full screen functionality to be used on a specific domain. Use * to allow from all domains
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedFullScreenDomain(string $domain): self {
+        $this->fullscreenDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Allows to use the geolocation on a specific domain. Use * to allow from all domains
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedGeoLocationDomain(string $domain): self {
+        $this->geolocationDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Allows to use the microphone on a specific domain. Use * to allow from all domains
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedMicrophoneDomain(string $domain): self {
+        $this->microphoneDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Allows to use the payment API on a specific domain. Use * to allow from all domains
+     *
+     * @param string $domain Domain to whitelist. Any passed value needs to be properly sanitized.
+     * @return $this
+     * @since 17.0.0
+     */
+    public function addAllowedPaymentDomain(string $domain): self {
+        $this->paymentDomains[] = $domain;
+        return $this;
+    }
+
+    /**
+     * Get the generated Feature-Policy as a string
+     *
+     * @return string
+     * @since 17.0.0
+     */
+    public function buildPolicy(): string {
+        $policy = '';
+
+        if (empty($this->autoplayDomains)) {
+            $policy .= "autoplay 'none';";
+        } else {
+            $policy .= 'autoplay ' . implode(' ', $this->autoplayDomains);
+            $policy .= ';';
+        }
+
+        if (empty($this->cameraDomains)) {
+            $policy .= "camera 'none';";
+        } else {
+            $policy .= 'camera ' . implode(' ', $this->cameraDomains);
+            $policy .= ';';
+        }
+
+        if (empty($this->fullscreenDomains)) {
+            $policy .= "fullscreen 'none';";
+        } else {
+            $policy .= 'fullscreen ' . implode(' ', $this->fullscreenDomains);
+            $policy .= ';';
+        }
+
+        if (empty($this->geolocationDomains)) {
+            $policy .= "geolocation 'none';";
+        } else {
+            $policy .= 'geolocation ' . implode(' ', $this->geolocationDomains);
+            $policy .= ';';
+        }
+
+        if (empty($this->microphoneDomains)) {
+            $policy .= "microphone 'none';";
+        } else {
+            $policy .= 'microphone ' . implode(' ', $this->microphoneDomains);
+            $policy .= ';';
+        }
+
+        if (empty($this->paymentDomains)) {
+            $policy .= "payment 'none';";
+        } else {
+            $policy .= 'payment ' . implode(' ', $this->paymentDomains);
+            $policy .= ';';
+        }
+
+        return rtrim($policy, ';');
+    }
 }

lib/public/AppFramework/Http/FeaturePolicy.php

Indentation +14 added lines, -14 removed lines

@@ -37,23 +37,23 @@
  * @since 17.0.0
  */
 class FeaturePolicy extends EmptyFeaturePolicy {
-	protected $autoplayDomains = [
-		'\'self\'',
-	];
+    protected $autoplayDomains = [
+        '\'self\'',
+    ];
 
-	/** @var string[] of allowed domains that can access the camera */
-	protected $cameraDomains = [];
+    /** @var string[] of allowed domains that can access the camera */
+    protected $cameraDomains = [];
 
-	protected $fullscreenDomains = [
-		'\'self\'',
-	];
+    protected $fullscreenDomains = [
+        '\'self\'',
+    ];
 
-	/** @var string[] of allowed domains that can use the geolocation of the device */
-	protected $geolocationDomains = [];
+    /** @var string[] of allowed domains that can use the geolocation of the device */
+    protected $geolocationDomains = [];
 
-	/** @var string[] of allowed domains that can use the microphone */
-	protected $microphoneDomains = [];
+    /** @var string[] of allowed domains that can use the microphone */
+    protected $microphoneDomains = [];
 
-	/** @var string[] of allowed domains that can use the payment API */
-	protected $paymentDomains = [];
+    /** @var string[] of allowed domains that can use the payment API */
+    protected $paymentDomains = [];
 }

apps/dav/lib/CalDAV/Reminder/NotificationTypeDoesNotExistException.php

Indentation +10 added lines, -10 removed lines

@@ -26,15 +26,15 @@
 
 class NotificationTypeDoesNotExistException extends \Exception {
 
-	/**
-	 * NotificationTypeDoesNotExistException constructor.
-	 *
-	 * @since 16.0.0
-	 *
-	 * @param string $type ReminderType
-	 */
-	public function __construct(string $type) {
-		parent::__construct("Type $type is not an accepted type of notification");
-	}
+    /**
+     * NotificationTypeDoesNotExistException constructor.
+     *
+     * @since 16.0.0
+     *
+     * @param string $type ReminderType
+     */
+    public function __construct(string $type) {
+        parent::__construct("Type $type is not an accepted type of notification");
+    }
 
 }

apps/dav/lib/CalDAV/Reminder/NotificationProvider/ProviderNotAvailableException.php

Indentation +10 added lines, -10 removed lines

@@ -25,15 +25,15 @@
 
 class ProviderNotAvailableException extends \Exception {
 
-	/**
-	 * ProviderNotAvailableException constructor.
-	 *
-	 * @since 16.0.0
-	 *
-	 * @param string $type ReminderType
-	 */
-	public function __construct(string $type) {
-		parent::__construct("No notification provider for type $type available");
-	}
+    /**
+     * ProviderNotAvailableException constructor.
+     *
+     * @since 16.0.0
+     *
+     * @param string $type ReminderType
+     */
+    public function __construct(string $type) {
+        parent::__construct("No notification provider for type $type available");
+    }
 
 }

apps/dav/lib/CalDAV/Reminder/NotificationProvider/AudioProvider.php

Indentation +2 added lines, -2 removed lines

@@ -32,6 +32,6 @@
  */
 class AudioProvider extends PushProvider {
 
-	/** @var string */
-	public const NOTIFICATION_TYPE = 'AUDIO';
+    /** @var string */
+    public const NOTIFICATION_TYPE = 'AUDIO';
 }

lib/private/Preview/ProviderV1Adapter.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
 	}
 
 	public function getMimeType(): string {
-		return (string)$this->providerV1->getMimeType();
+		return (string) $this->providerV1->getMimeType();
 	}
 
 	public function isAvailable(FileInfo $file): bool {
-		return (bool)$this->providerV1->isAvailable($file);
+		return (bool) $this->providerV1->isAvailable($file);
 	}
 
 	public function getThumbnail(File $file, int $maxX, int $maxY): ?IImage {

Indentation +20 added lines, -20 removed lines

@@ -34,30 +34,30 @@
 use OCP\Preview\IProviderV2;
 
 class ProviderV1Adapter implements IProviderV2 {
-	private $providerV1;
+    private $providerV1;
 
-	public function __construct(IProvider $providerV1) {
-		$this->providerV1 = $providerV1;
-	}
+    public function __construct(IProvider $providerV1) {
+        $this->providerV1 = $providerV1;
+    }
 
-	public function getMimeType(): string {
-		return (string)$this->providerV1->getMimeType();
-	}
+    public function getMimeType(): string {
+        return (string)$this->providerV1->getMimeType();
+    }
 
-	public function isAvailable(FileInfo $file): bool {
-		return (bool)$this->providerV1->isAvailable($file);
-	}
+    public function isAvailable(FileInfo $file): bool {
+        return (bool)$this->providerV1->isAvailable($file);
+    }
 
-	public function getThumbnail(File $file, int $maxX, int $maxY): ?IImage {
-		[$view, $path] = $this->getViewAndPath($file);
-		$thumbnail = $this->providerV1->getThumbnail($path, $maxX, $maxY, false, $view);
-		return $thumbnail === false ? null: $thumbnail;
-	}
+    public function getThumbnail(File $file, int $maxX, int $maxY): ?IImage {
+        [$view, $path] = $this->getViewAndPath($file);
+        $thumbnail = $this->providerV1->getThumbnail($path, $maxX, $maxY, false, $view);
+        return $thumbnail === false ? null: $thumbnail;
+    }
 
-	private function getViewAndPath(File $file) {
-		$view = new View(dirname($file->getPath()));
-		$path = $file->getName();
+    private function getViewAndPath(File $file) {
+        $view = new View(dirname($file->getPath()));
+        $path = $file->getName();
 
-		return [$view, $path];
-	}
+        return [$view, $path];
+    }
 }

apps/workflowengine/lib/Check/FileSystemTags.php

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 	}
 
 	public function supportedEntities(): array {
-		return [ File::class ];
+		return [File::class];
 	}
 
 	public function isAvailableForScope(int $scope): bool {

Indentation +134 added lines, -134 removed lines

@@ -21,138 +21,138 @@
 use OCP\WorkflowEngine\IFileCheck;
 
 class FileSystemTags implements ICheck, IFileCheck {
-	use TFileCheck;
-
-	/** @var array */
-	protected $fileIds;
-
-	/** @var array */
-	protected $fileSystemTags;
-
-	public function __construct(
-		protected IL10N $l,
-		protected ISystemTagManager $systemTagManager,
-		protected ISystemTagObjectMapper $systemTagObjectMapper,
-		protected IUserSession $userSession,
-		protected IGroupManager $groupManager,
-	) {
-	}
-
-	/**
-	 * @param string $operator
-	 * @param string $value
-	 * @return bool
-	 */
-	public function executeCheck($operator, $value) {
-		$systemTags = $this->getSystemTags();
-		return ($operator === 'is') === in_array($value, $systemTags);
-	}
-
-	/**
-	 * @param string $operator
-	 * @param string $value
-	 * @throws \UnexpectedValueException
-	 */
-	public function validateCheck($operator, $value) {
-		if (!in_array($operator, ['is', '!is'])) {
-			throw new \UnexpectedValueException($this->l->t('The given operator is invalid'), 1);
-		}
-
-		try {
-			$tags = $this->systemTagManager->getTagsByIds($value);
-
-			$user = $this->userSession->getUser();
-			$isAdmin = $user instanceof IUser && $this->groupManager->isAdmin($user->getUID());
-
-			if (!$isAdmin) {
-				foreach ($tags as $tag) {
-					if (!$tag->isUserVisible()) {
-						throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 4);
-					}
-				}
-			}
-		} catch (TagNotFoundException $e) {
-			throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 2);
-		} catch (\InvalidArgumentException $e) {
-			throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 3);
-		}
-	}
-
-	/**
-	 * Get the ids of the assigned system tags
-	 * @return string[]
-	 */
-	protected function getSystemTags() {
-		$cache = $this->storage->getCache();
-		$fileIds = $this->getFileIds($cache, $this->path, !$this->storage->instanceOfStorage(IHomeStorage::class) || $this->storage->instanceOfStorage(SharedStorage::class));
-
-		$systemTags = [];
-		foreach ($fileIds as $i => $fileId) {
-			if (isset($this->fileSystemTags[$fileId])) {
-				$systemTags[] = $this->fileSystemTags[$fileId];
-				unset($fileIds[$i]);
-			}
-		}
-
-		if (!empty($fileIds)) {
-			$mappedSystemTags = $this->systemTagObjectMapper->getTagIdsForObjects($fileIds, 'files');
-			foreach ($mappedSystemTags as $fileId => $fileSystemTags) {
-				$this->fileSystemTags[$fileId] = $fileSystemTags;
-				$systemTags[] = $fileSystemTags;
-			}
-		}
-
-		$systemTags = call_user_func_array('array_merge', $systemTags);
-		$systemTags = array_unique($systemTags);
-		return $systemTags;
-	}
-
-	/**
-	 * Get the file ids of the given path and its parents
-	 * @param ICache $cache
-	 * @param string $path
-	 * @param bool $isExternalStorage
-	 * @return int[]
-	 */
-	protected function getFileIds(ICache $cache, $path, $isExternalStorage) {
-		$cacheId = $cache->getNumericStorageId();
-		if ($this->storage->instanceOfStorage(Jail::class)) {
-			$absolutePath = $this->storage->getUnjailedPath($path);
-		} else {
-			$absolutePath = $path;
-		}
-
-		if (isset($this->fileIds[$cacheId][$absolutePath])) {
-			return $this->fileIds[$cacheId][$absolutePath];
-		}
-
-		$parentIds = [];
-		if ($path !== $this->dirname($path)) {
-			$parentIds = $this->getFileIds($cache, $this->dirname($path), $isExternalStorage);
-		} elseif (!$isExternalStorage) {
-			return [];
-		}
-
-		$fileId = $cache->getId($path);
-		if ($fileId !== -1) {
-			$parentIds[] = $fileId;
-		}
-
-		$this->fileIds[$cacheId][$absolutePath] = $parentIds;
-
-		return $parentIds;
-	}
-
-	protected function dirname($path) {
-		$dir = dirname($path);
-		return $dir === '.' ? '' : $dir;
-	}
-
-	public function supportedEntities(): array {
-		return [ File::class ];
-	}
-
-	public function isAvailableForScope(int $scope): bool {
-		return true;
-	}
+    use TFileCheck;
+
+    /** @var array */
+    protected $fileIds;
+
+    /** @var array */
+    protected $fileSystemTags;
+
+    public function __construct(
+        protected IL10N $l,
+        protected ISystemTagManager $systemTagManager,
+        protected ISystemTagObjectMapper $systemTagObjectMapper,
+        protected IUserSession $userSession,
+        protected IGroupManager $groupManager,
+    ) {
+    }
+
+    /**
+     * @param string $operator
+     * @param string $value
+     * @return bool
+     */
+    public function executeCheck($operator, $value) {
+        $systemTags = $this->getSystemTags();
+        return ($operator === 'is') === in_array($value, $systemTags);
+    }
+
+    /**
+     * @param string $operator
+     * @param string $value
+     * @throws \UnexpectedValueException
+     */
+    public function validateCheck($operator, $value) {
+        if (!in_array($operator, ['is', '!is'])) {
+            throw new \UnexpectedValueException($this->l->t('The given operator is invalid'), 1);
+        }
+
+        try {
+            $tags = $this->systemTagManager->getTagsByIds($value);
+
+            $user = $this->userSession->getUser();
+            $isAdmin = $user instanceof IUser && $this->groupManager->isAdmin($user->getUID());
+
+            if (!$isAdmin) {
+                foreach ($tags as $tag) {
+                    if (!$tag->isUserVisible()) {
+                        throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 4);
+                    }
+                }
+            }
+        } catch (TagNotFoundException $e) {
+            throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 2);
+        } catch (\InvalidArgumentException $e) {
+            throw new \UnexpectedValueException($this->l->t('The given tag id is invalid'), 3);
+        }
+    }
+
+    /**
+     * Get the ids of the assigned system tags
+     * @return string[]
+     */
+    protected function getSystemTags() {
+        $cache = $this->storage->getCache();
+        $fileIds = $this->getFileIds($cache, $this->path, !$this->storage->instanceOfStorage(IHomeStorage::class) || $this->storage->instanceOfStorage(SharedStorage::class));
+
+        $systemTags = [];
+        foreach ($fileIds as $i => $fileId) {
+            if (isset($this->fileSystemTags[$fileId])) {
+                $systemTags[] = $this->fileSystemTags[$fileId];
+                unset($fileIds[$i]);
+            }
+        }
+
+        if (!empty($fileIds)) {
+            $mappedSystemTags = $this->systemTagObjectMapper->getTagIdsForObjects($fileIds, 'files');
+            foreach ($mappedSystemTags as $fileId => $fileSystemTags) {
+                $this->fileSystemTags[$fileId] = $fileSystemTags;
+                $systemTags[] = $fileSystemTags;
+            }
+        }
+
+        $systemTags = call_user_func_array('array_merge', $systemTags);
+        $systemTags = array_unique($systemTags);
+        return $systemTags;
+    }
+
+    /**
+     * Get the file ids of the given path and its parents
+     * @param ICache $cache
+     * @param string $path
+     * @param bool $isExternalStorage
+     * @return int[]
+     */
+    protected function getFileIds(ICache $cache, $path, $isExternalStorage) {
+        $cacheId = $cache->getNumericStorageId();
+        if ($this->storage->instanceOfStorage(Jail::class)) {
+            $absolutePath = $this->storage->getUnjailedPath($path);
+        } else {
+            $absolutePath = $path;
+        }
+
+        if (isset($this->fileIds[$cacheId][$absolutePath])) {
+            return $this->fileIds[$cacheId][$absolutePath];
+        }
+
+        $parentIds = [];
+        if ($path !== $this->dirname($path)) {
+            $parentIds = $this->getFileIds($cache, $this->dirname($path), $isExternalStorage);
+        } elseif (!$isExternalStorage) {
+            return [];
+        }
+
+        $fileId = $cache->getId($path);
+        if ($fileId !== -1) {
+            $parentIds[] = $fileId;
+        }
+
+        $this->fileIds[$cacheId][$absolutePath] = $parentIds;
+
+        return $parentIds;
+    }
+
+    protected function dirname($path) {
+        $dir = dirname($path);
+        return $dir === '.' ? '' : $dir;
+    }
+
+    public function supportedEntities(): array {
+        return [ File::class ];
+    }
+
+    public function isAvailableForScope(int $scope): bool {
+        return true;
+    }
 }

lib/public/WorkflowEngine/IEntityCheck.php

Indentation +16 added lines, -16 removed lines

@@ -34,21 +34,21 @@
  * @since 18.0.0
  */
 interface IEntityCheck {
-	/**
-	 * Equips the check with a subject fitting the Entity. For instance, an
-	 * entity of File will receive an instance of OCP\Files\Node, or a comment
-	 * entity might get an IComment.
-	 *
-	 * The implementing check must be aware of the incoming type.
-	 *
-	 * If an unsupported subject is passed the implementation MAY throw an
-	 * \UnexpectedValueException.
-	 *
-	 * @param IEntity $entity
-	 * @param mixed $subject
-	 * @throws \UnexpectedValueException
-	 * @since 18.0.0
-	 */
-	public function setEntitySubject(IEntity $entity, $subject): void;
+    /**
+     * Equips the check with a subject fitting the Entity. For instance, an
+     * entity of File will receive an instance of OCP\Files\Node, or a comment
+     * entity might get an IComment.
+     *
+     * The implementing check must be aware of the incoming type.
+     *
+     * If an unsupported subject is passed the implementation MAY throw an
+     * \UnexpectedValueException.
+     *
+     * @param IEntity $entity
+     * @param mixed $subject
+     * @throws \UnexpectedValueException
+     * @since 18.0.0
+     */
+    public function setEntitySubject(IEntity $entity, $subject): void;
 
 }

lib/public/WorkflowEngine/ICheck.php

Indentation +36 added lines, -36 removed lines

@@ -31,43 +31,43 @@
  * @since 9.1
  */
 interface ICheck {
-	/**
-	 * @param string $operator
-	 * @param string $value
-	 * @return bool
-	 * @since 9.1
-	 */
-	public function executeCheck($operator, $value);
+    /**
+     * @param string $operator
+     * @param string $value
+     * @return bool
+     * @since 9.1
+     */
+    public function executeCheck($operator, $value);
 
-	/**
-	 * @param string $operator
-	 * @param string $value
-	 * @throws \UnexpectedValueException
-	 * @since 9.1
-	 */
-	public function validateCheck($operator, $value);
+    /**
+     * @param string $operator
+     * @param string $value
+     * @throws \UnexpectedValueException
+     * @since 9.1
+     */
+    public function validateCheck($operator, $value);
 
-	/**
-	 * returns a list of Entities the checker supports. The values must match
-	 * the class name of the entity.
-	 *
-	 * An empty result means the check is universally available.
-	 *
-	 * @since 18.0.0
-	 */
-	public function supportedEntities(): array;
+    /**
+     * returns a list of Entities the checker supports. The values must match
+     * the class name of the entity.
+     *
+     * An empty result means the check is universally available.
+     *
+     * @since 18.0.0
+     */
+    public function supportedEntities(): array;
 
-	/**
-	 * returns whether the operation can be used in the requested scope.
-	 *
-	 * Scope IDs are defined as constants in OCP\WorkflowEngine\IManager. At
-	 * time of writing these are SCOPE_ADMIN and SCOPE_USER.
-	 *
-	 * For possibly unknown future scopes the recommended behaviour is: if
-	 * user scope is permitted, the default behaviour should return `true`,
-	 * otherwise `false`.
-	 *
-	 * @since 18.0.0
-	 */
-	public function isAvailableForScope(int $scope): bool;
+    /**
+     * returns whether the operation can be used in the requested scope.
+     *
+     * Scope IDs are defined as constants in OCP\WorkflowEngine\IManager. At
+     * time of writing these are SCOPE_ADMIN and SCOPE_USER.
+     *
+     * For possibly unknown future scopes the recommended behaviour is: if
+     * user scope is permitted, the default behaviour should return `true`,
+     * otherwise `false`.
+     *
+     * @since 18.0.0
+     */
+    public function isAvailableForScope(int $scope): bool;
 }