nextcloud / server

core/Controller/WipeController.php

Indentation +52 added lines, -52 removed lines

@@ -35,64 +35,64 @@
 
 class WipeController extends Controller {
 
-	/** @var RemoteWipe */
-	private $remoteWipe;
+    /** @var RemoteWipe */
+    private $remoteWipe;
 
-	public function __construct(string $appName,
-								IRequest $request,
-								RemoteWipe $remoteWipe) {
-		parent::__construct($appName, $request);
+    public function __construct(string $appName,
+                                IRequest $request,
+                                RemoteWipe $remoteWipe) {
+        parent::__construct($appName, $request);
 
-		$this->remoteWipe = $remoteWipe;
-	}
+        $this->remoteWipe = $remoteWipe;
+    }
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 *
-	 * @AnonRateThrottle(limit=10, period=300)
-	 *
-	 * @param string $token
-	 *
-	 * @return JSONResponse
-	 */
-	public function checkWipe(string $token): JSONResponse {
-		try {
-			if ($this->remoteWipe->start($token)) {
-				return new JSONResponse([
-					'wipe' => true
-				]);
-			}
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @PublicPage
+     *
+     * @AnonRateThrottle(limit=10, period=300)
+     *
+     * @param string $token
+     *
+     * @return JSONResponse
+     */
+    public function checkWipe(string $token): JSONResponse {
+        try {
+            if ($this->remoteWipe->start($token)) {
+                return new JSONResponse([
+                    'wipe' => true
+                ]);
+            }
 
-			return new JSONResponse([], Http::STATUS_NOT_FOUND);
-		} catch (InvalidTokenException $e) {
-			return new JSONResponse([], Http::STATUS_NOT_FOUND);
-		}
-	}
+            return new JSONResponse([], Http::STATUS_NOT_FOUND);
+        } catch (InvalidTokenException $e) {
+            return new JSONResponse([], Http::STATUS_NOT_FOUND);
+        }
+    }
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 *
-	 * @AnonRateThrottle(limit=10, period=300)
-	 *
-	 * @param string $token
-	 *
-	 * @return JSONResponse
-	 */
-	public function wipeDone(string $token): JSONResponse {
-		try {
-			if ($this->remoteWipe->finish($token)) {
-				return new JSONResponse([]);
-			}
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @PublicPage
+     *
+     * @AnonRateThrottle(limit=10, period=300)
+     *
+     * @param string $token
+     *
+     * @return JSONResponse
+     */
+    public function wipeDone(string $token): JSONResponse {
+        try {
+            if ($this->remoteWipe->finish($token)) {
+                return new JSONResponse([]);
+            }
 
-			return new JSONResponse([], Http::STATUS_NOT_FOUND);
-		} catch (InvalidTokenException $e) {
-			return new JSONResponse([], Http::STATUS_NOT_FOUND);
-		}
-	}
+            return new JSONResponse([], Http::STATUS_NOT_FOUND);
+        } catch (InvalidTokenException $e) {
+            return new JSONResponse([], Http::STATUS_NOT_FOUND);
+        }
+    }
 
 }

lib/private/L10N/LazyL10N.php

Indentation +28 added lines, -28 removed lines

