nextcloud / server

lib/public/Broadcast/Events/IBroadcastEvent.php

Indentation +22 added lines, -22 removed lines

@@ -31,27 +31,27 @@
  */
 interface IBroadcastEvent {
 
-	/**
-	 * @return string the name of the event
-	 * @since 18.0.0
-	 */
-	public function getName(): string;
-
-	/**
-	 * @return string[]
-	 * @since 18.0.0
-	 */
-	public function getUids(): array;
-
-	/**
-	 * @return JsonSerializable the data to be sent to the client
-	 * @since 18.0.0
-	 */
-	public function getPayload(): JsonSerializable;
-
-	/**
-	 * @since 18.0.0
-	 */
-	public function setBroadcasted(): void;
+    /**
+     * @return string the name of the event
+     * @since 18.0.0
+     */
+    public function getName(): string;
+
+    /**
+     * @return string[]
+     * @since 18.0.0
+     */
+    public function getUids(): array;
+
+    /**
+     * @return JsonSerializable the data to be sent to the client
+     * @since 18.0.0
+     */
+    public function getPayload(): JsonSerializable;
+
+    /**
+     * @since 18.0.0
+     */
+    public function setBroadcasted(): void;
 
 }

lib/public/EventDispatcher/ABroadcastedEvent.php

Indentation +32 added lines, -32 removed lines

@@ -31,41 +31,41 @@
  */
 abstract class ABroadcastedEvent extends Event implements JsonSerializable {
 
-	/**
-	 * @since 18.0.0
-	 */
-	private $broadcasted = false;
+    /**
+     * @since 18.0.0
+     */
+    private $broadcasted = false;
 
-	/**
-	 * Get the name of the event, as received on the client-side
-	 *
-	 * Uses the fully qualified event class name by default
-	 *
-	 * @return string
-	 * @since 18.0.0
-	 */
-	public function broadcastAs(): string {
-		return get_class($this);
-	}
+    /**
+     * Get the name of the event, as received on the client-side
+     *
+     * Uses the fully qualified event class name by default
+     *
+     * @return string
+     * @since 18.0.0
+     */
+    public function broadcastAs(): string {
+        return get_class($this);
+    }
 
-	/**
-	 * @return string[]
-	 * @since 18.0.0
-	 */
-	abstract public function getUids(): array;
+    /**
+     * @return string[]
+     * @since 18.0.0
+     */
+    abstract public function getUids(): array;
 
-	/**
-	 * @since 18.0.0
-	 */
-	public function setBroadcasted(): void {
-		$this->broadcasted = true;
-	}
+    /**
+     * @since 18.0.0
+     */
+    public function setBroadcasted(): void {
+        $this->broadcasted = true;
+    }
 
-	/**
-	 * @since 18.0.0
-	 */
-	public function isBroadcasted(): bool {
-		return $this->broadcasted;
-	}
+    /**
+     * @since 18.0.0
+     */
+    public function isBroadcasted(): bool {
+        return $this->broadcasted;
+    }
 
 }

lib/private/Broadcast/Events/BroadcastEvent.php

Indentation +18 added lines, -18 removed lines

@@ -31,29 +31,29 @@
 
 class BroadcastEvent extends Event implements IBroadcastEvent {
 
-	/** @var ABroadcastedEvent */
-	private $event;
+    /** @var ABroadcastedEvent */
+    private $event;
 
-	public function __construct(ABroadcastedEvent $event) {
-		parent::__construct();
+    public function __construct(ABroadcastedEvent $event) {
+        parent::__construct();
 
-		$this->event = $event;
-	}
+        $this->event = $event;
+    }
 
-	public function getName(): string {
-		return $this->event->broadcastAs();
-	}
+    public function getName(): string {
+        return $this->event->broadcastAs();
+    }
 
-	public function getUids(): array {
-		return $this->event->getUids();
-	}
+    public function getUids(): array {
+        return $this->event->getUids();
+    }
 
-	public function getPayload(): JsonSerializable {
-		return $this->event;
-	}
+    public function getPayload(): JsonSerializable {
+        return $this->event;
+    }
 
-	public function setBroadcasted(): void {
-		$this->event->setBroadcasted();
-	}
+    public function setBroadcasted(): void {
+        $this->event->setBroadcasted();
+    }
 
 }

apps/files_sharing/lib/Command/ExiprationNotification.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 		//Current time
 		$minTime = $this->time->getDateTime();
 		$minTime->add(new \DateInterval('P1D'));
