recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Fix PHP 8.1 "strlen(null)" exception on auth email message body when PhabricatorAuthMessage is unset

Summary:
This is a fix for running Phorge on PHP 8.2+. Other fixes were already merged.

Closes T15536

Test Plan: Unclear.

Reviewers: O1 Blessed Committers, avivey, aklapper

Reviewed By: O1 Blessed Committers, avivey, aklapper

Subscribers: aklapper, avivey, speck, tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Tags: #php_8_support

Maniphest Tasks: T15064, T15536

Differential Revision: https://we.phorge.it/D25256

Merula Turdus 99834dd0 0f7e76ff

+1 -1
+1 -1
src/applications/people/mail/PhabricatorPeopleEmailLoginMailEngine.php
··· 64 64 $message_body = PhabricatorAuthMessage::loadMessageText( 65 65 $recipient, 66 66 $message_key); 67 - if (strlen($message_body)) { 67 + if (phutil_nonempty_string($message_body)) { 68 68 $body[] = $this->newRemarkupText($message_body); 69 69 } 70 70