nextcloud / server

apps/lookup_server_connector/lib/UpdateLookupServer.php

Indentation +42 added lines, -42 removed lines

@@ -34,51 +34,51 @@
  * @package OCA\LookupServerConnector
  */
 class UpdateLookupServer {
-	/** @var IConfig */
-	private $config;
-	/** @var IJobList */
-	private $jobList;
+    /** @var IConfig */
+    private $config;
+    /** @var IJobList */
+    private $jobList;
 
-	/**
-	 * @param IJobList $jobList
-	 * @param IConfig $config
-	 */
-	public function __construct(IJobList $jobList,
-								IConfig $config) {
-		$this->config = $config;
-		$this->jobList = $jobList;
-	}
+    /**
+     * @param IJobList $jobList
+     * @param IConfig $config
+     */
+    public function __construct(IJobList $jobList,
+                                IConfig $config) {
+        $this->config = $config;
+        $this->jobList = $jobList;
+    }
 
-	/**
-	 * @param IUser $user
-	 */
-	public function userUpdated(IUser $user): void {
-		if (!$this->shouldUpdateLookupServer()) {
-			return;
-		}
+    /**
+     * @param IUser $user
+     */
+    public function userUpdated(IUser $user): void {
+        if (!$this->shouldUpdateLookupServer()) {
+            return;
+        }
 
-		// Reset retry counter
-		$this->config->deleteUserValue(
-			$user->getUID(),
-			'lookup_server_connector',
-			'update_retries'
-		);
-		$this->jobList->add(RetryJob::class, ['userId' => $user->getUID()]);
-	}
+        // Reset retry counter
+        $this->config->deleteUserValue(
+            $user->getUID(),
+            'lookup_server_connector',
+            'update_retries'
+        );
+        $this->jobList->add(RetryJob::class, ['userId' => $user->getUID()]);
+    }
 
-	/**
-	 * check if we should update the lookup server, we only do it if
-	 *
-	 * + we have an internet connection
-	 * + the lookup server update was not disabled by the admin
-	 * + we have a valid lookup server URL
-	 *
-	 * @return bool
-	 */
-	private function shouldUpdateLookupServer(): bool {
-		return $this->config->getSystemValueBool('has_internet_connection', true) === true &&
-			$this->config->getAppValue('files_sharing', 'lookupServerUploadEnabled', 'yes') === 'yes' &&
-			$this->config->getSystemValueString('lookup_server', 'https://lookup.nextcloud.com') !== '';
-	}
+    /**
+     * check if we should update the lookup server, we only do it if
+     *
+     * + we have an internet connection
+     * + the lookup server update was not disabled by the admin
+     * + we have a valid lookup server URL
+     *
+     * @return bool
+     */
+    private function shouldUpdateLookupServer(): bool {
+        return $this->config->getSystemValueBool('has_internet_connection', true) === true &&
+            $this->config->getAppValue('files_sharing', 'lookupServerUploadEnabled', 'yes') === 'yes' &&
+            $this->config->getSystemValueString('lookup_server', 'https://lookup.nextcloud.com') !== '';
+    }
 
 }

apps/dav/lib/Exception/UnsupportedLimitOnInitialSyncException.php

Indentation +6 added lines, -6 removed lines

@@ -32,10 +32,10 @@
  */
 class UnsupportedLimitOnInitialSyncException extends InsufficientStorage {
 
-	/**
-	 * @inheritDoc
-	 */
-	public function serialize(Server $server, \DOMElement $errorNode) {
-		$errorNode->appendChild($errorNode->ownerDocument->createElementNS('DAV:', 'd:number-of-matches-within-limits'));
-	}
+    /**
+     * @inheritDoc
+     */
+    public function serialize(Server $server, \DOMElement $errorNode) {
+        $errorNode->appendChild($errorNode->ownerDocument->createElementNS('DAV:', 'd:number-of-matches-within-limits'));
+    }
 }

lib/public/AppFramework/Http/StrictContentSecurityPolicy.php

Indentation +40 added lines, -40 removed lines

