nextcloud / server

lib/public/Share/Events/VerifyMountPointEvent.php

Indentation +41 added lines, -41 removed lines

@@ -33,51 +33,51 @@
  */
 class VerifyMountPointEvent extends Event {
 
-	/** @var IShare */
-	private $share;
-	/** @var View */
-	private $view;
-	/** @var string */
-	private $parent;
+    /** @var IShare */
+    private $share;
+    /** @var View */
+    private $view;
+    /** @var string */
+    private $parent;
 
-	/**
-	 * @since 19.0.0
-	 */
-	public function __construct(IShare $share,
-								View $view,
-								string $parent) {
-		parent::__construct();
+    /**
+     * @since 19.0.0
+     */
+    public function __construct(IShare $share,
+                                View $view,
+                                string $parent) {
+        parent::__construct();
 
-		$this->share = $share;
-		$this->view = $view;
-		$this->parent = $parent;
-	}
+        $this->share = $share;
+        $this->view = $view;
+        $this->parent = $parent;
+    }
 
-	/**
-	 * @since 19.0.0
-	 */
-	public function getShare(): IShare {
-		return $this->share;
-	}
+    /**
+     * @since 19.0.0
+     */
+    public function getShare(): IShare {
+        return $this->share;
+    }
 
-	/**
-	 * @since 19.0.0
-	 */
-	public function getView(): View {
-		return $this->view;
-	}
+    /**
+     * @since 19.0.0
+     */
+    public function getView(): View {
+        return $this->view;
+    }
 
-	/**
-	 * @since 19.0.0
-	 */
-	public function getParent(): string {
-		return $this->parent;
-	}
+    /**
+     * @since 19.0.0
+     */
+    public function getParent(): string {
+        return $this->parent;
+    }
 
-	/**
-	 * @since 19.0.0
-	 */
-	public function setParent(string $parent): void {
-		$this->parent = $parent;
-	}
+    /**
+     * @since 19.0.0
+     */
+    public function setParent(string $parent): void {
+        $this->parent = $parent;
+    }
 }

apps/files_versions/lib/Versions/IVersionBackend.php

Indentation +55 added lines, -55 removed lines

@@ -37,64 +37,64 @@
  * @since 15.0.0
  */
 interface IVersionBackend {
-	/**
-	 * Whether or not this version backend should be used for a storage
-	 *
-	 * If false is returned then the next applicable backend will be used
-	 *
-	 * @param IStorage $storage
-	 * @return bool
-	 * @since 17.0.0
-	 */
-	public function useBackendForStorage(IStorage $storage): bool;
+    /**
+     * Whether or not this version backend should be used for a storage
+     *
+     * If false is returned then the next applicable backend will be used
+     *
+     * @param IStorage $storage
+     * @return bool
+     * @since 17.0.0
+     */
+    public function useBackendForStorage(IStorage $storage): bool;
 
-	/**
-	 * Get all versions for a file
-	 *
-	 * @param IUser $user
-	 * @param FileInfo $file
-	 * @return IVersion[]
-	 * @since 15.0.0
-	 */
-	public function getVersionsForFile(IUser $user, FileInfo $file): array;
+    /**
+     * Get all versions for a file
+     *
+     * @param IUser $user
+     * @param FileInfo $file
+     * @return IVersion[]
+     * @since 15.0.0
+     */
+    public function getVersionsForFile(IUser $user, FileInfo $file): array;
 
-	/**
-	 * Create a new version for a file
-	 *
-	 * @param IUser $user
-	 * @param FileInfo $file
-	 * @since 15.0.0
-	 */
-	public function createVersion(IUser $user, FileInfo $file);
+    /**
+     * Create a new version for a file
+     *
+     * @param IUser $user
+     * @param FileInfo $file
+     * @since 15.0.0
+     */
+    public function createVersion(IUser $user, FileInfo $file);
 
-	/**
-	 * Restore this version
-	 *
-	 * @param IVersion $version
-	 * @since 15.0.0
-	 */
-	public function rollback(IVersion $version);
+    /**
+     * Restore this version
+     *
+     * @param IVersion $version
+     * @since 15.0.0
+     */
+    public function rollback(IVersion $version);
 
-	/**
-	 * Open the file for reading
-	 *
-	 * @param IVersion $version
-	 * @return resource
-	 * @throws NotFoundException
-	 * @since 15.0.0
-	 */
-	public function read(IVersion $version);
+    /**
+     * Open the file for reading
+     *
+     * @param IVersion $version
+     * @return resource
+     * @throws NotFoundException
+     * @since 15.0.0
+     */
+    public function read(IVersion $version);
 
-	/**
-	 * Get the preview for a specific version of a file
-	 *
-	 * @param IUser $user
-	 * @param FileInfo $sourceFile
-	 * @param int|string $revision
-	 *
-	 * @return File
-	 *
-	 * @since 15.0.0
-	 */
-	public function getVersionFile(IUser $user, FileInfo $sourceFile, $revision): File;
+    /**
+     * Get the preview for a specific version of a file
+     *
+     * @param IUser $user
+     * @param FileInfo $sourceFile
+     * @param int|string $revision
+     *
+     * @return File
+     *
+     * @since 15.0.0
+     */
+    public function getVersionFile(IUser $user, FileInfo $sourceFile, $revision): File;
 }

