nextcloud / server

apps/files/lib/Controller/DirectEditingViewController.php

Indentation +28 added lines, -28 removed lines

@@ -36,37 +36,37 @@
 
 class DirectEditingViewController extends Controller {
 
-	/** @var IEventDispatcher */
-	private $eventDispatcher;
+    /** @var IEventDispatcher */
+    private $eventDispatcher;
 
-	/** @var IManager */
-	private $directEditingManager;
+    /** @var IManager */
+    private $directEditingManager;
 
-	/** @var ILogger */
-	private $logger;
+    /** @var ILogger */
+    private $logger;
 
-	public function __construct($appName, IRequest $request, IEventDispatcher $eventDispatcher, IManager $manager, ILogger $logger) {
-		parent::__construct($appName, $request);
+    public function __construct($appName, IRequest $request, IEventDispatcher $eventDispatcher, IManager $manager, ILogger $logger) {
+        parent::__construct($appName, $request);
 
-		$this->eventDispatcher = $eventDispatcher;
-		$this->directEditingManager = $manager;
-		$this->logger = $logger;
-	}
+        $this->eventDispatcher = $eventDispatcher;
+        $this->directEditingManager = $manager;
+        $this->logger = $logger;
+    }
 
-	/**
-	 * @PublicPage
-	 * @NoCSRFRequired
-	 *
-	 * @param string $token
-	 * @return Response
-	 */
-	public function edit(string $token): Response {
-		$this->eventDispatcher->dispatchTyped(new RegisterDirectEditorEvent($this->directEditingManager));
-		try {
-			return $this->directEditingManager->edit($token);
-		} catch (Exception $e) {
-			$this->logger->logException($e);
-			return new NotFoundResponse();
-		}
-	}
+    /**
+     * @PublicPage
+     * @NoCSRFRequired
+     *
+     * @param string $token
+     * @return Response
+     */
+    public function edit(string $token): Response {
+        $this->eventDispatcher->dispatchTyped(new RegisterDirectEditorEvent($this->directEditingManager));
+        try {
+            return $this->directEditingManager->edit($token);
+        } catch (Exception $e) {
+            $this->logger->logException($e);
+            return new NotFoundResponse();
+        }
+    }
 }

lib/public/Files/File.php

Indentation +62 added lines, -62 removed lines

@@ -42,72 +42,72 @@
  * @since 6.0.0
  */
 interface File extends Node {
-	/**
-	 * Get the content of the file as string
-	 *
-	 * @return string
-	 * @throws NotPermittedException
-	 * @throws LockedException
-	 * @since 6.0.0
-	 */
-	public function getContent();
+    /**
+     * Get the content of the file as string
+     *
+     * @return string
+     * @throws NotPermittedException
+     * @throws LockedException
+     * @since 6.0.0
+     */
+    public function getContent();
 
-	/**
-	 * Write to the file from string data
-	 *
-	 * @param string|resource $data
-	 * @throws NotPermittedException
-	 * @throws GenericFileException
-	 * @throws LockedException
-	 * @since 6.0.0
-	 */
-	public function putContent($data);
+    /**
+     * Write to the file from string data
+     *
+     * @param string|resource $data
+     * @throws NotPermittedException
+     * @throws GenericFileException
+     * @throws LockedException
+     * @since 6.0.0
+     */
+    public function putContent($data);
 
-	/**
-	 * Get the mimetype of the file
-	 *
-	 * @return string
-	 * @since 6.0.0
-	 */
-	public function getMimeType();
+    /**
+     * Get the mimetype of the file
+     *
+     * @return string
+     * @since 6.0.0
+     */
+    public function getMimeType();
 
-	/**
-	 * Open the file as stream, resulting resource can be operated as stream like the result from php's own fopen
-	 *
-	 * @param string $mode
-	 * @return resource
-	 * @throws NotPermittedException
-	 * @throws LockedException
-	 * @since 6.0.0
-	 */
-	public function fopen($mode);
+    /**
+     * Open the file as stream, resulting resource can be operated as stream like the result from php's own fopen
+     *
+     * @param string $mode
+     * @return resource
+     * @throws NotPermittedException
+     * @throws LockedException
+     * @since 6.0.0
+     */
+    public function fopen($mode);
 
-	/**
-	 * Compute the hash of the file
-	 * Type of hash is set with $type and can be anything supported by php's hash_file
-	 *
-	 * @param string $type
-	 * @param bool $raw
-	 * @return string
-	 * @since 6.0.0
-	 */
-	public function hash($type, $raw = false);
+    /**
+     * Compute the hash of the file
+     * Type of hash is set with $type and can be anything supported by php's hash_file
+     *
+     * @param string $type
+     * @param bool $raw
+     * @return string
+     * @since 6.0.0
+     */
+    public function hash($type, $raw = false);
 
-	/**
-	 * Get the stored checksum for this file
-	 *
-	 * @return string
-	 * @since 9.0.0
-	 * @throws InvalidPathException
-	 * @throws NotFoundException
-	 */
-	public function getChecksum();
+    /**
+     * Get the stored checksum for this file
+     *
+     * @return string
+     * @since 9.0.0
+     * @throws InvalidPathException
+     * @throws NotFoundException
+     */
+    public function getChecksum();
 
-	/**
-	 * Get the extension of this file
-	 *
-	 * @return string
-	 * @since 15.0.0
-	 */
-	public function getExtension(): string;
+    /**
+     * Get the extension of this file
+     *
+     * @return string
+     * @since 15.0.0
+     */
+    public function getExtension(): string;
 }