@@ -42,45 +42,45 @@
  * @deprecated 17.0.0
  */
 class StrictContentSecurityPolicy extends EmptyContentSecurityPolicy {
-	/** @var bool Whether inline JS snippets are allowed */
-	protected $inlineScriptAllowed = false;
-	/** @var bool Whether eval in JS scripts is allowed */
-	protected $evalScriptAllowed = false;
-	/** @var array Domains from which scripts can get loaded */
-	protected $allowedScriptDomains = [
-		'\'self\'',
-	];
-	/** @var bool Whether inline CSS is allowed */
-	protected $inlineStyleAllowed = false;
-	/** @var array Domains from which CSS can get loaded */
-	protected $allowedStyleDomains = [
-		'\'self\'',
-	];
-	/** @var array Domains from which images can get loaded */
-	protected $allowedImageDomains = [
-		'\'self\'',
-		'data:',
-		'blob:',
-	];
-	/** @var array Domains to which connections can be done */
-	protected $allowedConnectDomains = [
-		'\'self\'',
-	];
-	/** @var array Domains from which media elements can be loaded */
-	protected $allowedMediaDomains = [
-		'\'self\'',
-	];
-	/** @var array Domains from which object elements can be loaded */
-	protected $allowedObjectDomains = [];
-	/** @var array Domains from which iframes can be loaded */
-	protected $allowedFrameDomains = [];
-	/** @var array Domains from which fonts can be loaded */
-	protected $allowedFontDomains = [
-		'\'self\'',
-	];
-	/** @var array Domains from which web-workers and nested browsing content can load elements */
-	protected $allowedChildSrcDomains = [];
+    /** @var bool Whether inline JS snippets are allowed */
+    protected $inlineScriptAllowed = false;
+    /** @var bool Whether eval in JS scripts is allowed */
+    protected $evalScriptAllowed = false;
+    /** @var array Domains from which scripts can get loaded */
+    protected $allowedScriptDomains = [
+        '\'self\'',
+    ];
+    /** @var bool Whether inline CSS is allowed */
+    protected $inlineStyleAllowed = false;
+    /** @var array Domains from which CSS can get loaded */
+    protected $allowedStyleDomains = [
+        '\'self\'',
+    ];
+    /** @var array Domains from which images can get loaded */
+    protected $allowedImageDomains = [
+        '\'self\'',
+        'data:',
+        'blob:',
+    ];
+    /** @var array Domains to which connections can be done */
+    protected $allowedConnectDomains = [
+        '\'self\'',
+    ];
+    /** @var array Domains from which media elements can be loaded */
+    protected $allowedMediaDomains = [
+        '\'self\'',
+    ];
+    /** @var array Domains from which object elements can be loaded */
+    protected $allowedObjectDomains = [];
+    /** @var array Domains from which iframes can be loaded */
+    protected $allowedFrameDomains = [];
+    /** @var array Domains from which fonts can be loaded */
+    protected $allowedFontDomains = [
+        '\'self\'',
+    ];
+    /** @var array Domains from which web-workers and nested browsing content can load elements */
+    protected $allowedChildSrcDomains = [];
 
-	/** @var array Domains which can embed this Nextcloud instance */
-	protected $allowedFrameAncestors = [];
+    /** @var array Domains which can embed this Nextcloud instance */
+    protected $allowedFrameAncestors = [];
 }

lib/public/Calendar/IMetadataProvider.php

Spacing +1 added lines, -1 removed lines

@@ -61,5 +61,5 @@
 	 * @return string|null - The value stored for the key, null if no value stored
 	 * @since 17.0.0
 	 */
-	public function getMetadataForKey(string $key):?string;
+	public function getMetadataForKey(string $key): ?string;
 }

Indentation +27 added lines, -27 removed lines