lib/private/Authentication/TwoFactorAuth/EnforcementState.php

Indentation +45 added lines, -45 removed lines

@@ -30,57 +30,57 @@
 
 class EnforcementState implements JsonSerializable {
 
-	/** @var bool */
-	private $enforced;
+    /** @var bool */
+    private $enforced;
 
-	/** @var array */
-	private $enforcedGroups;
+    /** @var array */
+    private $enforcedGroups;
 
-	/** @var array */
-	private $excludedGroups;
+    /** @var array */
+    private $excludedGroups;
 
-	/**
-	 * EnforcementState constructor.
-	 *
-	 * @param bool $enforced
-	 * @param string[] $enforcedGroups
-	 * @param string[] $excludedGroups
-	 */
-	public function __construct(bool $enforced,
-								array $enforcedGroups = [],
-								array $excludedGroups = []) {
-		$this->enforced = $enforced;
-		$this->enforcedGroups = $enforcedGroups;
-		$this->excludedGroups = $excludedGroups;
-	}
+    /**
+     * EnforcementState constructor.
+     *
+     * @param bool $enforced
+     * @param string[] $enforcedGroups
+     * @param string[] $excludedGroups
+     */
+    public function __construct(bool $enforced,
+                                array $enforcedGroups = [],
+                                array $excludedGroups = []) {
+        $this->enforced = $enforced;
+        $this->enforcedGroups = $enforcedGroups;
+        $this->excludedGroups = $excludedGroups;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function isEnforced(): bool {
-		return $this->enforced;
-	}
+    /**
+     * @return bool
+     */
+    public function isEnforced(): bool {
+        return $this->enforced;
+    }
 
-	/**
-	 * @return string[]
-	 */
-	public function getEnforcedGroups(): array {
-		return $this->enforcedGroups;
-	}
+    /**
+     * @return string[]
+     */
+    public function getEnforcedGroups(): array {
+        return $this->enforcedGroups;
+    }
 
-	/**
-	 * @return string[]
-	 */
-	public function getExcludedGroups(): array {
-		return $this->excludedGroups;
-	}
+    /**
+     * @return string[]
+     */
+    public function getExcludedGroups(): array {
+        return $this->excludedGroups;
+    }
 
-	public function jsonSerialize(): array {
-		return [
-			'enforced' => $this->enforced,
-			'enforcedGroups' => $this->enforcedGroups,
-			'excludedGroups' => $this->excludedGroups,
-		];
-	}
+    public function jsonSerialize(): array {
+        return [
+            'enforced' => $this->enforced,
+            'enforcedGroups' => $this->enforcedGroups,
+            'excludedGroups' => $this->excludedGroups,
+        ];
+    }
 
 }

apps/oauth2/lib/Controller/LoginRedirectorController.php

Indentation +63 added lines, -63 removed lines

@@ -39,72 +39,72 @@
 use OCP\IURLGenerator;
 
 class LoginRedirectorController extends Controller {
-	/** @var IURLGenerator */
-	private $urlGenerator;
-	/** @var ClientMapper */
-	private $clientMapper;
-	/** @var ISession */
-	private $session;
-	/** @var IL10N */
-	private $l;
+    /** @var IURLGenerator */
+    private $urlGenerator;
+    /** @var ClientMapper */
+    private $clientMapper;
+    /** @var ISession */
+    private $session;
+    /** @var IL10N */
+    private $l;
 
-	/**
-	 * @param string $appName
-	 * @param IRequest $request
-	 * @param IURLGenerator $urlGenerator
-	 * @param ClientMapper $clientMapper
-	 * @param ISession $session
-	 * @param IL10N $l
-	 */
-	public function __construct(string $appName,
-								IRequest $request,
-								IURLGenerator $urlGenerator,
-								ClientMapper $clientMapper,
-								ISession $session,
-								IL10N $l) {
-		parent::__construct($appName, $request);
-		$this->urlGenerator = $urlGenerator;
-		$this->clientMapper = $clientMapper;
-		$this->session = $session;
-		$this->l = $l;
-	}
+    /**
+     * @param string $appName
+     * @param IRequest $request
+     * @param IURLGenerator $urlGenerator
+     * @param ClientMapper $clientMapper
+     * @param ISession $session
+     * @param IL10N $l
+     */
+    public function __construct(string $appName,
+                                IRequest $request,
+                                IURLGenerator $urlGenerator,
+                                ClientMapper $clientMapper,
+                                ISession $session,
+                                IL10N $l) {
+        parent::__construct($appName, $request);
+        $this->urlGenerator = $urlGenerator;
+        $this->clientMapper = $clientMapper;
+        $this->session = $session;
+        $this->l = $l;
+    }
 
-	/**
-	 * @PublicPage
-	 * @NoCSRFRequired
-	 * @UseSession
-	 *
-	 * @param string $client_id
-	 * @param string $state
-	 * @param string $response_type
-	 * @return Response
-	 */
-	public function authorize($client_id,
-							  $state,
-							  $response_type): Response {
-		try {
-			$client = $this->clientMapper->getByIdentifier($client_id);
-		} catch (ClientNotFoundException $e) {
-			$params = [
-				'content' => $this->l->t('Your client is not authorized to connect. Please inform the administrator of your client.'),
-			];
-			return new TemplateResponse('core', '404', $params, 'guest');
-		}
+    /**
+     * @PublicPage
+     * @NoCSRFRequired
+     * @UseSession
+     *
+     * @param string $client_id
+     * @param string $state
+     * @param string $response_type
+     * @return Response
+     */
+    public function authorize($client_id,
+                                $state,
+                                $response_type): Response {
+        try {
+            $client = $this->clientMapper->getByIdentifier($client_id);
+        } catch (ClientNotFoundException $e) {
+            $params = [
+                'content' => $this->l->t('Your client is not authorized to connect. Please inform the administrator of your client.'),
+            ];
+            return new TemplateResponse('core', '404', $params, 'guest');
+        }
 
-		if ($response_type !== 'code') {
-			//Fail
-			$url = $client->getRedirectUri() . '?error=unsupported_response_type&state=' . $state;
-			return new RedirectResponse($url);
-		}
+        if ($response_type !== 'code') {
+            //Fail
+            $url = $client->getRedirectUri() . '?error=unsupported_response_type&state=' . $state;
+            return new RedirectResponse($url);
+        }
 
-		$this->session->set('oauth.state', $state);
+        $this->session->set('oauth.state', $state);
 
-		$targetUrl = $this->urlGenerator->linkToRouteAbsolute(
-			'core.ClientFlowLogin.showAuthPickerPage',
-			[
-				'clientIdentifier' => $client->getClientIdentifier(),
-			]
-		);
-		return new RedirectResponse($targetUrl);
-	}
+        $targetUrl = $this->urlGenerator->linkToRouteAbsolute(
+            'core.ClientFlowLogin.showAuthPickerPage',
+            [
+                'clientIdentifier' => $client->getClientIdentifier(),
+            ]
+        );
+        return new RedirectResponse($targetUrl);
+    }
 }

apps/contactsinteraction/lib/Migration/Version010000Date20200304152605.php

Indentation +54 added lines, -54 removed lines

@@ -33,61 +33,61 @@
 
 class Version010000Date20200304152605 extends SimpleMigrationStep {
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 *
-	 * @return ISchemaWrapper
-	 */
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ISchemaWrapper {
-		/** @var ISchemaWrapper $schema */
-		$schema = $schemaClosure();
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     *
+     * @return ISchemaWrapper
+     */
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ISchemaWrapper {
+        /** @var ISchemaWrapper $schema */
+        $schema = $schemaClosure();
 
-		$table = $schema->createTable(RecentContactMapper::TABLE_NAME);
-		$table->addColumn('id', 'integer', [
-			'autoincrement' => true,
-			'notnull' => true,
-			'length' => 4,
-		]);
-		$table->addColumn('actor_uid', 'string', [
-			'notnull' => true,
-			'length' => 64,
-		]);
-		$table->addColumn('uid', 'string', [
-			'notnull' => false,
-			'length' => 64,
-		]);
-		$table->addColumn('email', 'string', [
-			'notnull' => false,
-			'length' => 255,
-		]);
-		$table->addColumn('federated_cloud_id', 'string', [
-			'notnull' => false,
-			'length' => 255,
-		]);
-		$table->addColumn('card', 'blob', [
-			'notnull' => true,
-		]);
-		$table->addColumn('last_contact', 'integer', [
-			'notnull' => true,
-			'length' => 4,
-		]);
-		$table->setPrimaryKey(['id']);
-		// To find all recent entries
-		$table->addIndex(['actor_uid'], RecentContactMapper::TABLE_NAME . '_actor_uid');
-		// To find a specific entry
-		$table->addIndex(['id', 'actor_uid'], RecentContactMapper::TABLE_NAME . '_id_uid');
-		// To find all recent entries with a given UID
-		$table->addIndex(['uid'], RecentContactMapper::TABLE_NAME . '_uid');
-		// To find all recent entries with a given email address
-		$table->addIndex(['email'], RecentContactMapper::TABLE_NAME . '_email');
-		// To find all recent entries with a give federated cloud id
-		$table->addIndex(['federated_cloud_id'], RecentContactMapper::TABLE_NAME . '_fed_id');
-		// For the cleanup
-		$table->addIndex(['last_contact'], RecentContactMapper::TABLE_NAME . '_last_contact');
+        $table = $schema->createTable(RecentContactMapper::TABLE_NAME);
+        $table->addColumn('id', 'integer', [
+            'autoincrement' => true,
+            'notnull' => true,
+            'length' => 4,
+        ]);
+        $table->addColumn('actor_uid', 'string', [
+            'notnull' => true,
+            'length' => 64,
+        ]);
+        $table->addColumn('uid', 'string', [
+            'notnull' => false,
+            'length' => 64,
+        ]);
+        $table->addColumn('email', 'string', [
+            'notnull' => false,
+            'length' => 255,
+        ]);
+        $table->addColumn('federated_cloud_id', 'string', [
+            'notnull' => false,
+            'length' => 255,
+        ]);
+        $table->addColumn('card', 'blob', [
+            'notnull' => true,
+        ]);
+        $table->addColumn('last_contact', 'integer', [
+            'notnull' => true,
+            'length' => 4,
+        ]);
+        $table->setPrimaryKey(['id']);
+        // To find all recent entries
+        $table->addIndex(['actor_uid'], RecentContactMapper::TABLE_NAME . '_actor_uid');
+        // To find a specific entry
+        $table->addIndex(['id', 'actor_uid'], RecentContactMapper::TABLE_NAME . '_id_uid');
+        // To find all recent entries with a given UID
+        $table->addIndex(['uid'], RecentContactMapper::TABLE_NAME . '_uid');
+        // To find all recent entries with a given email address
+        $table->addIndex(['email'], RecentContactMapper::TABLE_NAME . '_email');
+        // To find all recent entries with a give federated cloud id
+        $table->addIndex(['federated_cloud_id'], RecentContactMapper::TABLE_NAME . '_fed_id');
+        // For the cleanup
+        $table->addIndex(['last_contact'], RecentContactMapper::TABLE_NAME . '_last_contact');
 
-		return $schema;
-	}
+        return $schema;
+    }
 
 }

apps/contactsinteraction/lib/AddressBookProvider.php

Indentation +37 added lines, -37 removed lines

@@ -33,49 +33,49 @@
 
 class AddressBookProvider implements IAddressBookProvider {
 
-	/** @var RecentContactMapper */
-	private $mapper;
+    /** @var RecentContactMapper */
+    private $mapper;
 
-	/** @var IL10N */
-	private $l10n;
+    /** @var IL10N */
+    private $l10n;
 
-	public function __construct(RecentContactMapper $mapper, IL10N $l10n) {
-		$this->mapper = $mapper;
-		$this->l10n = $l10n;
-	}
+    public function __construct(RecentContactMapper $mapper, IL10N $l10n) {
+        $this->mapper = $mapper;
+        $this->l10n = $l10n;
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function getAppId(): string {
-		return Application::APP_ID;
-	}
+    /**
+     * @inheritDoc
+     */
+    public function getAppId(): string {
+        return Application::APP_ID;
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function fetchAllForAddressBookHome(string $principalUri): array {
-		return [
-			new AddressBook($this->mapper, $this->l10n, $principalUri)
-		];
-	}
+    /**
+     * @inheritDoc
+     */
+    public function fetchAllForAddressBookHome(string $principalUri): array {
+        return [
+            new AddressBook($this->mapper, $this->l10n, $principalUri)
+        ];
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function hasAddressBookInAddressBookHome(string $principalUri, string $uri): bool {
-		return $uri === AddressBook::URI;
-	}
+    /**
+     * @inheritDoc
+     */
+    public function hasAddressBookInAddressBookHome(string $principalUri, string $uri): bool {
+        return $uri === AddressBook::URI;
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function getAddressBookInAddressBookHome(string $principalUri, string $uri): ?ExternalAddressBook {
-		if ($uri === AddressBook::URI) {
-			return new AddressBook($this->mapper, $this->l10n, $principalUri);
-		}
+    /**
+     * @inheritDoc
+     */
+    public function getAddressBookInAddressBookHome(string $principalUri, string $uri): ?ExternalAddressBook {
+        if ($uri === AddressBook::URI) {
+            return new AddressBook($this->mapper, $this->l10n, $principalUri);
+        }
 
-		return null;
-	}
+        return null;
+    }
 
 }

lib/public/Contacts/Events/ContactInteractedWithEvent.php

Indentation +92 added lines, -92 removed lines

@@ -40,97 +40,97 @@
  */
 class ContactInteractedWithEvent extends Event {
 
-	/** @var IUser */
-	private $actor;
-
-	/** @var string|null */
-	private $uid;
-
-	/** @var string|null */
-	private $email;
-
-	/** @var string|null */
-	private $federatedCloudId;
-
-	/**
-	 * @param IUser $actor the user who started the interaction
-	 *
-	 * @since 19.0.0
-	 */
-	public function __construct(IUser $actor) {
-		parent::__construct();
-		$this->actor = $actor;
-	}
-
-	/**
-	 * @return IUser
-	 * @since 19.0.0
-	 */
-	public function getActor(): IUser {
-		return $this->actor;
-	}
-
-	/**
-	 * @return string|null
-	 * @since 19.0.0
-	 */
-	public function getUid(): ?string {
-		return $this->uid;
-	}
-
-	/**
-	 * Set the uid of the person interacted with, if known
-	 *
-	 * @param string $uid
-	 *
-	 * @return self
-	 * @since 19.0.0
-	 */
-	public function setUid(string $uid): self {
-		$this->uid = $uid;
-		return $this;
-	}
-
-	/**
-	 * @return string|null
-	 * @since 19.0.0
-	 */
-	public function getEmail(): ?string {
-		return $this->email;
-	}
-
-	/**
-	 * Set the email of the person interacted with, if known
-	 *
-	 * @param string $email
-	 *
-	 * @return self
-	 * @since 19.0.0
-	 */
-	public function setEmail(string $email): self {
-		$this->email = $email;
-		return $this;
-	}
-
-	/**
-	 * @return string|null
-	 * @since 19.0.0
-	 */
-	public function getFederatedCloudId(): ?string {
-		return $this->federatedCloudId;
-	}
-
-	/**
-	 * Set the federated cloud of the person interacted with, if known
-	 *
-	 * @param string $federatedCloudId
-	 *
-	 * @return self
-	 * @since 19.0.0
-	 */
-	public function setFederatedCloudId(string $federatedCloudId): self {
-		$this->federatedCloudId = $federatedCloudId;
-		return $this;
-	}
+    /** @var IUser */
+    private $actor;
+
+    /** @var string|null */
+    private $uid;
+
+    /** @var string|null */
+    private $email;
+
+    /** @var string|null */
+    private $federatedCloudId;
+
+    /**
+     * @param IUser $actor the user who started the interaction
+     *
+     * @since 19.0.0
+     */
+    public function __construct(IUser $actor) {
+        parent::__construct();
+        $this->actor = $actor;
+    }
+
+    /**
+     * @return IUser
+     * @since 19.0.0
+     */
+    public function getActor(): IUser {
+        return $this->actor;
+    }
+
+    /**
+     * @return string|null
+     * @since 19.0.0
+     */
+    public function getUid(): ?string {
+        return $this->uid;
+    }
+
+    /**
+     * Set the uid of the person interacted with, if known
+     *
+     * @param string $uid
+     *
+     * @return self
+     * @since 19.0.0
+     */
+    public function setUid(string $uid): self {
+        $this->uid = $uid;
+        return $this;
+    }
+
+    /**
+     * @return string|null
+     * @since 19.0.0
+     */
+    public function getEmail(): ?string {
+        return $this->email;
+    }
+
+    /**
+     * Set the email of the person interacted with, if known
+     *
+     * @param string $email
+     *
+     * @return self
+     * @since 19.0.0
+     */
+    public function setEmail(string $email): self {
+        $this->email = $email;
+        return $this;
+    }
+
+    /**
+     * @return string|null
+     * @since 19.0.0
+     */
+    public function getFederatedCloudId(): ?string {
+        return $this->federatedCloudId;
+    }
+
+    /**
+     * Set the federated cloud of the person interacted with, if known
+     *
+     * @param string $federatedCloudId
+     *
+     * @return self
+     * @since 19.0.0
+     */
+    public function setFederatedCloudId(string $federatedCloudId): self {
+        $this->federatedCloudId = $federatedCloudId;
+        return $this;
+    }
 
 }

apps/files/lib/Service/TagService.php

Indentation +97 added lines, -97 removed lines

@@ -38,110 +38,110 @@
  */
 class TagService {
 
-	/** @var IUserSession */
-	private $userSession;
-	/** @var IManager */
-	private $activityManager;
-	/** @var ITags */
-	private $tagger;
-	/** @var Folder */
-	private $homeFolder;
-	/** @var EventDispatcherInterface */
-	private $dispatcher;
+    /** @var IUserSession */
+    private $userSession;
+    /** @var IManager */
+    private $activityManager;
+    /** @var ITags */
+    private $tagger;
+    /** @var Folder */
+    private $homeFolder;
+    /** @var EventDispatcherInterface */
+    private $dispatcher;
 
-	/**
-	 * @param IUserSession $userSession
-	 * @param IManager $activityManager
-	 * @param ITags $tagger
-	 * @param Folder $homeFolder
-	 * @param EventDispatcherInterface $dispatcher
-	 */
-	public function __construct(
-		IUserSession $userSession,
-		IManager $activityManager,
-		ITags $tagger,
-		Folder $homeFolder,
-		EventDispatcherInterface $dispatcher
-	) {
-		$this->userSession = $userSession;
-		$this->activityManager = $activityManager;
-		$this->tagger = $tagger;
-		$this->homeFolder = $homeFolder;
-		$this->dispatcher = $dispatcher;
-	}
+    /**
+     * @param IUserSession $userSession
+     * @param IManager $activityManager
+     * @param ITags $tagger
+     * @param Folder $homeFolder
+     * @param EventDispatcherInterface $dispatcher
+     */
+    public function __construct(
+        IUserSession $userSession,
+        IManager $activityManager,
+        ITags $tagger,
+        Folder $homeFolder,
+        EventDispatcherInterface $dispatcher
+    ) {
+        $this->userSession = $userSession;
+        $this->activityManager = $activityManager;
+        $this->tagger = $tagger;
+        $this->homeFolder = $homeFolder;
+        $this->dispatcher = $dispatcher;
+    }
 
-	/**
-	 * Updates the tags of the specified file path.
-	 * The passed tags are absolute, which means they will
-	 * replace the actual tag selection.
-	 *
-	 * @param string $path path
-	 * @param array  $tags array of tags
-	 * @return array list of tags
-	 * @throws \OCP\Files\NotFoundException if the file does not exist
-	 */
-	public function updateFileTags($path, $tags) {
-		$fileId = $this->homeFolder->get($path)->getId();
+    /**
+     * Updates the tags of the specified file path.
+     * The passed tags are absolute, which means they will
+     * replace the actual tag selection.
+     *
+     * @param string $path path
+     * @param array  $tags array of tags
+     * @return array list of tags
+     * @throws \OCP\Files\NotFoundException if the file does not exist
+     */
+    public function updateFileTags($path, $tags) {
+        $fileId = $this->homeFolder->get($path)->getId();
 
-		$currentTags = $this->tagger->getTagsForObjects([$fileId]);
+        $currentTags = $this->tagger->getTagsForObjects([$fileId]);
 
-		if (!empty($currentTags)) {
-			$currentTags = current($currentTags);
-		}
+        if (!empty($currentTags)) {
+            $currentTags = current($currentTags);
+        }
 
-		$newTags = array_diff($tags, $currentTags);
-		foreach ($newTags as $tag) {
-			if ($tag === ITags::TAG_FAVORITE) {
-				$this->addActivity(true, $fileId, $path);
-			}
-			$this->tagger->tagAs($fileId, $tag);
-		}
-		$deletedTags = array_diff($currentTags, $tags);
-		foreach ($deletedTags as $tag) {
-			if ($tag === ITags::TAG_FAVORITE) {
-				$this->addActivity(false, $fileId, $path);
-			}
-			$this->tagger->unTag($fileId, $tag);
-		}
+        $newTags = array_diff($tags, $currentTags);
+        foreach ($newTags as $tag) {
+            if ($tag === ITags::TAG_FAVORITE) {
+                $this->addActivity(true, $fileId, $path);
+            }
+            $this->tagger->tagAs($fileId, $tag);
+        }
+        $deletedTags = array_diff($currentTags, $tags);
+        foreach ($deletedTags as $tag) {
+            if ($tag === ITags::TAG_FAVORITE) {
+                $this->addActivity(false, $fileId, $path);
+            }
+            $this->tagger->unTag($fileId, $tag);
+        }
 
-		// TODO: re-read from tagger to make sure the
-		// list is up to date, in case of concurrent changes ?
-		return $tags;
-	}
+        // TODO: re-read from tagger to make sure the
+        // list is up to date, in case of concurrent changes ?
+        return $tags;
+    }
 
-	/**
-	 * @param bool $addToFavorite
-	 * @param int $fileId
-	 * @param string $path
-	 */
-	protected function addActivity($addToFavorite, $fileId, $path) {
-		$user = $this->userSession->getUser();
-		if (!$user instanceof IUser) {
-			return;
-		}
+    /**
+     * @param bool $addToFavorite
+     * @param int $fileId
+     * @param string $path
+     */
+    protected function addActivity($addToFavorite, $fileId, $path) {
+        $user = $this->userSession->getUser();
+        if (!$user instanceof IUser) {
+            return;
+        }
 
-		$eventName = $addToFavorite ? 'addFavorite' : 'removeFavorite';
-		$this->dispatcher->dispatch(self::class . '::' . $eventName, new GenericEvent(null, [
-			'userId' => $user->getUID(),
-			'fileId' => $fileId,
-			'path' => $path,
-		]));
+        $eventName = $addToFavorite ? 'addFavorite' : 'removeFavorite';
+        $this->dispatcher->dispatch(self::class . '::' . $eventName, new GenericEvent(null, [
+            'userId' => $user->getUID(),
+            'fileId' => $fileId,
+            'path' => $path,
+        ]));
 
-		$event = $this->activityManager->generateEvent();
-		try {
-			$event->setApp('files')
-				->setObject('files', $fileId, $path)
-				->setType('favorite')
-				->setAuthor($user->getUID())
-				->setAffectedUser($user->getUID())
-				->setTimestamp(time())
-				->setSubject(
-					$addToFavorite ? FavoriteProvider::SUBJECT_ADDED : FavoriteProvider::SUBJECT_REMOVED,
-					['id' => $fileId, 'path' => $path]
-				);
-			$this->activityManager->publish($event);
-		} catch (\InvalidArgumentException $e) {
-		} catch (\BadMethodCallException $e) {
-		}
-	}
+        $event = $this->activityManager->generateEvent();
+        try {
+            $event->setApp('files')
+                ->setObject('files', $fileId, $path)
+                ->setType('favorite')
+                ->setAuthor($user->getUID())
+                ->setAffectedUser($user->getUID())
+                ->setTimestamp(time())
+                ->setSubject(
+                    $addToFavorite ? FavoriteProvider::SUBJECT_ADDED : FavoriteProvider::SUBJECT_REMOVED,
+                    ['id' => $fileId, 'path' => $path]
+                );
+            $this->activityManager->publish($event);
+        } catch (\InvalidArgumentException $e) {
+        } catch (\BadMethodCallException $e) {
+        }
+    }
 }

apps/files_external/ajax/applicable.php

Indentation +5 added lines, -5 removed lines

@@ -32,23 +32,23 @@
 $limit = null;
 $offset = null;
 if (isset($_GET['pattern'])) {
-	$pattern = (string)$_GET['pattern'];
+    $pattern = (string)$_GET['pattern'];
 }
 if (isset($_GET['limit'])) {
-	$limit = (int)$_GET['limit'];
+    $limit = (int)$_GET['limit'];
 }
 if (isset($_GET['offset'])) {
-	$offset = (int)$_GET['offset'];
+    $offset = (int)$_GET['offset'];
 }
 
 $groups = [];
 foreach (\OC::$server->getGroupManager()->search($pattern, $limit, $offset) as $group) {
-	$groups[$group->getGID()] = $group->getDisplayName();
+    $groups[$group->getGID()] = $group->getDisplayName();
 }
 
 $users = [];
 foreach (\OC::$server->getUserManager()->searchDisplayName($pattern, $limit, $offset) as $user) {
-	$users[$user->getUID()] = $user->getDisplayName();
+    $users[$user->getUID()] = $user->getDisplayName();
 }
 
 $results = ['groups' => $groups, 'users' => $users];