nextcloud / server

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;
+    }
 }

apps/workflowengine/lib/Controller/GlobalWorkflowsController.php

Indentation +8 added lines, -8 removed lines

@@ -30,13 +30,13 @@
 
 class GlobalWorkflowsController extends AWorkflowController {
 
-	/** @var ScopeContext */
-	private $scopeContext;
+    /** @var ScopeContext */
+    private $scopeContext;
 
-	protected function getScopeContext(): ScopeContext {
-		if ($this->scopeContext === null) {
-			$this->scopeContext = new ScopeContext(IManager::SCOPE_ADMIN);
-		}
-		return $this->scopeContext;
-	}
+    protected function getScopeContext(): ScopeContext {
+        if ($this->scopeContext === null) {
+            $this->scopeContext = new ScopeContext(IManager::SCOPE_ADMIN);
+        }
+        return $this->scopeContext;
+    }
 }

apps/files_sharing/lib/Listener/UserShareAcceptanceListener.php

Indentation +36 added lines, -36 removed lines

@@ -38,47 +38,47 @@
 
 class UserShareAcceptanceListener implements IEventListener {
 
-	/** @var IConfig */
-	private $config;
-	/** @var IManager */
-	private $shareManager;
-	/** @var IGroupManager */
-	private $groupManager;
+    /** @var IConfig */
+    private $config;
+    /** @var IManager */
+    private $shareManager;
+    /** @var IGroupManager */
+    private $groupManager;
 
-	public function __construct(IConfig $config, IManager $shareManager, IGroupManager $groupManager) {
-		$this->config = $config;
-		$this->shareManager = $shareManager;
-		$this->groupManager = $groupManager;
-	}
+    public function __construct(IConfig $config, IManager $shareManager, IGroupManager $groupManager) {
+        $this->config = $config;
+        $this->shareManager = $shareManager;
+        $this->groupManager = $groupManager;
+    }
 
-	public function handle(Event $event): void {
-		if (!($event instanceof ShareCreatedEvent)) {
-			return;
-		}
+    public function handle(Event $event): void {
+        if (!($event instanceof ShareCreatedEvent)) {
+            return;
+        }
 
-		$share = $event->getShare();
+        $share = $event->getShare();
 
-		if ($share->getShareType() === IShare::TYPE_USER) {
-			$this->handleAutoAccept($share, $share->getSharedWith());
-		} elseif ($share->getShareType() === IShare::TYPE_GROUP) {
-			$group = $this->groupManager->get($share->getSharedWith());
+        if ($share->getShareType() === IShare::TYPE_USER) {
+            $this->handleAutoAccept($share, $share->getSharedWith());
+        } elseif ($share->getShareType() === IShare::TYPE_GROUP) {
+            $group = $this->groupManager->get($share->getSharedWith());
 
-			if ($group === null) {
-				return;
-			}
+            if ($group === null) {
+                return;
+            }
 
-			$users = $group->getUsers();
-			foreach ($users as $user) {
-				$this->handleAutoAccept($share, $user->getUID());
-			}
-		}
-	}
+            $users = $group->getUsers();
+            foreach ($users as $user) {
+                $this->handleAutoAccept($share, $user->getUID());
+            }
+        }
+    }
 
-	private function handleAutoAccept(IShare $share, string $userId) {
-		$defaultAcceptSystemConfig = $this->config->getSystemValueBool('sharing.enable_share_accept', false) ? 'no' : 'yes';
-		$acceptDefault = $this->config->getUserValue($userId, Application::APP_ID, 'default_accept', $defaultAcceptSystemConfig) === 'yes';
-		if (!$this->config->getSystemValueBool('sharing.force_share_accept', false) && $acceptDefault) {
-			$this->shareManager->acceptShare($share, $userId);
-		}
-	}
+    private function handleAutoAccept(IShare $share, string $userId) {
+        $defaultAcceptSystemConfig = $this->config->getSystemValueBool('sharing.enable_share_accept', false) ? 'no' : 'yes';
+        $acceptDefault = $this->config->getUserValue($userId, Application::APP_ID, 'default_accept', $defaultAcceptSystemConfig) === 'yes';
+        if (!$this->config->getSystemValueBool('sharing.force_share_accept', false) && $acceptDefault) {
+            $this->shareManager->acceptShare($share, $userId);
+        }
+    }
 }

apps/files_external/lib/Lib/Auth/OAuth1/OAuth1.php

Indentation +19 added lines, -19 removed lines

@@ -32,23 +32,23 @@
  * OAuth1 authentication
  */
 class OAuth1 extends AuthMechanism {
-	public function __construct(IL10N $l) {
-		$this
-			->setIdentifier('oauth1::oauth1')
-			->setScheme(self::SCHEME_OAUTH1)
-			->setText($l->t('OAuth1'))
-			->addParameters([
-				(new DefinitionParameter('configured', 'configured'))
-					->setType(DefinitionParameter::VALUE_HIDDEN),
-				new DefinitionParameter('app_key', $l->t('App key')),
-				(new DefinitionParameter('app_secret', $l->t('App secret')))
-					->setType(DefinitionParameter::VALUE_PASSWORD),
-				(new DefinitionParameter('token', 'token'))
-					->setType(DefinitionParameter::VALUE_HIDDEN),
-				(new DefinitionParameter('token_secret', 'token_secret'))
-					->setType(DefinitionParameter::VALUE_HIDDEN),
-			])
-			->addCustomJs('oauth1')
-		;
-	}
+    public function __construct(IL10N $l) {
+        $this
+            ->setIdentifier('oauth1::oauth1')
+            ->setScheme(self::SCHEME_OAUTH1)
+            ->setText($l->t('OAuth1'))
+            ->addParameters([
+                (new DefinitionParameter('configured', 'configured'))
+                    ->setType(DefinitionParameter::VALUE_HIDDEN),
+                new DefinitionParameter('app_key', $l->t('App key')),
+                (new DefinitionParameter('app_secret', $l->t('App secret')))
+                    ->setType(DefinitionParameter::VALUE_PASSWORD),
+                (new DefinitionParameter('token', 'token'))
+                    ->setType(DefinitionParameter::VALUE_HIDDEN),
+                (new DefinitionParameter('token_secret', 'token_secret'))
+                    ->setType(DefinitionParameter::VALUE_HIDDEN),
+            ])
+            ->addCustomJs('oauth1')
+        ;
+    }
 }