lib/public/WorkflowEngine/IEntity.php

Indentation +37 added lines, -37 removed lines

@@ -40,46 +40,46 @@
  */
 interface IEntity {
 
-	/**
-	 * returns a translated name to be presented in the web interface.
-	 *
-	 * Example: "File" (en), "Dosiero" (eo)
-	 *
-	 * @since 18.0.0
-	 */
-	public function getName(): string;
+    /**
+     * returns a translated name to be presented in the web interface.
+     *
+     * Example: "File" (en), "Dosiero" (eo)
+     *
+     * @since 18.0.0
+     */
+    public function getName(): string;
 
-	/**
-	 * returns the URL to the icon of the entity for display in the web interface.
-	 *
-	 * Usually, the implementation would utilize the `imagePath()` method of the
-	 * `\OCP\IURLGenerator` instance and simply return its result.
-	 *
-	 * Example implementation: return $this->urlGenerator->imagePath('myApp', 'cat.svg');
-	 *
-	 * @since 18.0.0
-	 */
-	public function getIcon(): string;
+    /**
+     * returns the URL to the icon of the entity for display in the web interface.
+     *
+     * Usually, the implementation would utilize the `imagePath()` method of the
+     * `\OCP\IURLGenerator` instance and simply return its result.
+     *
+     * Example implementation: return $this->urlGenerator->imagePath('myApp', 'cat.svg');
+     *
+     * @since 18.0.0
+     */
+    public function getIcon(): string;
 
-	/**
-	 * returns a list of supported events
-	 *
-	 * @return IEntityEvent[]
-	 * @since 18.0.0
-	 */
-	public function getEvents(): array;
+    /**
+     * returns a list of supported events
+     *
+     * @return IEntityEvent[]
+     * @since 18.0.0
+     */
+    public function getEvents(): array;
 
-	/**
-	 * @since 18.0.0
-	 */
-	public function prepareRuleMatcher(IRuleMatcher $ruleMatcher, string $eventName, Event $event): void;
+    /**
+     * @since 18.0.0
+     */
+    public function prepareRuleMatcher(IRuleMatcher $ruleMatcher, string $eventName, Event $event): void;
 
-	/**
-	 * returns whether the provided user id is allowed to run a flow against
-	 * the known context
-	 *
-	 * @since 18.0.0
-	 */
-	public function isLegitimatedForUserId(string $userId): bool;
+    /**
+     * returns whether the provided user id is allowed to run a flow against
+     * the known context
+     *
+     * @since 18.0.0
+     */
+    public function isLegitimatedForUserId(string $userId): bool;
 
 }

lib/public/WorkflowEngine/EntityContext/IUrl.php

Indentation +6 added lines, -6 removed lines

