[PR #2890] [MERGED] Apply fixes from StyleCI #6082

New Issue

OVERLORD · 2026-02-05T10:24:09+03:00

OVERLORD commented

2026-02-05 10:24:09 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/2890
Author: @ssddanbrown
Created: 8/21/2021
Status: ✅ Merged
Merged: 8/21/2021
Merged by: @ssddanbrown

Base: master ← Head: analysis-kabPRy

📝 Commits (1)

64785ed Apply fixes from StyleCI

📊 Changes

31 files changed (+173 additions, -161 deletions)

View changed files

📝 app/Auth/Access/EmailConfirmationService.php (+1 -0)
📝 app/Auth/Access/LoginService.php (+8 -4)
📝 app/Auth/Access/Mfa/BackupCodeService.php (+3 -1)
📝 app/Auth/Access/Mfa/MfaSession.php (+1 -2)
📝 app/Auth/Access/Mfa/MfaValue.php (+3 -3)
📝 app/Auth/Access/Mfa/TotpService.php (+3 -2)
📝 app/Auth/Access/Mfa/TotpValidationRule.php (+0 -1)
📝 app/Auth/Access/RegistrationService.php (+1 -0)
📝 app/Auth/Access/Saml2Service.php (+1 -4)
📝 app/Auth/Access/SocialAuthService.php (+1 -0)
📝 app/Console/Commands/ResetMfa.php (+3 -0)
📝 app/Exceptions/StoppedAuthenticationException.php (+2 -3)
📝 app/Http/Controllers/Auth/ConfirmEmailController.php (+2 -2)
📝 app/Http/Controllers/Auth/HandlesPartialLogins.php (+1 -1)
📝 app/Http/Controllers/Auth/LoginController.php (+5 -2)
📝 app/Http/Controllers/Auth/MfaBackupCodesController.php (+7 -4)
📝 app/Http/Controllers/Auth/MfaController.php (+5 -3)
📝 app/Http/Controllers/Auth/MfaTotpController.php (+6 -3)
📝 app/Http/Controllers/Auth/RegisterController.php (+1 -2)
📝 app/Http/Controllers/Auth/SocialController.php (+1 -2)

...and 11 more files

📄 Description

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/2890 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 8/21/2021 **Status:** ✅ Merged **Merged:** 8/21/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `analysis-kabPRy` --- ### 📝 Commits (1) - [`64785ed`](https://github.com/BookStackApp/BookStack/commit/64785ed9da0d3bac61e4b3976da2e47a804335ec) Apply fixes from StyleCI ### 📊 Changes **31 files changed** (+173 additions, -161 deletions) <details> <summary>View changed files</summary> 📝 `app/Auth/Access/EmailConfirmationService.php` (+1 -0) 📝 `app/Auth/Access/LoginService.php` (+8 -4) 📝 `app/Auth/Access/Mfa/BackupCodeService.php` (+3 -1) 📝 `app/Auth/Access/Mfa/MfaSession.php` (+1 -2) 📝 `app/Auth/Access/Mfa/MfaValue.php` (+3 -3) 📝 `app/Auth/Access/Mfa/TotpService.php` (+3 -2) 📝 `app/Auth/Access/Mfa/TotpValidationRule.php` (+0 -1) 📝 `app/Auth/Access/RegistrationService.php` (+1 -0) 📝 `app/Auth/Access/Saml2Service.php` (+1 -4) 📝 `app/Auth/Access/SocialAuthService.php` (+1 -0) 📝 `app/Console/Commands/ResetMfa.php` (+3 -0) 📝 `app/Exceptions/StoppedAuthenticationException.php` (+2 -3) 📝 `app/Http/Controllers/Auth/ConfirmEmailController.php` (+2 -2) 📝 `app/Http/Controllers/Auth/HandlesPartialLogins.php` (+1 -1) 📝 `app/Http/Controllers/Auth/LoginController.php` (+5 -2) 📝 `app/Http/Controllers/Auth/MfaBackupCodesController.php` (+7 -4) 📝 `app/Http/Controllers/Auth/MfaController.php` (+5 -3) 📝 `app/Http/Controllers/Auth/MfaTotpController.php` (+6 -3) 📝 `app/Http/Controllers/Auth/RegisterController.php` (+1 -2) 📝 `app/Http/Controllers/Auth/SocialController.php` (+1 -2) _...and 11 more files_ </details> ### 📄 Description This pull request applies code style fixes from an analysis carried out by [StyleCI](https://github.styleci.io). --- For more information, click [here](https://github.styleci.io/analyses/kabPRy). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:24:09 +03:00

OVERLORD closed this issue

2026-02-05 10:24:10 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6082