@@ -30,33 +30,33 @@
  * @since 17.0.0
  */
 interface IMetadataProvider {
-	/**
-	 * Get a list of all metadata keys available for this room
-	 *
-	 * Room backends are allowed to return custom keys, beyond the ones
-	 * defined in this class. If they do, they should make sure to use their
-	 * own namespace.
-	 *
-	 * @return String[] - A list of available keys
-	 * @since 17.0.0
-	 */
-	public function getAllAvailableMetadataKeys():array;
+    /**
+     * Get a list of all metadata keys available for this room
+     *
+     * Room backends are allowed to return custom keys, beyond the ones
+     * defined in this class. If they do, they should make sure to use their
+     * own namespace.
+     *
+     * @return String[] - A list of available keys
+     * @since 17.0.0
+     */
+    public function getAllAvailableMetadataKeys():array;
 
-	/**
-	 * Get whether or not a metadata key is set for this room
-	 *
-	 * @param string $key - The key to check for
-	 * @return bool - Whether or not key is available
-	 * @since 17.0.0
-	 */
-	public function hasMetadataForKey(string $key):bool;
+    /**
+     * Get whether or not a metadata key is set for this room
+     *
+     * @param string $key - The key to check for
+     * @return bool - Whether or not key is available
+     * @since 17.0.0
+     */
+    public function hasMetadataForKey(string $key):bool;
 
-	/**
-	 * Get the value for a metadata key
-	 *
-	 * @param string $key - The key to check for
-	 * @return string|null - The value stored for the key, null if no value stored
-	 * @since 17.0.0
-	 */
-	public function getMetadataForKey(string $key):?string;
+    /**
+     * Get the value for a metadata key
+     *
+     * @param string $key - The key to check for
+     * @return string|null - The value stored for the key, null if no value stored
+     * @since 17.0.0
+     */
+    public function getMetadataForKey(string $key):?string;
 }

lib/private/Security/FeaturePolicy/FeaturePolicyManager.php

Spacing +3 added lines, -3 removed lines

