Differential D25317

Fix "Register with Approval flow" for php 8
ClosedPublic
Actions

Authored by avivey on Jun 30 2023, 08:34.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 28, 11:51

	Unknown Object (File)
	Fri, Apr 5, 04:27

	Unknown Object (File)
	Apr 1 2024, 02:13

	Unknown Object (File)
	Apr 1 2024, 02:13

	Unknown Object (File)
	Apr 1 2024, 02:13

	Unknown Object (File)
	Mar 31 2024, 03:10

	Unknown Object (File)
	Mar 27 2024, 16:13

	Unknown Object (File)
	Mar 24 2024, 07:10

View All 63 Files

Subscribers

valerio.bozzolan

Details

Reviewers

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15064: Make Phorge compatible with PHP 8.1/8.2/8.3

Commits

rPd725ffaa7728: Fix "Register with Approval flow" for php 8

Summary

Test Plan

Register as a new user
as admin, view user's page and approve it
As new user, hit "wait patiently" to go back to Home

No crashes in this flow.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avivey created this revision.Jun 30 2023, 08:34

Owners added a reviewer: O1: Blessed Committers.Jun 30 2023, 08:34

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan and 2 others. · View Herald TranscriptJun 30 2023, 08:34

avivey requested review of this revision.Jun 30 2023, 08:34

Harbormaster completed remote builds in B612: Diff 1037.Jun 30 2023, 08:34

valerio.bozzolan added inline comments.Jun 30 2023, 08:37

src/applications/auth/engine/PhabricatorAuthPasswordEngine.php
153	If I understand correctly this removes empty lines. In your opinion, where does an empty blocklist entry come from?

avivey added inline comments.Jun 30 2023, 08:52

src/applications/auth/engine/PhabricatorAuthPasswordEngine.php
153	Removes empty ("falsy", probably) entries. The blocklist is made up of a bunch of different sources - possibly some of them can only be null in debug mode; I wasn't able to determine which one was null in this case.

chris accepted this revision.Jun 30 2023, 11:33

This revision is now accepted and ready to land.Jun 30 2023, 11:33

Closed by commit rPd725ffaa7728: Fix "Register with Approval flow" for php 8. · Explain WhyJul 2 2023, 15:49

This revision was automatically updated to reflect the committed changes.

avivey added a commit: rPd725ffaa7728: Fix "Register with Approval flow" for php 8.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

auth/

controller/

PhabricatorAuthNeedsApprovalController.php

2 lines

O1: Blessed Committers

engine/

PhabricatorAuthPasswordEngine.php

2 lines

O1: Blessed Committers

infrastructure/

customfield/

standard/

PhabricatorStandardCustomField.php

2 lines

O1: Blessed Committers

Diff 1047

src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php

Loading...

src/applications/auth/engine/PhabricatorAuthPasswordEngine.php

Loading...

src/infrastructure/customfield/standard/PhabricatorStandardCustomField.php

Loading...