Skip to content

fix(http): guard against null user in MailboxesController#12418

Merged
ChristophWurst merged 1 commit intomainfrom
fix/http/mailboxes-controller-null-user
Feb 13, 2026
Merged

fix(http): guard against null user in MailboxesController#12418
ChristophWurst merged 1 commit intomainfrom
fix/http/mailboxes-controller-null-user

Conversation

@ChristophWurst
Copy link
Member

For #11224

Error: lib/Controller/MailboxesController.php:73:42: PossiblyNullArgument: Argument 1 of OCA\Mail\Service\AccountService::find cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:80:38: PossiblyFalseReference: Cannot call method getDelimiter on possibly false value (see https://psalm.dev/105)
Error: lib/Controller/MailboxesController.php:97:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:139:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:181:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:199:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:219:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:255:42: PossiblyNullArgument: Argument 1 of OCA\Mail\Service\AccountService::find cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:271:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)
Error: lib/Controller/MailboxesController.php:290:45: PossiblyNullArgument: Argument 1 of OCA\Mail\Contracts\IMailManager::getMailbox cannot be null, possibly null value provided (see https://psalm.dev/078)

Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@ChristophWurst ChristophWurst merged commit 4570c30 into main Feb 13, 2026
39 of 42 checks passed
@ChristophWurst ChristophWurst deleted the fix/http/mailboxes-controller-null-user branch February 13, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments