nextcloud / server

apps/settings/templates/settings-vue.php

Indentation +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
 // Do we have some data to inject ?
 if (is_array($_['serverData'])) {
-	?>
+    ?>
 <span id="serverData" data-server="<?php p(json_encode($_['serverData'])); ?>"></span>
 <?php
 } ?>

apps/files/lib/Settings/PersonalSettings.php

Indentation +9 added lines, -9 removed lines

@@ -31,15 +31,15 @@
 use OCP\Settings\ISettings;
 
 class PersonalSettings implements ISettings {
-	public function getForm(): TemplateResponse {
-		return new TemplateResponse(Application::APP_ID, 'settings-personal');
-	}
+    public function getForm(): TemplateResponse {
+        return new TemplateResponse(Application::APP_ID, 'settings-personal');
+    }
 
-	public function getSection(): string {
-		return 'sharing';
-	}
+    public function getSection(): string {
+        return 'sharing';
+    }
 
-	public function getPriority(): int {
-		return 90;
-	}
+    public function getPriority(): int {
+        return 90;
+    }
 }

apps/files/lib/Collaboration/Resources/ResourceProvider.php

Indentation +96 added lines, -96 removed lines

@@ -37,100 +37,100 @@
 use OCP\IUser;
 
 class ResourceProvider implements IProvider {
-	public const RESOURCE_TYPE = 'file';
-
-	/** @var IRootFolder */
-	protected $rootFolder;
-	/** @var IPreview */
-	private $preview;
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	/** @var array */
-	protected $nodes = [];
-
-	public function __construct(IRootFolder $rootFolder,
-								IPreview $preview,
-								IURLGenerator $urlGenerator) {
-		$this->rootFolder = $rootFolder;
-		$this->preview = $preview;
-		$this->urlGenerator = $urlGenerator;
-	}
-
-	private function getNode(IResource $resource): ?Node {
-		if (isset($this->nodes[(int) $resource->getId()])) {
-			return $this->nodes[(int) $resource->getId()];
-		}
-		$nodes = $this->rootFolder->getById((int) $resource->getId());
-		if (!empty($nodes)) {
-			$this->nodes[(int) $resource->getId()] = array_shift($nodes);
-			return $this->nodes[(int) $resource->getId()];
-		}
-		return null;
-	}
-
-	/**
-	 * @param IResource $resource
-	 * @return array
-	 * @since 16.0.0
-	 */
-	public function getResourceRichObject(IResource $resource): array {
-		if (isset($this->nodes[(int) $resource->getId()])) {
-			$node = $this->nodes[(int) $resource->getId()]->getPath();
-		} else {
-			$node = $this->getNode($resource);
-		}
-
-		if ($node instanceof Node) {
-			$link = $this->urlGenerator->linkToRouteAbsolute(
-				'files.viewcontroller.showFile',
-				['fileid' => $resource->getId()]
-			);
-			return [
-				'type' => 'file',
-				'id' => $resource->getId(),
-				'name' => $node->getName(),
-				'path' => $node->getInternalPath(),
-				'link' => $link,
-				'mimetype' => $node->getMimetype(),
-				'preview-available' => $this->preview->isAvailable($node),
-			];
-		}
-
-		throw new ResourceException('File not found');
-	}
-
-	/**
-	 * Can a user/guest access the collection
-	 *
-	 * @param IResource $resource
-	 * @param IUser $user
-	 * @return bool
-	 * @since 16.0.0
-	 */
-	public function canAccessResource(IResource $resource, IUser $user = null): bool {
-		if (!$user instanceof IUser) {
-			return false;
-		}
-
-		$userFolder = $this->rootFolder->getUserFolder($user->getUID());
-		$nodes = $userFolder->getById((int) $resource->getId());
-
-		if (!empty($nodes)) {
-			$this->nodes[(int) $resource->getId()] = array_shift($nodes);
-			return true;
-		}
-
-		return false;
-	}
-
-	/**
-	 * Get the resource type of the provider
-	 *
-	 * @return string
-	 * @since 16.0.0
-	 */
-	public function getType(): string {
-		return self::RESOURCE_TYPE;
-	}
+    public const RESOURCE_TYPE = 'file';
+
+    /** @var IRootFolder */
+    protected $rootFolder;
+    /** @var IPreview */
+    private $preview;
+    /** @var IURLGenerator */
+    private $urlGenerator;
+
+    /** @var array */
+    protected $nodes = [];
+
+    public function __construct(IRootFolder $rootFolder,
+                                IPreview $preview,
+                                IURLGenerator $urlGenerator) {
+        $this->rootFolder = $rootFolder;
+        $this->preview = $preview;
+        $this->urlGenerator = $urlGenerator;
+    }
+
+    private function getNode(IResource $resource): ?Node {
+        if (isset($this->nodes[(int) $resource->getId()])) {
+            return $this->nodes[(int) $resource->getId()];
+        }
+        $nodes = $this->rootFolder->getById((int) $resource->getId());
+        if (!empty($nodes)) {
+            $this->nodes[(int) $resource->getId()] = array_shift($nodes);
+            return $this->nodes[(int) $resource->getId()];
+        }
+        return null;
+    }
+
+    /**
+     * @param IResource $resource
+     * @return array
+     * @since 16.0.0
+     */
+    public function getResourceRichObject(IResource $resource): array {
+        if (isset($this->nodes[(int) $resource->getId()])) {
+            $node = $this->nodes[(int) $resource->getId()]->getPath();
+        } else {
+            $node = $this->getNode($resource);
+        }
+
+        if ($node instanceof Node) {
+            $link = $this->urlGenerator->linkToRouteAbsolute(
+                'files.viewcontroller.showFile',
+                ['fileid' => $resource->getId()]
+            );
+            return [
+                'type' => 'file',
+                'id' => $resource->getId(),
+                'name' => $node->getName(),
+                'path' => $node->getInternalPath(),
+                'link' => $link,
+                'mimetype' => $node->getMimetype(),
+                'preview-available' => $this->preview->isAvailable($node),
+            ];
+        }
+
+        throw new ResourceException('File not found');
+    }
+
+    /**
+     * Can a user/guest access the collection
+     *
+     * @param IResource $resource
+     * @param IUser $user
+     * @return bool
+     * @since 16.0.0
+     */
+    public function canAccessResource(IResource $resource, IUser $user = null): bool {
+        if (!$user instanceof IUser) {
+            return false;
+        }
+
+        $userFolder = $this->rootFolder->getUserFolder($user->getUID());
+        $nodes = $userFolder->getById((int) $resource->getId());
+
+        if (!empty($nodes)) {
+            $this->nodes[(int) $resource->getId()] = array_shift($nodes);
+            return true;
+        }
+
+        return false;
+    }
+
+    /**
+     * Get the resource type of the provider
+     *
+     * @return string
+     * @since 16.0.0
+     */
+    public function getType(): string {
+        return self::RESOURCE_TYPE;
+    }
 }

apps/files/lib/BackgroundJob/TransferOwnership.php

Indentation +137 added lines, -137 removed lines

@@ -42,141 +42,141 @@
 
 class TransferOwnership extends QueuedJob {
 
-	/** @var IUserManager $userManager */
-	private $userManager;
-
-	/** @var OwnershipTransferService */
-	private $transferService;
-
-	/** @var ILogger */
-	private $logger;
-
-	/** @var NotificationManager */
-	private $notificationManager;
-
-	/** @var TransferOwnershipMapper */
-	private $mapper;
-	/** @var IRootFolder */
-	private $rootFolder;
-
-	public function __construct(ITimeFactory $timeFactory,
-								IUserManager $userManager,
-								OwnershipTransferService $transferService,
-								ILogger $logger,
-								NotificationManager $notificationManager,
-								TransferOwnershipMapper $mapper,
-								IRootFolder $rootFolder) {
-		parent::__construct($timeFactory);
-
-		$this->userManager = $userManager;
-		$this->transferService = $transferService;
-		$this->logger = $logger;
-		$this->notificationManager = $notificationManager;
-		$this->mapper = $mapper;
-		$this->rootFolder = $rootFolder;
-	}
-
-	protected function run($argument) {
-		$id = $argument['id'];
-
-		$transfer = $this->mapper->getById($id);
-		$sourceUser = $transfer->getSourceUser();
-		$destinationUser = $transfer->getTargetUser();
-		$fileId = $transfer->getFileId();
-
-		$userFolder = $this->rootFolder->getUserFolder($sourceUser);
-		$nodes = $userFolder->getById($fileId);
-
-		if (empty($nodes)) {
-			$this->logger->alert('Could not transfer ownership: Node not found');
-			$this->failedNotication($transfer);
-			return;
-		}
-		$path = $userFolder->getRelativePath($nodes[0]->getPath());
-
-		$sourceUserObject = $this->userManager->get($sourceUser);
-		$destinationUserObject = $this->userManager->get($destinationUser);
-
-		if (!$sourceUserObject instanceof IUser) {
-			$this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
-			$this->failedNotication($transfer);
-			return;
-		}
-
-		if (!$destinationUserObject instanceof IUser) {
-			$this->logger->alert("Unknown destination user $destinationUser");
-			$this->failedNotication($transfer);
-			return;
-		}
-
-		try {
-			$this->transferService->transfer(
-				$sourceUserObject,
-				$destinationUserObject,
-				ltrim($path, '/')
-			);
-			$this->successNotification($transfer);
-		} catch (TransferOwnershipException $e) {
-			$this->logger->logException($e);
-			$this->failedNotication($transfer);
-		}
-
-		$this->mapper->delete($transfer);
-	}
-
-	private function failedNotication(Transfer $transfer): void {
-		// Send notification to source user
-		$notification = $this->notificationManager->createNotification();
-		$notification->setUser($transfer->getSourceUser())
-			->setApp(Application::APP_ID)
-			->setDateTime($this->time->getDateTime())
-			->setSubject('transferOwnershipFailedSource', [
-				'sourceUser' => $transfer->getSourceUser(),
-				'targetUser' => $transfer->getTargetUser(),
-				'nodeName' => $transfer->getNodeName(),
-			])
-			->setObject('transfer', (string)$transfer->getId());
-		$this->notificationManager->notify($notification);
-
-		// Send notification to source user
-		$notification = $this->notificationManager->createNotification();
-		$notification->setUser($transfer->getTargetUser())
-			->setApp(Application::APP_ID)
-			->setDateTime($this->time->getDateTime())
-			->setSubject('transferOwnershipFailedTarget', [
-				'sourceUser' => $transfer->getSourceUser(),
-				'targetUser' => $transfer->getTargetUser(),
-				'nodeName' => $transfer->getNodeName(),
-			])
-			->setObject('transfer', (string)$transfer->getId());
-		$this->notificationManager->notify($notification);
-	}
-
-	private function successNotification(Transfer $transfer): void {
-		// Send notification to source user
-		$notification = $this->notificationManager->createNotification();
-		$notification->setUser($transfer->getSourceUser())
-			->setApp(Application::APP_ID)
-			->setDateTime($this->time->getDateTime())
-			->setSubject('transferOwnershipDoneSource', [
-				'sourceUser' => $transfer->getSourceUser(),
-				'targetUser' => $transfer->getTargetUser(),
-				'nodeName' => $transfer->getNodeName(),
-			])
-			->setObject('transfer', (string)$transfer->getId());
-		$this->notificationManager->notify($notification);
-
-		// Send notification to source user
-		$notification = $this->notificationManager->createNotification();
-		$notification->setUser($transfer->getTargetUser())
-			->setApp(Application::APP_ID)
-			->setDateTime($this->time->getDateTime())
-			->setSubject('transferOwnershipDoneTarget', [
-				'sourceUser' => $transfer->getSourceUser(),
-				'targetUser' => $transfer->getTargetUser(),
-				'nodeName' => $transfer->getNodeName(),
-			])
-			->setObject('transfer', (string)$transfer->getId());
-		$this->notificationManager->notify($notification);
-	}
+    /** @var IUserManager $userManager */
+    private $userManager;
+
+    /** @var OwnershipTransferService */
+    private $transferService;
+
+    /** @var ILogger */
+    private $logger;
+
+    /** @var NotificationManager */
+    private $notificationManager;
+
+    /** @var TransferOwnershipMapper */
+    private $mapper;
+    /** @var IRootFolder */
+    private $rootFolder;
+
+    public function __construct(ITimeFactory $timeFactory,
+                                IUserManager $userManager,
+                                OwnershipTransferService $transferService,
+                                ILogger $logger,
+                                NotificationManager $notificationManager,
+                                TransferOwnershipMapper $mapper,
+                                IRootFolder $rootFolder) {
+        parent::__construct($timeFactory);
+
+        $this->userManager = $userManager;
+        $this->transferService = $transferService;
+        $this->logger = $logger;
+        $this->notificationManager = $notificationManager;
+        $this->mapper = $mapper;
+        $this->rootFolder = $rootFolder;
+    }
+
+    protected function run($argument) {
+        $id = $argument['id'];
+
+        $transfer = $this->mapper->getById($id);
+        $sourceUser = $transfer->getSourceUser();
+        $destinationUser = $transfer->getTargetUser();
+        $fileId = $transfer->getFileId();
+
+        $userFolder = $this->rootFolder->getUserFolder($sourceUser);
+        $nodes = $userFolder->getById($fileId);
+
+        if (empty($nodes)) {
+            $this->logger->alert('Could not transfer ownership: Node not found');
+            $this->failedNotication($transfer);
+            return;
+        }
+        $path = $userFolder->getRelativePath($nodes[0]->getPath());
+
+        $sourceUserObject = $this->userManager->get($sourceUser);
+        $destinationUserObject = $this->userManager->get($destinationUser);
+
+        if (!$sourceUserObject instanceof IUser) {
+            $this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
+            $this->failedNotication($transfer);
+            return;
+        }
+
+        if (!$destinationUserObject instanceof IUser) {
+            $this->logger->alert("Unknown destination user $destinationUser");
+            $this->failedNotication($transfer);
+            return;
+        }
+
+        try {
+            $this->transferService->transfer(
+                $sourceUserObject,
+                $destinationUserObject,
+                ltrim($path, '/')
+            );
+            $this->successNotification($transfer);
+        } catch (TransferOwnershipException $e) {
+            $this->logger->logException($e);
+            $this->failedNotication($transfer);
+        }
+
+        $this->mapper->delete($transfer);
+    }
+
+    private function failedNotication(Transfer $transfer): void {
+        // Send notification to source user
+        $notification = $this->notificationManager->createNotification();
+        $notification->setUser($transfer->getSourceUser())
+            ->setApp(Application::APP_ID)
+            ->setDateTime($this->time->getDateTime())
+            ->setSubject('transferOwnershipFailedSource', [
+                'sourceUser' => $transfer->getSourceUser(),
+                'targetUser' => $transfer->getTargetUser(),
+                'nodeName' => $transfer->getNodeName(),
+            ])
+            ->setObject('transfer', (string)$transfer->getId());
+        $this->notificationManager->notify($notification);
+
+        // Send notification to source user
+        $notification = $this->notificationManager->createNotification();
+        $notification->setUser($transfer->getTargetUser())
+            ->setApp(Application::APP_ID)
+            ->setDateTime($this->time->getDateTime())
+            ->setSubject('transferOwnershipFailedTarget', [
+                'sourceUser' => $transfer->getSourceUser(),
+                'targetUser' => $transfer->getTargetUser(),
+                'nodeName' => $transfer->getNodeName(),
+            ])
+            ->setObject('transfer', (string)$transfer->getId());
+        $this->notificationManager->notify($notification);
+    }
+
+    private function successNotification(Transfer $transfer): void {
+        // Send notification to source user
+        $notification = $this->notificationManager->createNotification();
+        $notification->setUser($transfer->getSourceUser())
+            ->setApp(Application::APP_ID)
+            ->setDateTime($this->time->getDateTime())
+            ->setSubject('transferOwnershipDoneSource', [
+                'sourceUser' => $transfer->getSourceUser(),
+                'targetUser' => $transfer->getTargetUser(),
+                'nodeName' => $transfer->getNodeName(),
+            ])
+            ->setObject('transfer', (string)$transfer->getId());
+        $this->notificationManager->notify($notification);
+
+        // Send notification to source user
+        $notification = $this->notificationManager->createNotification();
+        $notification->setUser($transfer->getTargetUser())
+            ->setApp(Application::APP_ID)
+            ->setDateTime($this->time->getDateTime())
+            ->setSubject('transferOwnershipDoneTarget', [
+                'sourceUser' => $transfer->getSourceUser(),
+                'targetUser' => $transfer->getTargetUser(),
+                'nodeName' => $transfer->getNodeName(),
+            ])
+            ->setObject('transfer', (string)$transfer->getId());
+        $this->notificationManager->notify($notification);
+    }
 }