-		$minTime->setTime(0,0,0);
+		$minTime->setTime(0, 0, 0);
 
 		$maxTime = clone $minTime;
 		$maxTime->setTime(23, 59, 59);

Indentation +50 added lines, -50 removed lines

@@ -36,64 +36,64 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ExiprationNotification extends Command {
-	/** @var NotificationManager */
-	private $notificationManager;
-	/** @var IDBConnection */
-	private $connection;
-	/** @var ITimeFactory */
-	private $time;
-	/** @var ShareManager */
-	private $shareManager;
+    /** @var NotificationManager */
+    private $notificationManager;
+    /** @var IDBConnection */
+    private $connection;
+    /** @var ITimeFactory */
+    private $time;
+    /** @var ShareManager */
+    private $shareManager;
 
-	public function __construct(ITimeFactory $time,
-								NotificationManager $notificationManager,
-								IDBConnection $connection,
-								ShareManager $shareManager) {
-		parent::__construct();
+    public function __construct(ITimeFactory $time,
+                                NotificationManager $notificationManager,
+                                IDBConnection $connection,
+                                ShareManager $shareManager) {
+        parent::__construct();
 
-		$this->notificationManager = $notificationManager;
-		$this->connection = $connection;
-		$this->time = $time;
-		$this->shareManager = $shareManager;
-	}
+        $this->notificationManager = $notificationManager;
+        $this->connection = $connection;
+        $this->time = $time;
+        $this->shareManager = $shareManager;
+    }
 
-	protected function configure() {
-		$this
-			->setName('sharing:expiration-notification')
-			->setDescription('Notify share initiators when a share will expire the next day.');
-	}
+    protected function configure() {
+        $this
+            ->setName('sharing:expiration-notification')
+            ->setDescription('Notify share initiators when a share will expire the next day.');
+    }
 
-	public function execute(InputInterface $input, OutputInterface $output): int {
-		//Current time
-		$minTime = $this->time->getDateTime();
-		$minTime->add(new \DateInterval('P1D'));
-		$minTime->setTime(0,0,0);
+    public function execute(InputInterface $input, OutputInterface $output): int {
+        //Current time
+        $minTime = $this->time->getDateTime();
+        $minTime->add(new \DateInterval('P1D'));
+        $minTime->setTime(0,0,0);
 
-		$maxTime = clone $minTime;
-		$maxTime->setTime(23, 59, 59);
+        $maxTime = clone $minTime;
+        $maxTime->setTime(23, 59, 59);
 
-		$shares = $this->shareManager->getAllShares();
+        $shares = $this->shareManager->getAllShares();
 
-		$now = $this->time->getDateTime();
+        $now = $this->time->getDateTime();
 
-		/** @var IShare $share */
-		foreach ($shares as $share) {
-			if ($share->getExpirationDate() === null
-				|| $share->getExpirationDate()->getTimestamp() < $minTime->getTimestamp()
-				|| $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()) {
-				continue;
-			}
+        /** @var IShare $share */
+        foreach ($shares as $share) {
+            if ($share->getExpirationDate() === null
+                || $share->getExpirationDate()->getTimestamp() < $minTime->getTimestamp()
+                || $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()) {
+                continue;
+            }
 
-			$notification = $this->notificationManager->createNotification();
-			$notification->setApp('files_sharing')
-				->setDateTime($now)
-				->setObject('share', $share->getFullId())
-				->setSubject('expiresTomorrow');
+            $notification = $this->notificationManager->createNotification();
+            $notification->setApp('files_sharing')
+                ->setDateTime($now)
+                ->setObject('share', $share->getFullId())
+                ->setSubject('expiresTomorrow');
 
-			// Only send to initiator for now
-			$notification->setUser($share->getSharedBy());
-			$this->notificationManager->notify($notification);
-		}
-		return 0;
-	}
+            // Only send to initiator for now
+            $notification->setUser($share->getSharedBy());
+            $this->notificationManager->notify($notification);
+        }
+        return 0;
+    }
 }

lib/public/WorkflowEngine/IOperationCompat.php

Indentation +11 added lines, -11 removed lines