@@ -60,10 +60,10 @@
 	 */
 	public function mergePolicies(FeaturePolicy $defaultPolicy,
 								  EmptyFeaturePolicy $originalPolicy): FeaturePolicy {
-		foreach ((object)(array)$originalPolicy as $name => $value) {
-			$setter = 'set' . ucfirst($name);
+		foreach ((object) (array) $originalPolicy as $name => $value) {
+			$setter = 'set'.ucfirst($name);
 			if (\is_array($value)) {
-				$getter = 'get' . ucfirst($name);
+				$getter = 'get'.ucfirst($name);
 				$currentValues = \is_array($defaultPolicy->$getter()) ? $defaultPolicy->$getter() : [];
 				$defaultPolicy->$setter(\array_values(\array_unique(\array_merge($currentValues, $value))));
 			} elseif (\is_bool($value)) {

Indentation +37 added lines, -37 removed lines

@@ -31,48 +31,48 @@
 use OCP\Security\FeaturePolicy\AddFeaturePolicyEvent;
 
 class FeaturePolicyManager {
-	/** @var EmptyFeaturePolicy[] */
-	private $policies = [];
+    /** @var EmptyFeaturePolicy[] */
+    private $policies = [];
 
-	/** @var IEventDispatcher */
-	private $dispatcher;
+    /** @var IEventDispatcher */
+    private $dispatcher;
 
-	public function __construct(IEventDispatcher $dispatcher) {
-		$this->dispatcher = $dispatcher;
-	}
+    public function __construct(IEventDispatcher $dispatcher) {
+        $this->dispatcher = $dispatcher;
+    }
 
-	public function addDefaultPolicy(EmptyFeaturePolicy $policy): void {
-		$this->policies[] = $policy;
-	}
+    public function addDefaultPolicy(EmptyFeaturePolicy $policy): void {
+        $this->policies[] = $policy;
+    }
 
-	public function getDefaultPolicy(): FeaturePolicy {
-		$event = new AddFeaturePolicyEvent($this);
-		$this->dispatcher->dispatchTyped($event);
+    public function getDefaultPolicy(): FeaturePolicy {
+        $event = new AddFeaturePolicyEvent($this);
+        $this->dispatcher->dispatchTyped($event);
 
-		$defaultPolicy = new FeaturePolicy();
-		foreach ($this->policies as $policy) {
-			$defaultPolicy = $this->mergePolicies($defaultPolicy, $policy);
-		}
-		return $defaultPolicy;
-	}
+        $defaultPolicy = new FeaturePolicy();
+        foreach ($this->policies as $policy) {
+            $defaultPolicy = $this->mergePolicies($defaultPolicy, $policy);
+        }
+        return $defaultPolicy;
+    }
 
-	/**
-	 * Merges the first given policy with the second one
-	 *
-	 */
-	public function mergePolicies(FeaturePolicy $defaultPolicy,
-								  EmptyFeaturePolicy $originalPolicy): FeaturePolicy {
-		foreach ((object)(array)$originalPolicy as $name => $value) {
-			$setter = 'set' . ucfirst($name);
-			if (\is_array($value)) {
-				$getter = 'get' . ucfirst($name);
-				$currentValues = \is_array($defaultPolicy->$getter()) ? $defaultPolicy->$getter() : [];
-				$defaultPolicy->$setter(\array_values(\array_unique(\array_merge($currentValues, $value))));
-			} elseif (\is_bool($value)) {
-				$defaultPolicy->$setter($value);
-			}
-		}
+    /**
+     * Merges the first given policy with the second one
+     *
+     */
+    public function mergePolicies(FeaturePolicy $defaultPolicy,
+                                    EmptyFeaturePolicy $originalPolicy): FeaturePolicy {
+        foreach ((object)(array)$originalPolicy as $name => $value) {
+            $setter = 'set' . ucfirst($name);
+            if (\is_array($value)) {
+                $getter = 'get' . ucfirst($name);
+                $currentValues = \is_array($defaultPolicy->$getter()) ? $defaultPolicy->$getter() : [];
+                $defaultPolicy->$setter(\array_values(\array_unique(\array_merge($currentValues, $value))));
+            } elseif (\is_bool($value)) {
+                $defaultPolicy->$setter($value);
+            }
+        }
 
-		return $defaultPolicy;
-	}
+        return $defaultPolicy;
+    }
 }

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/Proxy/Proxy.php

Indentation +11 added lines, -11 removed lines

@@ -36,16 +36,16 @@
  */
 class Proxy extends Entity {
 
-	/** @var string */
-	protected $ownerId;
-	/** @var string */
-	protected $proxyId;
-	/** @var int */
-	protected $permissions;
+    /** @var string */
+    protected $ownerId;
+    /** @var string */
+    protected $proxyId;
+    /** @var int */
+    protected $permissions;
 
-	public function __construct() {
-		$this->addType('ownerId', 'string');
-		$this->addType('proxyId', 'string');
-		$this->addType('permissions', 'int');
-	}
+    public function __construct() {
+        $this->addType('ownerId', 'string');
+        $this->addType('proxyId', 'string');
+        $this->addType('permissions', 'int');
+    }
 }

apps/dav/lib/Migration/Version1012Date20190808122342.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 */
 	public function changeSchema(IOutput $output,
 								 \Closure $schemaClosure,
-								 array $options):?ISchemaWrapper {
+								 array $options): ?ISchemaWrapper {
 		/** @var ISchemaWrapper $schema */
 		$schema = $schemaClosure();
 

Indentation +77 added lines, -77 removed lines

@@ -38,86 +38,86 @@
  */
 class Version1012Date20190808122342 extends SimpleMigrationStep {
 
-	/**
-	 * @param IOutput $output
-	 * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 * @since 17.0.0
-	 */
-	public function changeSchema(IOutput $output,
-								 \Closure $schemaClosure,
-								 array $options):?ISchemaWrapper {
-		/** @var ISchemaWrapper $schema */
-		$schema = $schemaClosure();
+    /**
+     * @param IOutput $output
+     * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     * @return null|ISchemaWrapper
+     * @since 17.0.0
+     */
+    public function changeSchema(IOutput $output,
+                                    \Closure $schemaClosure,
+                                    array $options):?ISchemaWrapper {
+        /** @var ISchemaWrapper $schema */
+        $schema = $schemaClosure();
 
-		if (!$schema->hasTable('calendar_reminders')) {
-			$table = $schema->createTable('calendar_reminders');
+        if (!$schema->hasTable('calendar_reminders')) {
+            $table = $schema->createTable('calendar_reminders');
 
-			$table->addColumn('id', Types::BIGINT, [
-				'autoincrement' => true,
-				'notnull' => true,
-				'length' => 11,
-				'unsigned' => true,
-			]);
-			$table->addColumn('calendar_id', Types::BIGINT, [
-				'notnull' => true,
-				'length' => 11,
-			]);
-			$table->addColumn('object_id', Types::BIGINT, [
-				'notnull' => true,
-				'length' => 11,
-			]);
-			$table->addColumn('is_recurring', Types::SMALLINT, [
-				'notnull' => false,
-				'length' => 1,
-			]);
-			$table->addColumn('uid', Types::STRING, [
-				'notnull' => true,
-				'length' => 255,
-			]);
-			$table->addColumn('recurrence_id', Types::BIGINT, [
-				'notnull' => false,
-				'length' => 11,
-				'unsigned' => true,
-			]);
-			$table->addColumn('is_recurrence_exception', Types::SMALLINT, [
-				'notnull' => true,
-				'length' => 1,
-			]);
-			$table->addColumn('event_hash', Types::STRING, [
-				'notnull' => true,
-				'length' => 255,
-			]);
-			$table->addColumn('alarm_hash', Types::STRING, [
-				'notnull' => true,
-				'length' => 255,
-			]);
-			$table->addColumn('type', Types::STRING, [
-				'notnull' => true,
-				'length' => 255,
-			]);
-			$table->addColumn('is_relative', Types::SMALLINT, [
-				'notnull' => true,
-				'length' => 1,
-			]);
-			$table->addColumn('notification_date', Types::BIGINT, [
-				'notnull' => true,
-				'length' => 11,
-				'unsigned' => true,
-			]);
-			$table->addColumn('is_repeat_based', Types::SMALLINT, [
-				'notnull' => true,
-				'length' => 1,
-			]);
+            $table->addColumn('id', Types::BIGINT, [
+                'autoincrement' => true,
+                'notnull' => true,
+                'length' => 11,
+                'unsigned' => true,
+            ]);
+            $table->addColumn('calendar_id', Types::BIGINT, [
+                'notnull' => true,
+                'length' => 11,
+            ]);
+            $table->addColumn('object_id', Types::BIGINT, [
+                'notnull' => true,
+                'length' => 11,
+            ]);
+            $table->addColumn('is_recurring', Types::SMALLINT, [
+                'notnull' => false,
+                'length' => 1,
+            ]);
+            $table->addColumn('uid', Types::STRING, [
+                'notnull' => true,
+                'length' => 255,
+            ]);
+            $table->addColumn('recurrence_id', Types::BIGINT, [
+                'notnull' => false,
+                'length' => 11,
+                'unsigned' => true,
+            ]);
+            $table->addColumn('is_recurrence_exception', Types::SMALLINT, [
+                'notnull' => true,
+                'length' => 1,
+            ]);
+            $table->addColumn('event_hash', Types::STRING, [
+                'notnull' => true,
+                'length' => 255,
+            ]);
+            $table->addColumn('alarm_hash', Types::STRING, [
+                'notnull' => true,
+                'length' => 255,
+            ]);
+            $table->addColumn('type', Types::STRING, [
+                'notnull' => true,
+                'length' => 255,
+            ]);
+            $table->addColumn('is_relative', Types::SMALLINT, [
+                'notnull' => true,
+                'length' => 1,
+            ]);
+            $table->addColumn('notification_date', Types::BIGINT, [
+                'notnull' => true,
+                'length' => 11,
+                'unsigned' => true,
+            ]);
+            $table->addColumn('is_repeat_based', Types::SMALLINT, [
+                'notnull' => true,
+                'length' => 1,
+            ]);
 
-			$table->setPrimaryKey(['id']);
-			$table->addIndex(['object_id'], 'calendar_reminder_objid');
-			$table->addIndex(['uid', 'recurrence_id'], 'calendar_reminder_uidrec');
+            $table->setPrimaryKey(['id']);
+            $table->addIndex(['object_id'], 'calendar_reminder_objid');
+            $table->addIndex(['uid', 'recurrence_id'], 'calendar_reminder_uidrec');
 
-			return $schema;
-		}
+            return $schema;
+        }
 
-		return null;
-	}
+        return null;
+    }
 }