@@ -32,10 +32,10 @@
  * @since 18.0.0
  */
 interface IUrl {
-	/**
-	 * returns a URL that is related to the entity, e.g. the link to a share
-	 *
-	 * @since 18.0.0
-	 */
-	public function getUrl(): string;
+    /**
+     * returns a URL that is related to the entity, e.g. the link to a share
+     *
+     * @since 18.0.0
+     */
+    public function getUrl(): string;
 }

lib/public/WorkflowEngine/EntityContext/IDisplayName.php

Indentation +6 added lines, -6 removed lines

@@ -32,10 +32,10 @@
  * @since 18.0.0
  */
 interface IDisplayName {
-	/**
-	 * returns the end user facing name of the object related to the entity
-	 *
-	 * @since 18.0.0
-	 */
-	public function getDisplayName(): string;
+    /**
+     * returns the end user facing name of the object related to the entity
+     *
+     * @since 18.0.0
+     */
+    public function getDisplayName(): string;
 }

lib/public/WorkflowEngine/EntityContext/IIcon.php

Indentation +7 added lines, -7 removed lines

@@ -32,11 +32,11 @@
  * @since 18.0.0
  */
 interface IIcon {
-	/**
-	 * returns a URL to an icon that is related to the entity, for instance
-	 * a group icon for groups.
-	 *
-	 * @since 18.0.0
-	 */
-	public function getIconUrl(): string;
+    /**
+     * returns a URL to an icon that is related to the entity, for instance
+     * a group icon for groups.
+     *
+     * @since 18.0.0
+     */
+    public function getIconUrl(): string;
 }

lib/public/WorkflowEngine/EntityContext/IDisplayText.php

Indentation +11 added lines, -11 removed lines

@@ -33,15 +33,15 @@
  */
 interface IDisplayText {
 
-	/**
-	 * returns translated text used for display to the end user. For instance,
-	 * it can describe the event in a human readable way.
-	 *
-	 * The entity may react to a verbosity level that is provided. With the
-	 * basic level, 0, it would return brief information, and more with higher
-	 * numbers. All information shall be shown at a level of 3.
-	 *
-	 * @since 18.0.0
-	 */
-	public function getDisplayText(int $verbosity = 0): string;
+    /**
+     * returns translated text used for display to the end user. For instance,
+     * it can describe the event in a human readable way.
+     *
+     * The entity may react to a verbosity level that is provided. With the
+     * basic level, 0, it would return brief information, and more with higher
+     * numbers. All information shall be shown at a level of 3.
+     *
+     * @since 18.0.0
+     */
+    public function getDisplayText(int $verbosity = 0): string;
 }

apps/files/lib/Db/TransferOwnership.php

Indentation +14 added lines, -14 removed lines

@@ -37,24 +37,24 @@
  * @method string getNodeName()
  */
 class TransferOwnership extends Entity {
-	/** @var string */
-	protected $sourceUser;
+    /** @var string */
+    protected $sourceUser;
 
-	/** @var string */
-	protected $targetUser;
+    /** @var string */
+    protected $targetUser;
 
-	/** @var integer */
-	protected $fileId;
+    /** @var integer */
+    protected $fileId;
 
-	/** @var string */
-	protected $nodeName;
+    /** @var string */
+    protected $nodeName;
 
-	public function __construct() {
-		$this->addType('sourceUser', 'string');
-		$this->addType('targetUser', 'string');
-		$this->addType('fileId', 'integer');
-		$this->addType('nodeName', 'string');
-	}
+    public function __construct() {
+        $this->addType('sourceUser', 'string');
+        $this->addType('targetUser', 'string');
+        $this->addType('fileId', 'integer');
+        $this->addType('nodeName', 'string');
+    }
 
 
 }

apps/files/lib/BackgroundJob/TransferOwnership.php

Spacing +5 added lines, -5 removed lines

@@ -98,7 +98,7 @@ 
 		$destinationUserObject = $this->userManager->get($destinationUser);
 
 		if (!$sourceUserObject instanceof IUser) {
-			$this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
+			$this->logger->alert('Could not transfer ownership: Unknown source user '.$sourceUser);
 			$this->failedNotication($transfer);
 			return;
 		}
@@ -136,7 +136,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -149,7 +149,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 
@@ -164,7 +164,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -177,7 +177,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 }

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