nextcloud / server

apps/settings/lib/Sections/Admin/Server.php

Indentation +20 added lines, -20 removed lines

@@ -30,30 +30,30 @@
 
 class Server implements IIconSection {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
-	public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
-		$this->l = $l;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
+        $this->l = $l;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	public function getIcon(): string {
-		return $this->urlGenerator->imagePath('core', 'actions/settings-dark.svg');
-	}
+    public function getIcon(): string {
+        return $this->urlGenerator->imagePath('core', 'actions/settings-dark.svg');
+    }
 
-	public function getID(): string {
-		return 'server';
-	}
+    public function getID(): string {
+        return 'server';
+    }
 
-	public function getName(): string {
-		return $this->l->t('Basic settings');
-	}
+    public function getName(): string {
+        return $this->l->t('Basic settings');
+    }
 
-	public function getPriority(): int {
-		return 1;
-	}
+    public function getPriority(): int {
+        return 1;
+    }
 }

apps/settings/lib/Sections/Admin/Overview.php

Indentation +20 added lines, -20 removed lines

@@ -30,30 +30,30 @@
 
 class Overview implements IIconSection {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
-	public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
-		$this->l = $l;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
+        $this->l = $l;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	public function getIcon(): string {
-		return $this->urlGenerator->imagePath('settings', 'admin.svg');
-	}
+    public function getIcon(): string {
+        return $this->urlGenerator->imagePath('settings', 'admin.svg');
+    }
 
-	public function getID(): string {
-		return 'overview';
-	}
+    public function getID(): string {
+        return 'overview';
+    }
 
-	public function getName(): string {
-		return $this->l->t('Overview');
-	}
+    public function getName(): string {
+        return $this->l->t('Overview');
+    }
 
-	public function getPriority(): int {
-		return 0;
-	}
+    public function getPriority(): int {
+        return 0;
+    }
 }

apps/settings/lib/Sections/Personal/SyncClients.php

Indentation +20 added lines, -20 removed lines

@@ -30,30 +30,30 @@
 
 class SyncClients implements IIconSection {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
-	public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
-		$this->l = $l;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
+        $this->l = $l;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	public function getIcon() {
-		return $this->urlGenerator->imagePath('core', 'clients/phone.svg');
-	}
+    public function getIcon() {
+        return $this->urlGenerator->imagePath('core', 'clients/phone.svg');
+    }
 
-	public function getID(): string {
-		return 'sync-clients';
-	}
+    public function getID(): string {
+        return 'sync-clients';
+    }
 
-	public function getName(): string {
-		return $this->l->t('Mobile & desktop');
-	}
+    public function getName(): string {
+        return $this->l->t('Mobile & desktop');
+    }
 
-	public function getPriority(): int {
-		return 15;
-	}
+    public function getPriority(): int {
+        return 15;
+    }
 }

apps/settings/lib/Sections/Personal/Security.php

Indentation +20 added lines, -20 removed lines

@@ -30,30 +30,30 @@
 
 class Security implements IIconSection {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
-	public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
-		$this->l = $l;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
+        $this->l = $l;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	public function getIcon() {
-		return $this->urlGenerator->imagePath('settings', 'password.svg');
-	}
+    public function getIcon() {
+        return $this->urlGenerator->imagePath('settings', 'password.svg');
+    }
 
-	public function getID(): string {
-		return 'security';
-	}
+    public function getID(): string {
+        return 'security';
+    }
 
-	public function getName(): string {
-		return $this->l->t('Security');
-	}
+    public function getName(): string {
+        return $this->l->t('Security');
+    }
 
-	public function getPriority(): int {
-		return 5;
-	}
+    public function getPriority(): int {
+        return 5;
+    }
 }

apps/settings/lib/Sections/Personal/PersonalInfo.php

Indentation +20 added lines, -20 removed lines

@@ -30,30 +30,30 @@
 
 class PersonalInfo implements IIconSection {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
-	public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
-		$this->l = $l;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
+        $this->l = $l;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	public function getIcon() {
-		return $this->urlGenerator->imagePath('core', 'actions/user.svg');
-	}
+    public function getIcon() {
+        return $this->urlGenerator->imagePath('core', 'actions/user.svg');
+    }
 
-	public function getID(): string {
-		return 'personal-info';
-	}
+    public function getID(): string {
+        return 'personal-info';
+    }
 
-	public function getName(): string {
-		return $this->l->t('Personal info');
-	}
+    public function getName(): string {
+        return $this->l->t('Personal info');
+    }
 
-	public function getPriority(): int {
-		return 0;
-	}
+    public function getPriority(): int {
+        return 0;
+    }
 }

