Skip to content

Commit

Permalink
Merge pull request #39449 from nextcloud/cleanup-2fa-manager
Browse files Browse the repository at this point in the history
docs: remove superfluous phpdocs
  • Loading branch information
AndyScherzinger authored Aug 10, 2023
2 parents b53d54c + 32303b6 commit 5408dbf
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions lib/private/Authentication/TwoFactorAuth/Manager.php
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,6 @@ public function __construct(ProviderLoader $providerLoader,

/**
* Determine whether the user must provide a second factor challenge
*
* @param IUser $user
* @return boolean
*/
public function isTwoFactorAuthenticated(IUser $user): bool {
if (isset($this->userIsTwoFactorAuthenticated[$user->getUID()])) {
Expand All @@ -138,18 +135,13 @@ public function isTwoFactorAuthenticated(IUser $user): bool {

/**
* Get a 2FA provider by its ID
*
* @param IUser $user
* @param string $challengeProviderId
* @return IProvider|null
*/
public function getProvider(IUser $user, string $challengeProviderId) {
public function getProvider(IUser $user, string $challengeProviderId): ?IProvider {
$providers = $this->getProviderSet($user)->getProviders();
return $providers[$challengeProviderId] ?? null;
}

/**
* @param IUser $user
* @return IActivatableAtLogin[]
* @throws Exception
*/
Expand Down

0 comments on commit 5408dbf

Please sign in to comment.