@@ -36,15 +36,15 @@
  * @deprecated
  */
 interface IOperationCompat {
-	/**
-	 * Like onEvent, but used with events that are not based on
-	 * \OCP\EventDispatcher\Event.
-	 *
-	 * This method is introduced for compatibility reasons and will be removed
-	 * in 2023 again.
-	 *
-	 * @since 18.0.0
-	 * @deprecated
-	 */
-	public function onEventCompat(string $eventName, $event, IRuleMatcher $ruleMatcher): void;
+    /**
+     * Like onEvent, but used with events that are not based on
+     * \OCP\EventDispatcher\Event.
+     *
+     * This method is introduced for compatibility reasons and will be removed
+     * in 2023 again.
+     *
+     * @since 18.0.0
+     * @deprecated
+     */
+    public function onEventCompat(string $eventName, $event, IRuleMatcher $ruleMatcher): void;
 }

lib/public/WorkflowEngine/IEntityCompat.php

Indentation +8 added lines, -8 removed lines

@@ -36,12 +36,12 @@
  * @deprecated
  */
 interface IEntityCompat extends IEntity {
-	/**
-	 * Like prepareRuleMatcherCompat, but works with events that are not based
-	 * on \OCP\EventDispatcher\Event.
-	 *
-	 * @since 18.0.0
-	 * @deprecated
-	 */
-	public function prepareRuleMatcherCompat(IRuleMatcher $ruleMatcher, string $eventName, $event): void;
+    /**
+     * Like prepareRuleMatcherCompat, but works with events that are not based
+     * on \OCP\EventDispatcher\Event.
+     *
+     * @since 18.0.0
+     * @deprecated
+     */
+    public function prepareRuleMatcherCompat(IRuleMatcher $ruleMatcher, string $eventName, $event): void;
 }

apps/files_trashbin/lib/Trash/TrashItem.php

Indentation +152 added lines, -152 removed lines