@@ -28,43 +28,43 @@
 
 class LazyL10N implements IL10N {
 
-	/** @var IL10N */
-	private $l;
+    /** @var IL10N */
+    private $l;
 
-	/** @var \Closure */
-	private $factory;
+    /** @var \Closure */
+    private $factory;
 
 
-	public function __construct(\Closure $factory) {
-		$this->factory = $factory;
-	}
+    public function __construct(\Closure $factory) {
+        $this->factory = $factory;
+    }
 
-	private function getL(): IL10N {
-		if ($this->l === null) {
-			$this->l = ($this->factory)();
-		}
+    private function getL(): IL10N {
+        if ($this->l === null) {
+            $this->l = ($this->factory)();
+        }
 
-		return $this->l;
-	}
+        return $this->l;
+    }
 
-	public function t(string $text, $parameters = []): string {
-		return $this->getL()->t($text, $parameters);
-	}
+    public function t(string $text, $parameters = []): string {
+        return $this->getL()->t($text, $parameters);
+    }
 
-	public function n(string $text_singular, string $text_plural, int $count, array $parameters = []): string {
-		return $this->getL()->n($text_singular, $text_plural, $count, $parameters);
-	}
+    public function n(string $text_singular, string $text_plural, int $count, array $parameters = []): string {
+        return $this->getL()->n($text_singular, $text_plural, $count, $parameters);
+    }
 
-	public function l(string $type, $data, array $options = []) {
-		return $this->getL()->l($type, $data, $options);
-	}
+    public function l(string $type, $data, array $options = []) {
+        return $this->getL()->l($type, $data, $options);
+    }
 
-	public function getLanguageCode(): string {
-		return $this->getL()->getLanguageCode();
-	}
+    public function getLanguageCode(): string {
+        return $this->getL()->getLanguageCode();
+    }
 
-	public function getLocaleCode(): string {
-		return $this->getL()->getLocaleCode();
-	}
+    public function getLocaleCode(): string {
+        return $this->getL()->getLocaleCode();
+    }
 
 }

apps/dav/lib/Migration/RegenerateBirthdayCalendars.php

Indentation +33 added lines, -33 removed lines