apps/settings/lib/Settings/Personal/Security/TwoFactor.php

Spacing +2 added lines, -2 removed lines

@@ -106,12 +106,12 @@
 		}
 
 		return [
-			'providers' => array_map(function (IProvidesPersonalSettings $provider) use ($user) {
+			'providers' => array_map(function(IProvidesPersonalSettings $provider) use ($user) {
 				return [
 					'provider' => $provider,
 					'settings' => $provider->getPersonalSettings($user)
 				];
-			}, array_filter($this->providerLoader->getProviders($user), function (IProvider $provider) {
+			}, array_filter($this->providerLoader->getProviders($user), function(IProvider $provider) {
 				return $provider instanceof IProvidesPersonalSettings;
 			}))
 		];

Indentation +91 added lines, -91 removed lines

@@ -42,95 +42,95 @@
 
 class TwoFactor implements ISettings {
 
-	/** @var ProviderLoader */
-	private $providerLoader;
-
-	/** @var MandatoryTwoFactor */
-	private $mandatoryTwoFactor;
-
-	/** @var IUserSession */
-	private $userSession;
-
-	/** @var string|null */
-	private $uid;
-
-	/** @var IConfig */
-	private $config;
-
-	public function __construct(ProviderLoader $providerLoader,
-								MandatoryTwoFactor $mandatoryTwoFactor,
-								IUserSession $userSession,
-								IConfig $config,
-								?string $UserId) {
-		$this->providerLoader = $providerLoader;
-		$this->mandatoryTwoFactor = $mandatoryTwoFactor;
-		$this->userSession = $userSession;
-		$this->uid = $UserId;
-		$this->config = $config;
-	}
-
-	public function getForm(): TemplateResponse {
-		return new TemplateResponse('settings', 'settings/personal/security/twofactor', [
-			'twoFactorProviderData' => $this->getTwoFactorProviderData(),
-		]);
-	}
-
-	public function getSection(): ?string {
-		if (!$this->shouldShow()) {
-			return null;
-		}
-		return 'security';
-	}
-
-	public function getPriority(): int {
-		return 15;
-	}
-
-	private function shouldShow(): bool {
-		$user = $this->userSession->getUser();
-		if (is_null($user)) {
-			// Actually impossible, but still …
-			return false;
-		}
-
-		// Anyone who's supposed to use 2FA should see 2FA settings
-		if ($this->mandatoryTwoFactor->isEnforcedFor($user)) {
-			return true;
-		}
-
-		// If there is at least one provider with personal settings but it's not
-		// the backup codes provider, then these settings should show.
-		try {
-			$providers = $this->providerLoader->getProviders($user);
-		} catch (Exception $e) {
-			// Let's hope for the best
-			return true;
-		}
-		foreach ($providers as $provider) {
-			if ($provider instanceof IProvidesPersonalSettings
-				&& !($provider instanceof BackupCodesProvider)) {
-				return true;
-			}
-		}
-		return false;
-	}
-
-	private function getTwoFactorProviderData(): array {
-		$user = $this->userSession->getUser();
-		if (is_null($user)) {
-			// Actually impossible, but still …
-			return [];
-		}
-
-		return [
-			'providers' => array_map(function (IProvidesPersonalSettings $provider) use ($user) {
-				return [
-					'provider' => $provider,
-					'settings' => $provider->getPersonalSettings($user)
-				];
-			}, array_filter($this->providerLoader->getProviders($user), function (IProvider $provider) {
-				return $provider instanceof IProvidesPersonalSettings;
-			}))
-		];
-	}
+    /** @var ProviderLoader */
+    private $providerLoader;
+
+    /** @var MandatoryTwoFactor */
+    private $mandatoryTwoFactor;
+
+    /** @var IUserSession */
+    private $userSession;
+
+    /** @var string|null */
+    private $uid;
+
+    /** @var IConfig */
+    private $config;
+
+    public function __construct(ProviderLoader $providerLoader,
+                                MandatoryTwoFactor $mandatoryTwoFactor,
+                                IUserSession $userSession,
+                                IConfig $config,
+                                ?string $UserId) {
+        $this->providerLoader = $providerLoader;
+        $this->mandatoryTwoFactor = $mandatoryTwoFactor;
+        $this->userSession = $userSession;
+        $this->uid = $UserId;
+        $this->config = $config;
+    }
+
+    public function getForm(): TemplateResponse {
+        return new TemplateResponse('settings', 'settings/personal/security/twofactor', [
+            'twoFactorProviderData' => $this->getTwoFactorProviderData(),
+        ]);
+    }
+
+    public function getSection(): ?string {
+        if (!$this->shouldShow()) {
+            return null;
+        }
+        return 'security';
+    }
+
+    public function getPriority(): int {
+        return 15;
+    }
+
+    private function shouldShow(): bool {
+        $user = $this->userSession->getUser();
+        if (is_null($user)) {
+            // Actually impossible, but still …
+            return false;
+        }
+
+        // Anyone who's supposed to use 2FA should see 2FA settings
+        if ($this->mandatoryTwoFactor->isEnforcedFor($user)) {
+            return true;
+        }
+
+        // If there is at least one provider with personal settings but it's not
+        // the backup codes provider, then these settings should show.
+        try {
+            $providers = $this->providerLoader->getProviders($user);
+        } catch (Exception $e) {
+            // Let's hope for the best
+            return true;
+        }
+        foreach ($providers as $provider) {
+            if ($provider instanceof IProvidesPersonalSettings
+                && !($provider instanceof BackupCodesProvider)) {
+                return true;
+            }
+        }
+        return false;
+    }
+
+    private function getTwoFactorProviderData(): array {
+        $user = $this->userSession->getUser();
+        if (is_null($user)) {
+            // Actually impossible, but still …
+            return [];
+        }
+
+        return [
+            'providers' => array_map(function (IProvidesPersonalSettings $provider) use ($user) {
+                return [
+                    'provider' => $provider,
+                    'settings' => $provider->getPersonalSettings($user)
+                ];
+            }, array_filter($this->providerLoader->getProviders($user), function (IProvider $provider) {
+                return $provider instanceof IProvidesPersonalSettings;
+            }))
+        ];
+    }
 }

apps/settings/lib/Settings/Personal/Security/Password.php

Indentation +25 added lines, -25 removed lines

@@ -34,35 +34,35 @@
 
 class Password implements ISettings {
 
-	/** @var IUserManager */
-	private $userManager;
+    /** @var IUserManager */
+    private $userManager;
 
-	/** @var string|null */
-	private $uid;
+    /** @var string|null */
+    private $uid;
 
-	public function __construct(IUserManager $userManager,
-								?string $UserId) {
-		$this->userManager = $userManager;
-		$this->uid = $UserId;
-	}
+    public function __construct(IUserManager $userManager,
+                                ?string $UserId) {
+        $this->userManager = $userManager;
+        $this->uid = $UserId;
+    }
 
-	public function getForm(): TemplateResponse {
-		$user = $this->userManager->get($this->uid);
-		$passwordChangeSupported = false;
-		if ($user !== null) {
-			$passwordChangeSupported = $user->canChangePassword();
-		}
+    public function getForm(): TemplateResponse {
+        $user = $this->userManager->get($this->uid);
+        $passwordChangeSupported = false;
+        if ($user !== null) {
+            $passwordChangeSupported = $user->canChangePassword();
+        }
 
-		return new TemplateResponse('settings', 'settings/personal/security/password', [
-			'passwordChangeSupported' => $passwordChangeSupported,
-		]);
-	}
+        return new TemplateResponse('settings', 'settings/personal/security/password', [
+            'passwordChangeSupported' => $passwordChangeSupported,
+        ]);
+    }
 
-	public function getSection(): string {
-		return 'security';
-	}
+    public function getSection(): string {
+        return 'security';
+    }
 
-	public function getPriority(): int {
-		return 10;
-	}
+    public function getPriority(): int {
+        return 10;
+    }
 }

lib/private/BackgroundJob/Job.php

Spacing +3 added lines, -3 removed lines

@@ -57,17 +57,17 @@
 
 		try {
 			$jobStartTime = time();
-			$logger->debug('Run ' . get_class($this) . ' job with ID ' . $this->getId(), ['app' => 'cron']);
+			$logger->debug('Run '.get_class($this).' job with ID '.$this->getId(), ['app' => 'cron']);
 			$this->run($this->argument);
 			$timeTaken = time() - $jobStartTime;
 
-			$logger->debug('Finished ' . get_class($this) . ' job with ID ' . $this->getId() . ' in ' . $timeTaken . ' seconds', ['app' => 'cron']);
+			$logger->debug('Finished '.get_class($this).' job with ID '.$this->getId().' in '.$timeTaken.' seconds', ['app' => 'cron']);
 			$jobList->setExecutionTime($this, $timeTaken);
 		} catch (\Throwable $e) {
 			if ($logger) {
 				$logger->logException($e, [
 					'app' => 'core',
-					'message' => 'Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . ')'
+					'message' => 'Error while running background job (class: '.get_class($this).', arguments: '.print_r($this->argument, true).')'
 				]);
 			}
 		}

Indentation +49 added lines, -49 removed lines

@@ -33,66 +33,66 @@
  * @deprecated internal class, use \OCP\BackgroundJob\Job
  */
 abstract class Job implements IJob {
-	/** @var int */
-	protected $id;
+    /** @var int */
+    protected $id;
 
-	/** @var int */
-	protected $lastRun;
+    /** @var int */
+    protected $lastRun;
 
-	/** @var mixed */
-	protected $argument;
+    /** @var mixed */
+    protected $argument;
 
-	public function execute(IJobList $jobList, ILogger $logger = null) {
-		$jobList->setLastRun($this);
-		if ($logger === null) {
-			$logger = \OC::$server->getLogger();
-		}
+    public function execute(IJobList $jobList, ILogger $logger = null) {
+        $jobList->setLastRun($this);
+        if ($logger === null) {
+            $logger = \OC::$server->getLogger();
+        }
 
-		try {
-			$jobStartTime = time();
-			$logger->debug('Run ' . get_class($this) . ' job with ID ' . $this->getId(), ['app' => 'cron']);
-			$this->run($this->argument);
-			$timeTaken = time() - $jobStartTime;
+        try {
+            $jobStartTime = time();
+            $logger->debug('Run ' . get_class($this) . ' job with ID ' . $this->getId(), ['app' => 'cron']);
+            $this->run($this->argument);
+            $timeTaken = time() - $jobStartTime;
 
-			$logger->debug('Finished ' . get_class($this) . ' job with ID ' . $this->getId() . ' in ' . $timeTaken . ' seconds', ['app' => 'cron']);
-			$jobList->setExecutionTime($this, $timeTaken);
-		} catch (\Throwable $e) {
-			if ($logger) {
-				$logger->logException($e, [
-					'app' => 'core',
-					'message' => 'Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . ')'
-				]);
-			}
-		}
-	}
+            $logger->debug('Finished ' . get_class($this) . ' job with ID ' . $this->getId() . ' in ' . $timeTaken . ' seconds', ['app' => 'cron']);
+            $jobList->setExecutionTime($this, $timeTaken);
+        } catch (\Throwable $e) {
+            if ($logger) {
+                $logger->logException($e, [
+                    'app' => 'core',
+                    'message' => 'Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . ')'
+                ]);
+            }
+        }
+    }
 
-	public function start(IJobList $jobList): void {
-		$this->execute($jobList);
-	}
+    public function start(IJobList $jobList): void {
+        $this->execute($jobList);
+    }
 
-	abstract protected function run($argument);
+    abstract protected function run($argument);
 
-	public function setId(int $id) {
-		$this->id = $id;
-	}
+    public function setId(int $id) {
+        $this->id = $id;
+    }
 
-	public function setLastRun(int $lastRun) {
-		$this->lastRun = $lastRun;
-	}
+    public function setLastRun(int $lastRun) {
+        $this->lastRun = $lastRun;
+    }
 
-	public function setArgument($argument) {
-		$this->argument = $argument;
-	}
+    public function setArgument($argument) {
+        $this->argument = $argument;
+    }
 
-	public function getId() {
-		return $this->id;
-	}
+    public function getId() {
+        return $this->id;
+    }
 
-	public function getLastRun() {
-		return $this->lastRun;
-	}
+    public function getLastRun() {
+        return $this->lastRun;
+    }
 
-	public function getArgument() {
-		return $this->argument;
-	}
+    public function getArgument() {
+        return $this->argument;
+    }
 }

lib/private/DB/SetTransactionIsolationLevel.php

Indentation +10 added lines, -10 removed lines

@@ -32,15 +32,15 @@
 use Doctrine\DBAL\TransactionIsolationLevel;
 
 class SetTransactionIsolationLevel implements EventSubscriber {
-	/**
-	 * @param ConnectionEventArgs $args
-	 * @return void
-	 */
-	public function postConnect(ConnectionEventArgs $args) {
-		$args->getConnection()->setTransactionIsolation(TransactionIsolationLevel::READ_COMMITTED);
-	}
+    /**
+     * @param ConnectionEventArgs $args
+     * @return void
+     */
+    public function postConnect(ConnectionEventArgs $args) {
+        $args->getConnection()->setTransactionIsolation(TransactionIsolationLevel::READ_COMMITTED);
+    }
 
-	public function getSubscribedEvents() {
-		return [Events::postConnect];
-	}
+    public function getSubscribedEvents() {
+        return [Events::postConnect];
+    }
 }