@@ -25,164 +25,164 @@
 use OCP\IUser;
 
 class TrashItem implements ITrashItem {
-	/** @var ITrashBackend */
-	private $backend;
-	/** @var string */
-	private $orignalLocation;
-	/** @var int */
-	private $deletedTime;
-	/** @var string */
-	private $trashPath;
-	/** @var FileInfo */
-	private $fileInfo;
-	/** @var IUser */
-	private $user;
-
-	public function __construct(
-		ITrashBackend $backend,
-		string $originalLocation,
-		int $deletedTime,
-		string $trashPath,
-		FileInfo $fileInfo,
-		IUser $user
-	) {
-		$this->backend = $backend;
-		$this->orignalLocation = $originalLocation;
-		$this->deletedTime = $deletedTime;
-		$this->trashPath = $trashPath;
-		$this->fileInfo = $fileInfo;
-		$this->user = $user;
-	}
-
-	public function getTrashBackend(): ITrashBackend {
-		return $this->backend;
-	}
-
-	public function getOriginalLocation(): string {
-		return $this->orignalLocation;
-	}
-
-	public function getDeletedTime(): int {
-		return $this->deletedTime;
-	}
-
-	public function getTrashPath(): string {
-		return $this->trashPath;
-	}
-
-	public function isRootItem(): bool {
-		return substr_count($this->getTrashPath(), '/') === 1;
-	}
-
-	public function getUser(): IUser {
-		return $this->user;
-	}
-
-	public function getEtag() {
-		return $this->fileInfo->getEtag();
-	}
-
-	public function getSize($includeMounts = true) {
-		return $this->fileInfo->getSize($includeMounts);
-	}
-
-	public function getMtime() {
-		return $this->fileInfo->getMtime();
-	}
-
-	public function getName() {
-		return $this->fileInfo->getName();
-	}
-
-	public function getInternalPath() {
-		return $this->fileInfo->getInternalPath();
-	}
-
-	public function getPath() {
-		return $this->fileInfo->getPath();
-	}
-
-	public function getMimetype() {
-		return $this->fileInfo->getMimetype();
-	}
-
-	public function getMimePart() {
-		return $this->fileInfo->getMimePart();
-	}
-
-	public function getStorage() {
-		return $this->fileInfo->getStorage();
-	}
-
-	public function getId() {
-		return $this->fileInfo->getId();
-	}
-
-	public function isEncrypted() {
-		return $this->fileInfo->isEncrypted();
-	}
-
-	public function getPermissions() {
-		return $this->fileInfo->getPermissions();
-	}
-
-	public function getType() {
-		return $this->fileInfo->getType();
-	}
-
-	public function isReadable() {
-		return $this->fileInfo->isReadable();
-	}
-
-	public function isUpdateable() {
-		return $this->fileInfo->isUpdateable();
-	}
-
-	public function isCreatable() {
-		return $this->fileInfo->isCreatable();
-	}
-
-	public function isDeletable() {
-		return $this->fileInfo->isDeletable();
-	}
-
-	public function isShareable() {
-		return $this->fileInfo->isShareable();
-	}
-
-	public function isShared() {
-		return $this->fileInfo->isShared();
-	}
+    /** @var ITrashBackend */
+    private $backend;
+    /** @var string */
+    private $orignalLocation;
+    /** @var int */
+    private $deletedTime;
+    /** @var string */
+    private $trashPath;
+    /** @var FileInfo */
+    private $fileInfo;
+    /** @var IUser */
+    private $user;
+
+    public function __construct(
+        ITrashBackend $backend,
+        string $originalLocation,
+        int $deletedTime,
+        string $trashPath,
+        FileInfo $fileInfo,
+        IUser $user
+    ) {
+        $this->backend = $backend;
+        $this->orignalLocation = $originalLocation;
+        $this->deletedTime = $deletedTime;
+        $this->trashPath = $trashPath;
+        $this->fileInfo = $fileInfo;
+        $this->user = $user;
+    }
+
+    public function getTrashBackend(): ITrashBackend {
+        return $this->backend;
+    }
+
+    public function getOriginalLocation(): string {
+        return $this->orignalLocation;
+    }
+
+    public function getDeletedTime(): int {
+        return $this->deletedTime;
+    }
+
+    public function getTrashPath(): string {
+        return $this->trashPath;
+    }
+
+    public function isRootItem(): bool {
+        return substr_count($this->getTrashPath(), '/') === 1;
+    }
+
+    public function getUser(): IUser {
+        return $this->user;
+    }
+
+    public function getEtag() {
+        return $this->fileInfo->getEtag();
+    }
+
+    public function getSize($includeMounts = true) {
+        return $this->fileInfo->getSize($includeMounts);
+    }
+
+    public function getMtime() {
+        return $this->fileInfo->getMtime();
+    }
+
+    public function getName() {
+        return $this->fileInfo->getName();
+    }
+
+    public function getInternalPath() {
+        return $this->fileInfo->getInternalPath();
+    }
+
+    public function getPath() {
+        return $this->fileInfo->getPath();
+    }
+
+    public function getMimetype() {
+        return $this->fileInfo->getMimetype();
+    }
+
+    public function getMimePart() {
+        return $this->fileInfo->getMimePart();
+    }
+
+    public function getStorage() {
+        return $this->fileInfo->getStorage();
+    }
+
+    public function getId() {
+        return $this->fileInfo->getId();
+    }
+
+    public function isEncrypted() {
+        return $this->fileInfo->isEncrypted();
+    }
+
+    public function getPermissions() {
+        return $this->fileInfo->getPermissions();
+    }
+
+    public function getType() {
+        return $this->fileInfo->getType();
+    }
+
+    public function isReadable() {
+        return $this->fileInfo->isReadable();
+    }
+
+    public function isUpdateable() {
+        return $this->fileInfo->isUpdateable();
+    }
+
+    public function isCreatable() {
+        return $this->fileInfo->isCreatable();
+    }
+
+    public function isDeletable() {
+        return $this->fileInfo->isDeletable();
+    }
+
+    public function isShareable() {
+        return $this->fileInfo->isShareable();
+    }
+
+    public function isShared() {
+        return $this->fileInfo->isShared();
+    }
 
-	public function isMounted() {
-		return $this->fileInfo->isMounted();
-	}
+    public function isMounted() {
+        return $this->fileInfo->isMounted();
+    }
 
-	public function getMountPoint() {
-		return $this->fileInfo->getMountPoint();
-	}
+    public function getMountPoint() {
+        return $this->fileInfo->getMountPoint();
+    }
 
-	public function getOwner() {
-		return $this->fileInfo->getOwner();
-	}
+    public function getOwner() {
+        return $this->fileInfo->getOwner();
+    }
 
-	public function getChecksum() {
-		return $this->fileInfo->getChecksum();
-	}
+    public function getChecksum() {
+        return $this->fileInfo->getChecksum();
+    }
 
-	public function getExtension(): string {
-		return $this->fileInfo->getExtension();
-	}
+    public function getExtension(): string {
+        return $this->fileInfo->getExtension();
+    }
 
-	public function getTitle(): string {
-		return $this->getOriginalLocation();
-	}
+    public function getTitle(): string {
+        return $this->getOriginalLocation();
+    }
 
-	public function getCreationTime(): int {
-		return $this->fileInfo->getCreationTime();
-	}
+    public function getCreationTime(): int {
+        return $this->fileInfo->getCreationTime();
+    }
 
-	public function getUploadTime(): int {
-		return $this->fileInfo->getUploadTime();
-	}
+    public function getUploadTime(): int {
+        return $this->fileInfo->getUploadTime();
+    }
 }