apps/dav/lib/CalDAV/Integration/ExternalCalendar.php

Indentation +84 added lines, -84 removed lines

@@ -33,99 +33,99 @@
  */
 abstract class ExternalCalendar implements CalDAV\ICalendar, DAV\IProperties {
 
-	/** @var string */
-	private const PREFIX = 'app-generated';
+    /** @var string */
+    private const PREFIX = 'app-generated';
 
-	/**
-	 * @var string
-	 *
-	 * Double dash is a valid delimiter,
-	 * because it will always split the calendarURIs correctly:
-	 * - our prefix contains only one dash and won't be split
-	 * - appIds are not allowed to contain dashes as per spec:
-	 * > must contain only lowercase ASCII characters and underscore
-	 * - explode has a limit of three, so even if the app-generated
-	 *   calendar uri has double dashes, it won't be split
-	 */
-	private const DELIMITER = '--';
+    /**
+     * @var string
+     *
+     * Double dash is a valid delimiter,
+     * because it will always split the calendarURIs correctly:
+     * - our prefix contains only one dash and won't be split
+     * - appIds are not allowed to contain dashes as per spec:
+     * > must contain only lowercase ASCII characters and underscore
+     * - explode has a limit of three, so even if the app-generated
+     *   calendar uri has double dashes, it won't be split
+     */
+    private const DELIMITER = '--';
 
-	/** @var string */
-	private $appId;
+    /** @var string */
+    private $appId;
 
-	/** @var string */
-	private $calendarUri;
+    /** @var string */
+    private $calendarUri;
 
-	/**
-	 * ExternalCalendar constructor.
-	 *
-	 * @param string $appId
-	 * @param string $calendarUri
-	 */
-	public function __construct(string $appId, string $calendarUri) {
-		$this->appId = $appId;
-		$this->calendarUri = $calendarUri;
-	}
+    /**
+     * ExternalCalendar constructor.
+     *
+     * @param string $appId
+     * @param string $calendarUri
+     */
+    public function __construct(string $appId, string $calendarUri) {
+        $this->appId = $appId;
+        $this->calendarUri = $calendarUri;
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	final public function getName() {
-		return implode(self::DELIMITER, [
-			self::PREFIX,
-			$this->appId,
-			$this->calendarUri,
-		]);
-	}
+    /**
+     * @inheritDoc
+     */
+    final public function getName() {
+        return implode(self::DELIMITER, [
+            self::PREFIX,
+            $this->appId,
+            $this->calendarUri,
+        ]);
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	final public function setName($name) {
-		throw new DAV\Exception\MethodNotAllowed('Renaming calendars is not yet supported');
-	}
+    /**
+     * @inheritDoc
+     */
+    final public function setName($name) {
+        throw new DAV\Exception\MethodNotAllowed('Renaming calendars is not yet supported');
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	final public function createDirectory($name) {
-		throw new DAV\Exception\MethodNotAllowed('Creating collections in calendar objects is not allowed');
-	}
+    /**
+     * @inheritDoc
+     */
+    final public function createDirectory($name) {
+        throw new DAV\Exception\MethodNotAllowed('Creating collections in calendar objects is not allowed');
+    }
 
-	/**
-	 * Checks whether the calendar uri is app-generated
-	 *
-	 * @param string $calendarUri
-	 * @return bool
-	 */
-	public static function isAppGeneratedCalendar(string $calendarUri):bool {
-		return strpos($calendarUri, self::PREFIX) === 0 && substr_count($calendarUri, self::DELIMITER) >= 2;
-	}
+    /**
+     * Checks whether the calendar uri is app-generated
+     *
+     * @param string $calendarUri
+     * @return bool
+     */
+    public static function isAppGeneratedCalendar(string $calendarUri):bool {
+        return strpos($calendarUri, self::PREFIX) === 0 && substr_count($calendarUri, self::DELIMITER) >= 2;
+    }
 
-	/**
-	 * Splits an app-generated calendar-uri into appId and calendarUri
-	 *
-	 * @param string $calendarUri
-	 * @return array
-	 */
-	public static function splitAppGeneratedCalendarUri(string $calendarUri):array {
-		$array = array_slice(explode(self::DELIMITER, $calendarUri, 3), 1);
-		// Check the array has expected amount of elements
-		// and none of them is an empty string
-		if (\count($array) !== 2 || \in_array('', $array, true)) {
-			throw new \InvalidArgumentException('Provided calendar uri was not app-generated');
-		}
+    /**
+     * Splits an app-generated calendar-uri into appId and calendarUri
+     *
+     * @param string $calendarUri
+     * @return array
+     */
+    public static function splitAppGeneratedCalendarUri(string $calendarUri):array {
+        $array = array_slice(explode(self::DELIMITER, $calendarUri, 3), 1);
+        // Check the array has expected amount of elements
+        // and none of them is an empty string
+        if (\count($array) !== 2 || \in_array('', $array, true)) {
+            throw new \InvalidArgumentException('Provided calendar uri was not app-generated');
+        }
 
-		return $array;
-	}
+        return $array;
+    }
 
-	/**
-	 * Checks whether a calendar-name, the user wants to create, violates
-	 * the reserved name for calendar uris
-	 *
-	 * @param string $calendarUri
-	 * @return bool
-	 */
-	public static function doesViolateReservedName(string $calendarUri):bool {
-		return strpos($calendarUri, self::PREFIX) === 0;
-	}
+    /**
+     * Checks whether a calendar-name, the user wants to create, violates
+     * the reserved name for calendar uris
+     *
+     * @param string $calendarUri
+     * @return bool
+     */
+    public static function doesViolateReservedName(string $calendarUri):bool {
+        return strpos($calendarUri, self::PREFIX) === 0;
+    }
 }

apps/dav/lib/Files/Sharing/FilesDropPlugin.php

Indentation +38 added lines, -38 removed lines

@@ -35,51 +35,51 @@
  */
 class FilesDropPlugin extends ServerPlugin {
 
-	/** @var View */
-	private $view;
+    /** @var View */
+    private $view;
 
-	/** @var bool */
-	private $enabled = false;
+    /** @var bool */
+    private $enabled = false;
 
-	/**
-	 * @param View $view
-	 */
-	public function setView($view) {
-		$this->view = $view;
-	}
+    /**
+     * @param View $view
+     */
+    public function setView($view) {
+        $this->view = $view;
+    }
 
-	public function enable() {
-		$this->enabled = true;
-	}
+    public function enable() {
+        $this->enabled = true;
+    }
 
 
-	/**
-	 * This initializes the plugin.
-	 *
-	 * @param \Sabre\DAV\Server $server Sabre server
-	 *
-	 * @return void
-	 * @throws MethodNotAllowed
-	 */
-	public function initialize(\Sabre\DAV\Server $server) {
-		$server->on('beforeMethod:*', [$this, 'beforeMethod'], 999);
-		$this->enabled = false;
-	}
+    /**
+     * This initializes the plugin.
+     *
+     * @param \Sabre\DAV\Server $server Sabre server
+     *
+     * @return void
+     * @throws MethodNotAllowed
+     */
+    public function initialize(\Sabre\DAV\Server $server) {
+        $server->on('beforeMethod:*', [$this, 'beforeMethod'], 999);
+        $this->enabled = false;
+    }
 
-	public function beforeMethod(RequestInterface $request, ResponseInterface $response) {
-		if (!$this->enabled) {
-			return;
-		}
+    public function beforeMethod(RequestInterface $request, ResponseInterface $response) {
+        if (!$this->enabled) {
+            return;
+        }
 
-		if ($request->getMethod() !== 'PUT') {
-			throw new MethodNotAllowed('Only PUT is allowed on files drop');
-		}
+        if ($request->getMethod() !== 'PUT') {
+            throw new MethodNotAllowed('Only PUT is allowed on files drop');
+        }
 
-		$path = explode('/', $request->getPath());
-		$path = array_pop($path);
+        $path = explode('/', $request->getPath());
+        $path = array_pop($path);
 
-		$newName = \OC_Helper::buildNotExistingFileNameForView('/', $path, $this->view);
-		$url = $request->getBaseUrl() . $newName;
-		$request->setUrl($url);
-	}
+        $newName = \OC_Helper::buildNotExistingFileNameForView('/', $path, $this->view);
+        $url = $request->getBaseUrl() . $newName;
+        $request->setUrl($url);
+    }
 }

apps/dav/lib/Migration/RefreshWebcalJobRegistrar.php

Indentation +41 added lines, -41 removed lines

@@ -34,52 +34,52 @@
 
 class RefreshWebcalJobRegistrar implements IRepairStep {
 
-	/** @var IDBConnection */
-	private $connection;
+    /** @var IDBConnection */
+    private $connection;
 
-	/** @var IJobList */
-	private $jobList;
+    /** @var IJobList */
+    private $jobList;
 
-	/**
-	 * FixBirthdayCalendarComponent constructor.
-	 *
-	 * @param IDBConnection $connection
-	 * @param IJobList $jobList
-	 */
-	public function __construct(IDBConnection $connection, IJobList $jobList) {
-		$this->connection = $connection;
-		$this->jobList = $jobList;
-	}
+    /**
+     * FixBirthdayCalendarComponent constructor.
+     *
+     * @param IDBConnection $connection
+     * @param IJobList $jobList
+     */
+    public function __construct(IDBConnection $connection, IJobList $jobList) {
+        $this->connection = $connection;
+        $this->jobList = $jobList;
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	public function getName() {
-		return 'Registering background jobs to update cache for webcal calendars';
-	}
+    /**
+     * @inheritdoc
+     */
+    public function getName() {
+        return 'Registering background jobs to update cache for webcal calendars';
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	public function run(IOutput $output) {
-		$query = $this->connection->getQueryBuilder();
-		$query->select(['principaluri', 'uri'])
-			->from('calendarsubscriptions');
-		$stmt = $query->execute();
+    /**
+     * @inheritdoc
+     */
+    public function run(IOutput $output) {
+        $query = $this->connection->getQueryBuilder();
+        $query->select(['principaluri', 'uri'])
+            ->from('calendarsubscriptions');
+        $stmt = $query->execute();
 
-		$count = 0;
-		while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
-			$args = [
-				'principaluri' => $row['principaluri'],
-				'uri' => $row['uri'],
-			];
+        $count = 0;
+        while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
+            $args = [
+                'principaluri' => $row['principaluri'],
+                'uri' => $row['uri'],
+            ];
 
-			if (!$this->jobList->has(RefreshWebcalJob::class, $args)) {
-				$this->jobList->add(RefreshWebcalJob::class, $args);
-				$count++;
-			}
-		}
+            if (!$this->jobList->has(RefreshWebcalJob::class, $args)) {
+                $this->jobList->add(RefreshWebcalJob::class, $args);
+                $count++;
+            }
+        }
 
-		$output->info("Added $count background jobs to update webcal calendars");
-	}
+        $output->info("Added $count background jobs to update webcal calendars");
+    }
 }

apps/workflowengine/lib/Helper/ScopeContext.php

Indentation +41 added lines, -41 removed lines

@@ -28,51 +28,51 @@
 use OCP\WorkflowEngine\IManager;
 
 class ScopeContext {
-	/** @var int */
-	private $scope;
-	/** @var string */
-	private $scopeId;
-	/** @var string */
-	private $hash;
+    /** @var int */
+    private $scope;
+    /** @var string */
+    private $scopeId;
+    /** @var string */
+    private $hash;
 
-	public function __construct(int $scope, string $scopeId = null) {
-		$this->scope = $this->evaluateScope($scope);
-		$this->scopeId = $this->evaluateScopeId($scopeId);
-	}
+    public function __construct(int $scope, string $scopeId = null) {
+        $this->scope = $this->evaluateScope($scope);
+        $this->scopeId = $this->evaluateScopeId($scopeId);
+    }
 
-	private function evaluateScope(int $scope): int {
-		if (in_array($scope, [IManager::SCOPE_ADMIN, IManager::SCOPE_USER], true)) {
-			return $scope;
-		}
-		throw new \InvalidArgumentException('Invalid scope');
-	}
+    private function evaluateScope(int $scope): int {
+        if (in_array($scope, [IManager::SCOPE_ADMIN, IManager::SCOPE_USER], true)) {
+            return $scope;
+        }
+        throw new \InvalidArgumentException('Invalid scope');
+    }
 
-	private function evaluateScopeId(string $scopeId = null): string {
-		if ($this->scope === IManager::SCOPE_USER
-			&& trim((string)$scopeId) === '') {
-			throw new \InvalidArgumentException('user scope requires a user id');
-		}
-		return trim((string)$scopeId);
-	}
+    private function evaluateScopeId(string $scopeId = null): string {
+        if ($this->scope === IManager::SCOPE_USER
+            && trim((string)$scopeId) === '') {
+            throw new \InvalidArgumentException('user scope requires a user id');
+        }
+        return trim((string)$scopeId);
+    }
 
-	/**
-	 * @return int
-	 */
-	public function getScope(): int {
-		return $this->scope;
-	}
+    /**
+     * @return int
+     */
+    public function getScope(): int {
+        return $this->scope;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getScopeId(): string {
-		return $this->scopeId;
-	}
+    /**
+     * @return string
+     */
+    public function getScopeId(): string {
+        return $this->scopeId;
+    }
 
-	public function getHash(): string {
-		if ($this->hash === null) {
-			$this->hash = \hash('sha256', $this->getScope() . '::' . $this->getScopeId());
-		}
-		return $this->hash;
-	}
+    public function getHash(): string {
+        if ($this->hash === null) {
+            $this->hash = \hash('sha256', $this->getScope() . '::' . $this->getScopeId());
+        }
+        return $this->hash;
+    }
 }

apps/workflowengine/lib/Helper/LogContext.php

Indentation +56 added lines, -56 removed lines

@@ -30,67 +30,67 @@
 use OCP\WorkflowEngine\IOperation;
 
 class LogContext {
-	/** @var array */
-	protected $details;
+    /** @var array */
+    protected $details;
 
-	public function setDescription(string $description): LogContext {
-		$this->details['message'] = $description;
-		return $this;
-	}
+    public function setDescription(string $description): LogContext {
+        $this->details['message'] = $description;
+        return $this;
+    }
 
-	public function setScopes(array $scopes): LogContext {
-		$this->details['scopes'] = [];
-		foreach ($scopes as $scope) {
-			if ($scope instanceof ScopeContext) {
-				switch ($scope->getScope()) {
-					case IManager::SCOPE_ADMIN:
-						$this->details['scopes'][] = ['scope' => 'admin'];
-						break;
-					case IManager::SCOPE_USER:
-						$this->details['scopes'][] = [
-							'scope' => 'user',
-							'uid' => $scope->getScopeId(),
-						];
-						break;
-					default:
-						continue 2;
-				}
-			}
-		}
-		return $this;
-	}
+    public function setScopes(array $scopes): LogContext {
+        $this->details['scopes'] = [];
+        foreach ($scopes as $scope) {
+            if ($scope instanceof ScopeContext) {
+                switch ($scope->getScope()) {
+                    case IManager::SCOPE_ADMIN:
+                        $this->details['scopes'][] = ['scope' => 'admin'];
+                        break;
+                    case IManager::SCOPE_USER:
+                        $this->details['scopes'][] = [
+                            'scope' => 'user',
+                            'uid' => $scope->getScopeId(),
+                        ];
+                        break;
+                    default:
+                        continue 2;
+                }
+            }
+        }
+        return $this;
+    }
 
-	public function setOperation(?IOperation $operation): LogContext {
-		if ($operation instanceof IOperation) {
-			$this->details['operation'] = [
-				'class' => get_class($operation),
-				'name' => $operation->getDisplayName(),
-			];
-		}
-		return $this;
-	}
+    public function setOperation(?IOperation $operation): LogContext {
+        if ($operation instanceof IOperation) {
+            $this->details['operation'] = [
+                'class' => get_class($operation),
+                'name' => $operation->getDisplayName(),
+            ];
+        }
+        return $this;
+    }
 
-	public function setEntity(?IEntity $entity): LogContext {
-		if ($entity instanceof IEntity) {
-			$this->details['entity'] = [
-				'class' => get_class($entity),
-				'name' => $entity->getName(),
-			];
-		}
-		return $this;
-	}
+    public function setEntity(?IEntity $entity): LogContext {
+        if ($entity instanceof IEntity) {
+            $this->details['entity'] = [
+                'class' => get_class($entity),
+                'name' => $entity->getName(),
+            ];
+        }
+        return $this;
+    }
 
-	public function setConfiguration(array $configuration): LogContext {
-		$this->details['configuration'] = $configuration;
-		return $this;
-	}
+    public function setConfiguration(array $configuration): LogContext {
+        $this->details['configuration'] = $configuration;
+        return $this;
+    }
 
-	public function setEventName(string $eventName): LogContext {
-		$this->details['eventName'] = $eventName;
-		return $this;
-	}
+    public function setEventName(string $eventName): LogContext {
+        $this->details['eventName'] = $eventName;
+        return $this;
+    }
 
-	public function getDetails(): array {
-		return $this->details;
-	}
+    public function getDetails(): array {
+        return $this->details;
+    }
 }