@@ -30,43 +30,43 @@
 
 class RegenerateBirthdayCalendars implements IRepairStep {
 
-	/** @var IJobList */
-	private $jobList;
+    /** @var IJobList */
+    private $jobList;
 
-	/** @var IConfig */
-	private $config;
+    /** @var IConfig */
+    private $config;
 
-	/**
-	 * @param IJobList $jobList
-	 * @param IConfig $config
-	 */
-	public function __construct(IJobList $jobList,
-								IConfig $config) {
-		$this->jobList = $jobList;
-		$this->config = $config;
-	}
+    /**
+     * @param IJobList $jobList
+     * @param IConfig $config
+     */
+    public function __construct(IJobList $jobList,
+                                IConfig $config) {
+        $this->jobList = $jobList;
+        $this->config = $config;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getName() {
-		return 'Regenerating birthday calendars to use new icons and fix old birthday events without year';
-	}
+    /**
+     * @return string
+     */
+    public function getName() {
+        return 'Regenerating birthday calendars to use new icons and fix old birthday events without year';
+    }
 
-	/**
-	 * @param IOutput $output
-	 */
-	public function run(IOutput $output) {
-		// only run once
-		if ($this->config->getAppValue('dav', 'regeneratedBirthdayCalendarsForYearFix') === 'yes') {
-			$output->info('Repair step already executed');
-			return;
-		}
+    /**
+     * @param IOutput $output
+     */
+    public function run(IOutput $output) {
+        // only run once
+        if ($this->config->getAppValue('dav', 'regeneratedBirthdayCalendarsForYearFix') === 'yes') {
+            $output->info('Repair step already executed');
+            return;
+        }
 
-		$output->info('Adding background jobs to regenerate birthday calendar');
-		$this->jobList->add(RegisterRegenerateBirthdayCalendars::class);
+        $output->info('Adding background jobs to regenerate birthday calendar');
+        $this->jobList->add(RegisterRegenerateBirthdayCalendars::class);
 
-		// if all were done, no need to redo the repair during next upgrade
-		$this->config->setAppValue('dav', 'regeneratedBirthdayCalendarsForYearFix', 'yes');
-	}
+        // if all were done, no need to redo the repair during next upgrade
+        $this->config->setAppValue('dav', 'regeneratedBirthdayCalendarsForYearFix', 'yes');
+    }
 }

apps/federatedfilesharing/lib/Settings/Personal.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 	 */
 	public function getForm() {
 		$cloudID = $this->userSession->getUser()->getCloudId();
-		$url = 'https://nextcloud.com/sharing#' . $cloudID;
+		$url = 'https://nextcloud.com/sharing#'.$cloudID;
 
 		$parameters = [
 			'message_with_URL' => $this->l->t('Share with me through my #Nextcloud Federated Cloud ID, see %s', [$url]),

Indentation +67 added lines, -67 removed lines

@@ -35,78 +35,78 @@
 use OCP\Settings\ISettings;
 
 class Personal implements ISettings {
-	private FederatedShareProvider $federatedShareProvider;
-	private IUserSession $userSession;
-	private IL10N $l;
-	private \OC_Defaults $defaults;
-	private IInitialState $initialState;
-	private IURLGenerator $urlGenerator;
+    private FederatedShareProvider $federatedShareProvider;
+    private IUserSession $userSession;
+    private IL10N $l;
+    private \OC_Defaults $defaults;
+    private IInitialState $initialState;
+    private IURLGenerator $urlGenerator;
 
-	public function __construct(
-		FederatedShareProvider $federatedShareProvider, #
-		IUserSession $userSession,
-		IL10N $l,
-		\OC_Defaults $defaults,
-		IInitialState $initialState,
-		IURLGenerator $urlGenerator
-	) {
-		$this->federatedShareProvider = $federatedShareProvider;
-		$this->userSession = $userSession;
-		$this->l = $l;
-		$this->defaults = $defaults;
-		$this->initialState = $initialState;
-		$this->urlGenerator = $urlGenerator;
-	}
+    public function __construct(
+        FederatedShareProvider $federatedShareProvider, #
+        IUserSession $userSession,
+        IL10N $l,
+        \OC_Defaults $defaults,
+        IInitialState $initialState,
+        IURLGenerator $urlGenerator
+    ) {
+        $this->federatedShareProvider = $federatedShareProvider;
+        $this->userSession = $userSession;
+        $this->l = $l;
+        $this->defaults = $defaults;
+        $this->initialState = $initialState;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	/**
-	 * @return TemplateResponse returns the instance with all parameters set, ready to be rendered
-	 * @since 9.1
-	 */
-	public function getForm() {
-		$cloudID = $this->userSession->getUser()->getCloudId();
-		$url = 'https://nextcloud.com/sharing#' . $cloudID;
+    /**
+     * @return TemplateResponse returns the instance with all parameters set, ready to be rendered
+     * @since 9.1
+     */
+    public function getForm() {
+        $cloudID = $this->userSession->getUser()->getCloudId();
+        $url = 'https://nextcloud.com/sharing#' . $cloudID;
 
-		$parameters = [
-			'message_with_URL' => $this->l->t('Share with me through my #Nextcloud Federated Cloud ID, see %s', [$url]),
-			'message_without_URL' => $this->l->t('Share with me through my #Nextcloud Federated Cloud ID', [$cloudID]),
-			'logoPath' => $this->defaults->getLogo(),
-			'reference' => $url,
-			'cloudId' => $cloudID,
-			'color' => $this->defaults->getColorPrimary(),
-			'textColor' => "#ffffff",
-		];
+        $parameters = [
+            'message_with_URL' => $this->l->t('Share with me through my #Nextcloud Federated Cloud ID, see %s', [$url]),
+            'message_without_URL' => $this->l->t('Share with me through my #Nextcloud Federated Cloud ID', [$cloudID]),
+            'logoPath' => $this->defaults->getLogo(),
+            'reference' => $url,
+            'cloudId' => $cloudID,
+            'color' => $this->defaults->getColorPrimary(),
+            'textColor' => "#ffffff",
+        ];
 
-		$this->initialState->provideInitialState('color', $this->defaults->getColorPrimary());
-		$this->initialState->provideInitialState('textColor', '#fffff');
-		$this->initialState->provideInitialState('logoPath', $this->defaults->getLogo());
-		$this->initialState->provideInitialState('reference', $url);
-		$this->initialState->provideInitialState('cloudId', $cloudID);
-		$this->initialState->provideInitialState('docUrlFederated', $this->urlGenerator->linkToDocs('user-sharing-federated'));
+        $this->initialState->provideInitialState('color', $this->defaults->getColorPrimary());
+        $this->initialState->provideInitialState('textColor', '#fffff');
+        $this->initialState->provideInitialState('logoPath', $this->defaults->getLogo());
+        $this->initialState->provideInitialState('reference', $url);
+        $this->initialState->provideInitialState('cloudId', $cloudID);
+        $this->initialState->provideInitialState('docUrlFederated', $this->urlGenerator->linkToDocs('user-sharing-federated'));
 
-		return new TemplateResponse('federatedfilesharing', 'settings-personal', $parameters, '');
-	}
+        return new TemplateResponse('federatedfilesharing', 'settings-personal', $parameters, '');
+    }
 
-	/**
-	 * @return string the section ID, e.g. 'sharing'
-	 * @since 9.1
-	 */
-	public function getSection() {
-		if ($this->federatedShareProvider->isIncomingServer2serverShareEnabled() ||
-			$this->federatedShareProvider->isIncomingServer2serverGroupShareEnabled()) {
-			return 'sharing';
-		}
-		return null;
-	}
+    /**
+     * @return string the section ID, e.g. 'sharing'
+     * @since 9.1
+     */
+    public function getSection() {
+        if ($this->federatedShareProvider->isIncomingServer2serverShareEnabled() ||
+            $this->federatedShareProvider->isIncomingServer2serverGroupShareEnabled()) {
+            return 'sharing';
+        }
+        return null;
+    }
 
-	/**
-	 * @return int whether the form should be rather on the top or bottom of
-	 * the admin section. The forms are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
-	 *
-	 * E.g.: 70
-	 * @since 9.1
-	 */
-	public function getPriority() {
-		return 40;
-	}
+    /**
+     * @return int whether the form should be rather on the top or bottom of
+     * the admin section. The forms are arranged in ascending order of the
+     * priority values. It is required to return a value between 0 and 100.
+     *
+     * E.g.: 70
+     * @since 9.1
+     */
+    public function getPriority() {
+        return 40;
+    }
 }

apps/files_versions/lib/Versions/Version.php

Indentation +84 added lines, -84 removed lines

@@ -26,88 +26,88 @@
 use OCP\IUser;
 
 class Version implements IVersion {
-	/** @var int */
-	private $timestamp;
-
-	/** @var int|string */
-	private $revisionId;
-
-	/** @var string */
-	private $name;
-
-	/** @var int */
-	private $size;
-
-	/** @var string */
-	private $mimetype;
-
-	/** @var string */
-	private $path;
-
-	/** @var FileInfo */
-	private $sourceFileInfo;
-
-	/** @var IVersionBackend */
-	private $backend;
-
-	/** @var IUser */
-	private $user;
-
-	public function __construct(
-		int $timestamp,
-		$revisionId,
-		string $name,
-		int $size,
-		string $mimetype,
-		string $path,
-		FileInfo $sourceFileInfo,
-		IVersionBackend $backend,
-		IUser $user
-	) {
-		$this->timestamp = $timestamp;
-		$this->revisionId = $revisionId;
-		$this->name = $name;
-		$this->size = $size;
-		$this->mimetype = $mimetype;
-		$this->path = $path;
-		$this->sourceFileInfo = $sourceFileInfo;
-		$this->backend = $backend;
-		$this->user = $user;
-	}
-
-	public function getBackend(): IVersionBackend {
-		return $this->backend;
-	}
-
-	public function getSourceFile(): FileInfo {
-		return $this->sourceFileInfo;
-	}
-
-	public function getRevisionId() {
-		return $this->revisionId;
-	}
-
-	public function getTimestamp(): int {
-		return $this->timestamp;
-	}
-
-	public function getSize(): int {
-		return $this->size;
-	}
-
-	public function getSourceFileName(): string {
-		return $this->name;
-	}
-
-	public function getMimeType(): string {
-		return $this->mimetype;
-	}
-
-	public function getVersionPath(): string {
-		return $this->path;
-	}
-
-	public function getUser(): IUser {
-		return $this->user;
-	}
+    /** @var int */
+    private $timestamp;
+
+    /** @var int|string */
+    private $revisionId;
+
+    /** @var string */
+    private $name;
+
+    /** @var int */
+    private $size;
+
+    /** @var string */
+    private $mimetype;
+
+    /** @var string */
+    private $path;
+
+    /** @var FileInfo */
+    private $sourceFileInfo;
+
+    /** @var IVersionBackend */
+    private $backend;
+
+    /** @var IUser */
+    private $user;
+
+    public function __construct(
+        int $timestamp,
+        $revisionId,
+        string $name,
+        int $size,
+        string $mimetype,
+        string $path,
+        FileInfo $sourceFileInfo,
+        IVersionBackend $backend,
+        IUser $user
+    ) {
+        $this->timestamp = $timestamp;
+        $this->revisionId = $revisionId;
+        $this->name = $name;
+        $this->size = $size;
+        $this->mimetype = $mimetype;
+        $this->path = $path;
+        $this->sourceFileInfo = $sourceFileInfo;
+        $this->backend = $backend;
+        $this->user = $user;
+    }
+
+    public function getBackend(): IVersionBackend {
+        return $this->backend;
+    }
+
+    public function getSourceFile(): FileInfo {
+        return $this->sourceFileInfo;
+    }
+
+    public function getRevisionId() {
+        return $this->revisionId;
+    }
+
+    public function getTimestamp(): int {
+        return $this->timestamp;
+    }
+
+    public function getSize(): int {
+        return $this->size;
+    }
+
+    public function getSourceFileName(): string {
+        return $this->name;
+    }
+
+    public function getMimeType(): string {
+        return $this->mimetype;
+    }
+
+    public function getVersionPath(): string {
+        return $this->path;
+    }
+
+    public function getUser(): IUser {
+        return $this->user;
+    }
 }

apps/files_versions/lib/Versions/IVersion.php

Indentation +59 added lines, -59 removed lines

@@ -29,71 +29,71 @@
  * @since 15.0.0
  */
 interface IVersion {
-	/**
-	 * @return IVersionBackend
-	 * @since 15.0.0
-	 */
-	public function getBackend(): IVersionBackend;
+    /**
+     * @return IVersionBackend
+     * @since 15.0.0
+     */
+    public function getBackend(): IVersionBackend;
 
-	/**
-	 * Get the file info of the source file
-	 *
-	 * @return FileInfo
-	 * @since 15.0.0
-	 */
-	public function getSourceFile(): FileInfo;
+    /**
+     * Get the file info of the source file
+     *
+     * @return FileInfo
+     * @since 15.0.0
+     */
+    public function getSourceFile(): FileInfo;
 
-	/**
-	 * Get the id of the revision for the file
-	 *
-	 * @return int|string
-	 * @since 15.0.0
-	 */
-	public function getRevisionId();
+    /**
+     * Get the id of the revision for the file
+     *
+     * @return int|string
+     * @since 15.0.0
+     */
+    public function getRevisionId();
 
-	/**
-	 * Get the timestamp this version was created
-	 *
-	 * @return int
-	 * @since 15.0.0
-	 */
-	public function getTimestamp(): int;
+    /**
+     * Get the timestamp this version was created
+     *
+     * @return int
+     * @since 15.0.0
+     */
+    public function getTimestamp(): int;
 
-	/**
-	 * Get the size of this version
-	 *
-	 * @return int
-	 * @since 15.0.0
-	 */
-	public function getSize(): int;
+    /**
+     * Get the size of this version
+     *
+     * @return int
+     * @since 15.0.0
+     */
+    public function getSize(): int;
 
-	/**
-	 * Get the name of the source file at the time of making this version
-	 *
-	 * @return string
-	 * @since 15.0.0
-	 */
-	public function getSourceFileName(): string;
+    /**
+     * Get the name of the source file at the time of making this version
+     *
+     * @return string
+     * @since 15.0.0
+     */
+    public function getSourceFileName(): string;
 
-	/**
-	 * Get the mimetype of this version
-	 *
-	 * @return string
-	 * @since 15.0.0
-	 */
-	public function getMimeType(): string;
+    /**
+     * Get the mimetype of this version
+     *
+     * @return string
+     * @since 15.0.0
+     */
+    public function getMimeType(): string;
 
-	/**
-	 * Get the path of this version
-	 *
-	 * @return string
-	 * @since 15.0.0
-	 */
-	public function getVersionPath(): string;
+    /**
+     * Get the path of this version
+     *
+     * @return string
+     * @since 15.0.0
+     */
+    public function getVersionPath(): string;
 
-	/**
-	 * @return IUser
-	 * @since 15.0.0
-	 */
-	public function getUser(): IUser;
+    /**
+     * @return IUser
+     * @since 15.0.0
+     */
+    public function getUser(): IUser;
 }

apps/user_ldap/lib/ILDAPUserPlugin.php

Indentation +56 added lines, -56 removed lines

@@ -26,68 +26,68 @@
 
 interface ILDAPUserPlugin {
 
-	/**
-	 * Check if plugin implements actions
-	 * @return int
-	 *
-	 * Returns the supported actions as int to be
-	 * compared with OC_USER_BACKEND_CREATE_USER etc.
-	 */
-	public function respondToActions();
+    /**
+     * Check if plugin implements actions
+     * @return int
+     *
+     * Returns the supported actions as int to be
+     * compared with OC_USER_BACKEND_CREATE_USER etc.
+     */
+    public function respondToActions();
 
-	/**
-	 * Create a new user in LDAP Backend
-	 *
-	 * @param string $uid The UID of the user to create
-	 * @param string $password The password of the new user
-	 * @return bool|string
-	 */
-	public function createUser($uid, $password);
+    /**
+     * Create a new user in LDAP Backend
+     *
+     * @param string $uid The UID of the user to create
+     * @param string $password The password of the new user
+     * @return bool|string
+     */
+    public function createUser($uid, $password);
 
-	/**
-	 * Set password
-	 *
-	 * @param string $uid The username
-	 * @param string $password The new password
-	 * @return bool
-	 *
-	 * Change the password of a user
-	 */
-	public function setPassword($uid, $password);
+    /**
+     * Set password
+     *
+     * @param string $uid The username
+     * @param string $password The new password
+     * @return bool
+     *
+     * Change the password of a user
+     */
+    public function setPassword($uid, $password);
 
-	/**
-	 * get the user's home directory
-	 * @param string $uid the username
-	 * @return boolean
-	 */
-	public function getHome($uid);
+    /**
+     * get the user's home directory
+     * @param string $uid the username
+     * @return boolean
+     */
+    public function getHome($uid);
 
-	/**
-	 * get display name of the user
-	 * @param string $uid user ID of the user
-	 * @return string display name
-	 */
-	public function getDisplayName($uid);
+    /**
+     * get display name of the user
+     * @param string $uid user ID of the user
+     * @return string display name
+     */
+    public function getDisplayName($uid);
 
-	/**
-	 * set display name of the user
-	 * @param string $uid user ID of the user
-	 * @param string $displayName new user's display name
-	 * @return string display name
-	 */
-	public function setDisplayName($uid, $displayName);
+    /**
+     * set display name of the user
+     * @param string $uid user ID of the user
+     * @param string $displayName new user's display name
+     * @return string display name
+     */
+    public function setDisplayName($uid, $displayName);
 
-	/**
-	 * checks whether the user is allowed to change his avatar in Nextcloud
-	 * @param string $uid the Nextcloud user name
-	 * @return boolean either the user can or cannot
-	 */
-	public function canChangeAvatar($uid);
+    /**
+     * checks whether the user is allowed to change his avatar in Nextcloud
+     * @param string $uid the Nextcloud user name
+     * @return boolean either the user can or cannot
+     */
+    public function canChangeAvatar($uid);
 
-	/**
-	 * Count the number of users
-	 * @return int|bool
-	 */
-	public function countUsers();
+    /**
+     * Count the number of users
+     * @return int|bool
+     */
+    public function countUsers();
 
 }

apps/twofactor_backupcodes/lib/Listener/ProviderEnabled.php

Indentation +25 added lines, -25 removed lines

@@ -33,30 +33,30 @@
 
 class ProviderEnabled implements IEventListener {
 
-	/** @var IRegistry */
-	private $registry;
-
-	/** @var IJobList */
-	private $jobList;
-
-	public function __construct(IRegistry $registry,
-								IJobList $jobList) {
-		$this->registry = $registry;
-		$this->jobList = $jobList;
-	}
-
-	public function handle(Event $event): void {
-		if (!($event instanceof RegistryEvent)) {
-			return;
-		}
-
-		$providers = $this->registry->getProviderStates($event->getUser());
-		if (isset($providers['backup_codes']) && $providers['backup_codes'] === true) {
-			// Backup codes already generated nothing to do here
-			return;
-		}
-
-		$this->jobList->add(RememberBackupCodesJob::class, ['uid' => $event->getUser()->getUID()]);
-	}
+    /** @var IRegistry */
+    private $registry;
+
+    /** @var IJobList */
+    private $jobList;
+
+    public function __construct(IRegistry $registry,
+                                IJobList $jobList) {
+        $this->registry = $registry;
+        $this->jobList = $jobList;
+    }
+
+    public function handle(Event $event): void {
+        if (!($event instanceof RegistryEvent)) {
+            return;
+        }
+
+        $providers = $this->registry->getProviderStates($event->getUser());
+        if (isset($providers['backup_codes']) && $providers['backup_codes'] === true) {
+            // Backup codes already generated nothing to do here
+            return;
+        }
+
+        $this->jobList->add(RememberBackupCodesJob::class, ['uid' => $event->getUser()->getUID()]);
+    }
 
 }

apps/twofactor_backupcodes/lib/Listener/ClearNotifications.php

Indentation +18 added lines, -18 removed lines

@@ -33,22 +33,22 @@
 
 class ClearNotifications implements IEventListener {
 
-	/** @var IManager */
-	private $manager;
-
-	public function __construct(IManager $manager) {
-		$this->manager = $manager;
-	}
-
-	public function handle(Event $event): void {
-		if (!($event instanceof CodesGenerated)) {
-			return;
-		}
-
-		$notification = $this->manager->createNotification();
-		$notification->setApp('twofactor_backupcodes')
-			->setUser($event->getUser()->getUID())
-			->setObject('create', 'codes');
-		$this->manager->markProcessed($notification);
-	}
+    /** @var IManager */
+    private $manager;
+
+    public function __construct(IManager $manager) {
+        $this->manager = $manager;
+    }
+
+    public function handle(Event $event): void {
+        if (!($event instanceof CodesGenerated)) {
+            return;
+        }
+
+        $notification = $this->manager->createNotification();
+        $notification->setApp('twofactor_backupcodes')
+            ->setUser($event->getUser()->getUID())
+            ->setObject('create', 'codes');
+        $this->manager->markProcessed($notification);
+    }
 }