apps/files_sharing/lib/Notification/Notifier.php

Spacing +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 
 		$placeholders = $replacements = [];
 		foreach ($subjectParameters as $placeholder => $parameter) {
-			$placeholders[] = '{' . $placeholder . '}';
+			$placeholders[] = '{'.$placeholder.'}';
 			$replacements[] = $parameter['name'];
 		}
 

Indentation +185 added lines, -185 removed lines

@@ -43,215 +43,215 @@
 use OCP\Share\IShare;
 
 class Notifier implements INotifier {
-	public const INCOMING_USER_SHARE = 'incoming_user_share';
-	public const INCOMING_GROUP_SHARE = 'incoming_group_share';
+    public const INCOMING_USER_SHARE = 'incoming_user_share';
+    public const INCOMING_GROUP_SHARE = 'incoming_group_share';
 
-	/** @var IFactory */
-	protected $l10nFactory;
-	/** @var IManager */
-	private $shareManager;
-	/** @var IRootFolder */
-	private $rootFolder;
-	/** @var IGroupManager  */
-	protected $groupManager;
-	/** @var IUserManager  */
-	protected $userManager;
-	/** @var IURLGenerator */
-	protected $url;
+    /** @var IFactory */
+    protected $l10nFactory;
+    /** @var IManager */
+    private $shareManager;
+    /** @var IRootFolder */
+    private $rootFolder;
+    /** @var IGroupManager  */
+    protected $groupManager;
+    /** @var IUserManager  */
+    protected $userManager;
+    /** @var IURLGenerator */
+    protected $url;
 
-	public function __construct(IFactory $l10nFactory,
-								IManager $shareManager,
-								IRootFolder $rootFolder,
-								IGroupManager $groupManager,
-								IUserManager $userManager,
-								IURLGenerator $url) {
-		$this->l10nFactory = $l10nFactory;
-		$this->shareManager = $shareManager;
-		$this->rootFolder = $rootFolder;
-		$this->groupManager = $groupManager;
-		$this->userManager = $userManager;
-		$this->url = $url;
-	}
+    public function __construct(IFactory $l10nFactory,
+                                IManager $shareManager,
+                                IRootFolder $rootFolder,
+                                IGroupManager $groupManager,
+                                IUserManager $userManager,
+                                IURLGenerator $url) {
+        $this->l10nFactory = $l10nFactory;
+        $this->shareManager = $shareManager;
+        $this->rootFolder = $rootFolder;
+        $this->groupManager = $groupManager;
+        $this->userManager = $userManager;
+        $this->url = $url;
+    }
 
-	/**
-	 * Identifier of the notifier, only use [a-z0-9_]
-	 *
-	 * @return string
-	 * @since 17.0.0
-	 */
-	public function getID(): string {
-		return 'files_sharing';
-	}
+    /**
+     * Identifier of the notifier, only use [a-z0-9_]
+     *
+     * @return string
+     * @since 17.0.0
+     */
+    public function getID(): string {
+        return 'files_sharing';
+    }
 
-	/**
-	 * Human readable name describing the notifier
-	 *
-	 * @return string
-	 * @since 17.0.0
-	 */
-	public function getName(): string {
-		return $this->l10nFactory->get('files_sharing')->t('File sharing');
-	}
+    /**
+     * Human readable name describing the notifier
+     *
+     * @return string
+     * @since 17.0.0
+     */
+    public function getName(): string {
+        return $this->l10nFactory->get('files_sharing')->t('File sharing');
+    }
 
-	/**
-	 * @param INotification $notification
-	 * @param string $languageCode The code of the language that should be used to prepare the notification
-	 * @return INotification
-	 * @throws \InvalidArgumentException When the notification was not prepared by a notifier
-	 * @throws AlreadyProcessedException When the notification is not needed anymore and should be deleted
-	 * @since 9.0.0
-	 */
-	public function prepare(INotification $notification, string $languageCode): INotification {
-		if ($notification->getApp() !== 'files_sharing' ||
-			($notification->getSubject() !== 'expiresTomorrow' &&
-				$notification->getObjectType() !== 'share')) {
-			throw new \InvalidArgumentException('Unhandled app or subject');
-		}
+    /**
+     * @param INotification $notification
+     * @param string $languageCode The code of the language that should be used to prepare the notification
+     * @return INotification
+     * @throws \InvalidArgumentException When the notification was not prepared by a notifier
+     * @throws AlreadyProcessedException When the notification is not needed anymore and should be deleted
+     * @since 9.0.0
+     */
+    public function prepare(INotification $notification, string $languageCode): INotification {
+        if ($notification->getApp() !== 'files_sharing' ||
+            ($notification->getSubject() !== 'expiresTomorrow' &&
+                $notification->getObjectType() !== 'share')) {
+            throw new \InvalidArgumentException('Unhandled app or subject');
+        }
 
-		$l = $this->l10nFactory->get('files_sharing', $languageCode);
-		$attemptId = $notification->getObjectId();
+        $l = $this->l10nFactory->get('files_sharing', $languageCode);
+        $attemptId = $notification->getObjectId();
 
-		try {
-			$share = $this->shareManager->getShareById($attemptId, $notification->getUser());
-		} catch (ShareNotFound $e) {
-			throw new AlreadyProcessedException();
-		}
+        try {
+            $share = $this->shareManager->getShareById($attemptId, $notification->getUser());
+        } catch (ShareNotFound $e) {
+            throw new AlreadyProcessedException();
+        }
 
-		if ($notification->getSubject() === 'expiresTomorrow') {
-			$notification = $this->parseShareExpiration($share, $notification, $l);
-		} else {
-			$notification = $this->parseShareInvitation($share, $notification, $l);
-		}
-		return $notification;
-	}
+        if ($notification->getSubject() === 'expiresTomorrow') {
+            $notification = $this->parseShareExpiration($share, $notification, $l);
+        } else {
+            $notification = $this->parseShareInvitation($share, $notification, $l);
+        }
+        return $notification;
+    }
 
-	protected function parseShareExpiration(IShare $share, INotification $notification, IL10N $l): INotification {
-		$node = $share->getNode();
-		$userFolder = $this->rootFolder->getUserFolder($notification->getUser());
-		$path = $userFolder->getRelativePath($node->getPath());
+    protected function parseShareExpiration(IShare $share, INotification $notification, IL10N $l): INotification {
+        $node = $share->getNode();
+        $userFolder = $this->rootFolder->getUserFolder($notification->getUser());
+        $path = $userFolder->getRelativePath($node->getPath());
 
-		$notification
-			->setParsedSubject($l->t('Share will expire tomorrow'))
-			->setParsedMessage($l->t('One or more of your shares will expire tomorrow'))
-			->setRichMessage(
-				$l->t('Your share of {node} will expire tomorrow'),
-				[
-					'node' => [
-						'type' => 'file',
-						'id' => $node->getId(),
-						'name' => $node->getName(),
-						'path' => $path,
-					],
-				]
-			);
+        $notification
+            ->setParsedSubject($l->t('Share will expire tomorrow'))
+            ->setParsedMessage($l->t('One or more of your shares will expire tomorrow'))
+            ->setRichMessage(
+                $l->t('Your share of {node} will expire tomorrow'),
+                [
+                    'node' => [
+                        'type' => 'file',
+                        'id' => $node->getId(),
+                        'name' => $node->getName(),
+                        'path' => $path,
+                    ],
+                ]
+            );
 
-		return $notification;
-	}
+        return $notification;
+    }
 
-	protected function parseShareInvitation(IShare $share, INotification $notification, IL10N $l): INotification {
-		if ($share->getShareType() === IShare::TYPE_USER) {
-			if ($share->getStatus() !== IShare::STATUS_PENDING) {
-				throw new AlreadyProcessedException();
-			}
-		} elseif ($share->getShareType() === IShare::TYPE_GROUP) {
-			if ($share->getStatus() !== IShare::STATUS_PENDING) {
-				throw new AlreadyProcessedException();
-			}
-		}
+    protected function parseShareInvitation(IShare $share, INotification $notification, IL10N $l): INotification {
+        if ($share->getShareType() === IShare::TYPE_USER) {
+            if ($share->getStatus() !== IShare::STATUS_PENDING) {
+                throw new AlreadyProcessedException();
+            }
+        } elseif ($share->getShareType() === IShare::TYPE_GROUP) {
+            if ($share->getStatus() !== IShare::STATUS_PENDING) {
+                throw new AlreadyProcessedException();
+            }
+        }
 
-		switch ($notification->getSubject()) {
-			case self::INCOMING_USER_SHARE:
-				if ($share->getSharedWith() !== $notification->getUser()) {
-					throw new AlreadyProcessedException();
-				}
+        switch ($notification->getSubject()) {
+            case self::INCOMING_USER_SHARE:
+                if ($share->getSharedWith() !== $notification->getUser()) {
+                    throw new AlreadyProcessedException();
+                }
 
-				$sharer = $this->userManager->get($share->getSharedBy());
-				if (!$sharer instanceof IUser) {
-					throw new \InvalidArgumentException('Temporary failure');
-				}
+                $sharer = $this->userManager->get($share->getSharedBy());
+                if (!$sharer instanceof IUser) {
+                    throw new \InvalidArgumentException('Temporary failure');
+                }
 
-				$subject = $l->t('You received {share} as a share by {user}');
-				$subjectParameters = [
-					'share' => [
-						'type' => 'highlight',
-						'id' => $notification->getObjectId(),
-						'name' => $share->getTarget(),
-					],
-					'user' => [
-						'type' => 'user',
-						'id' => $sharer->getUID(),
-						'name' => $sharer->getDisplayName(),
-					],
-				];
-				break;
+                $subject = $l->t('You received {share} as a share by {user}');
+                $subjectParameters = [
+                    'share' => [
+                        'type' => 'highlight',
+                        'id' => $notification->getObjectId(),
+                        'name' => $share->getTarget(),
+                    ],
+                    'user' => [
+                        'type' => 'user',
+                        'id' => $sharer->getUID(),
+                        'name' => $sharer->getDisplayName(),
+                    ],
+                ];
+                break;
 
-			case self::INCOMING_GROUP_SHARE:
-				$user = $this->userManager->get($notification->getUser());
-				if (!$user instanceof IUser) {
-					throw new AlreadyProcessedException();
-				}
+            case self::INCOMING_GROUP_SHARE:
+                $user = $this->userManager->get($notification->getUser());
+                if (!$user instanceof IUser) {
+                    throw new AlreadyProcessedException();
+                }
 
-				$group = $this->groupManager->get($share->getSharedWith());
-				if ($group === null || !$group->inGroup($user)) {
-					throw new AlreadyProcessedException();
-				}
+                $group = $this->groupManager->get($share->getSharedWith());
+                if ($group === null || !$group->inGroup($user)) {
+                    throw new AlreadyProcessedException();
+                }
 
-				if ($share->getPermissions() === 0) {
-					// Already rejected
-					throw new AlreadyProcessedException();
-				}
+                if ($share->getPermissions() === 0) {
+                    // Already rejected
+                    throw new AlreadyProcessedException();
+                }
 
-				$sharer = $this->userManager->get($share->getSharedBy());
-				if (!$sharer instanceof IUser) {
-					throw new \InvalidArgumentException('Temporary failure');
-				}
+                $sharer = $this->userManager->get($share->getSharedBy());
+                if (!$sharer instanceof IUser) {
+                    throw new \InvalidArgumentException('Temporary failure');
+                }
 
-				$subject = $l->t('You received {share} to group {group} as a share by {user}');
-				$subjectParameters = [
-					'share' => [
-						'type' => 'highlight',
-						'id' => $notification->getObjectId(),
-						'name' => $share->getTarget(),
-					],
-					'group' => [
-						'type' => 'user-group',
-						'id' => $group->getGID(),
-						'name' => $group->getDisplayName(),
-					],
-					'user' => [
-						'type' => 'user',
-						'id' => $sharer->getUID(),
-						'name' => $sharer->getDisplayName(),
-					],
-				];
-				break;
+                $subject = $l->t('You received {share} to group {group} as a share by {user}');
+                $subjectParameters = [
+                    'share' => [
+                        'type' => 'highlight',
+                        'id' => $notification->getObjectId(),
+                        'name' => $share->getTarget(),
+                    ],
+                    'group' => [
+                        'type' => 'user-group',
+                        'id' => $group->getGID(),
+                        'name' => $group->getDisplayName(),
+                    ],
+                    'user' => [
+                        'type' => 'user',
+                        'id' => $sharer->getUID(),
+                        'name' => $sharer->getDisplayName(),
+                    ],
+                ];
+                break;
 
-			default:
-				throw new \InvalidArgumentException('Invalid subject');
-		}
+            default:
+                throw new \InvalidArgumentException('Invalid subject');
+        }
 
-		$placeholders = $replacements = [];
-		foreach ($subjectParameters as $placeholder => $parameter) {
-			$placeholders[] = '{' . $placeholder . '}';
-			$replacements[] = $parameter['name'];
-		}
+        $placeholders = $replacements = [];
+        foreach ($subjectParameters as $placeholder => $parameter) {
+            $placeholders[] = '{' . $placeholder . '}';
+            $replacements[] = $parameter['name'];
+        }
 
-		$notification->setParsedSubject(str_replace($placeholders, $replacements, $subject))
-			->setRichSubject($subject, $subjectParameters)
-			->setIcon($this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/share.svg')));
+        $notification->setParsedSubject(str_replace($placeholders, $replacements, $subject))
+            ->setRichSubject($subject, $subjectParameters)
+            ->setIcon($this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/share.svg')));
 
-		$acceptAction = $notification->createAction();
-		$acceptAction->setParsedLabel($l->t('Accept'))
-			->setLink($this->url->linkToOCSRouteAbsolute('files_sharing.ShareAPI.acceptShare', ['id' => $share->getId()]), 'POST')
-			->setPrimary(true);
-		$notification->addParsedAction($acceptAction);
+        $acceptAction = $notification->createAction();
+        $acceptAction->setParsedLabel($l->t('Accept'))
+            ->setLink($this->url->linkToOCSRouteAbsolute('files_sharing.ShareAPI.acceptShare', ['id' => $share->getId()]), 'POST')
+            ->setPrimary(true);
+        $notification->addParsedAction($acceptAction);
 
-		$rejectAction = $notification->createAction();
-		$rejectAction->setParsedLabel($l->t('Reject'))
-			->setLink($this->url->linkToOCSRouteAbsolute('files_sharing.ShareAPI.deleteShare', ['id' => $share->getId()]), 'DELETE')
-			->setPrimary(false);
-		$notification->addParsedAction($rejectAction);
+        $rejectAction = $notification->createAction();
+        $rejectAction->setParsedLabel($l->t('Reject'))
+            ->setLink($this->url->linkToOCSRouteAbsolute('files_sharing.ShareAPI.deleteShare', ['id' => $share->getId()]), 'DELETE')
+            ->setPrimary(false);
+        $notification->addParsedAction($rejectAction);
 
-		return $notification;
-	}
+        return $notification;
+    }
 }

apps/files_sharing/lib/Migration/SetAcceptedStatus.php

Indentation +35 added lines, -35 removed lines

@@ -33,47 +33,47 @@
 
 class SetAcceptedStatus implements IRepairStep {
 
-	/** @var IDBConnection */
-	private $connection;
+    /** @var IDBConnection */
+    private $connection;
 
-	/** @var  IConfig */
-	private $config;
+    /** @var  IConfig */
+    private $config;
 
 
-	public function __construct(IDBConnection $connection, IConfig $config) {
-		$this->connection = $connection;
-		$this->config = $config;
-	}
+    public function __construct(IDBConnection $connection, IConfig $config) {
+        $this->connection = $connection;
+        $this->config = $config;
+    }
 
-	/**
-	 * Returns the step's name
-	 *
-	 * @return string
-	 * @since 9.1.0
-	 */
-	public function getName(): string {
-		return 'Set existing shares as accepted';
-	}
+    /**
+     * Returns the step's name
+     *
+     * @return string
+     * @since 9.1.0
+     */
+    public function getName(): string {
+        return 'Set existing shares as accepted';
+    }
 
-	/**
-	 * @param IOutput $output
-	 */
-	public function run(IOutput $output): void {
-		if (!$this->shouldRun()) {
-			return;
-		}
+    /**
+     * @param IOutput $output
+     */
+    public function run(IOutput $output): void {
+        if (!$this->shouldRun()) {
+            return;
+        }
 
-		$query = $this->connection->getQueryBuilder();
-		$query
-			->update('share')
-			->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))
-			->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));
-		$query->execute();
-	}
+        $query = $this->connection->getQueryBuilder();
+        $query
+            ->update('share')
+            ->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))
+            ->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));
+        $query->execute();
+    }
 
-	protected function shouldRun() {
-		$appVersion = $this->config->getAppValue('files_sharing', 'installed_version', '0.0.0');
-		return version_compare($appVersion, '1.10.1', '<');
-	}
+    protected function shouldRun() {
+        $appVersion = $this->config->getAppValue('files_sharing', 'installed_version', '0.0.0');
+        return version_compare($appVersion, '1.10.1', '<');
+    }
 
 }