Fix a PHP 8.1 string strictness issue in repositories
ClosedPublic
Actions

Authored by epriestley on Apr 25 2022, 7:28 PM.

Tags

None

Referenced Files

	F15392209: D21775.id51912.diff
	Sat, Mar 15, 1:48 PM

	F15392206: D21775.id51911.diff
	Sat, Mar 15, 1:48 PM

	F15392205: D21775.id.diff
	Sat, Mar 15, 1:48 PM

	F15377680: D21775.diff
	Thu, Mar 13, 10:11 AM

	F15368893: D21775.id.diff
	Wed, Mar 12, 4:26 AM

	F15322463: D21775.diff
	Thu, Mar 6, 11:46 PM

	Unknown Object (File)
	Sat, Feb 22, 9:10 AM

	Unknown Object (File)
	Sat, Feb 22, 12:27 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13588: PHP 8 Compatibility
Commits: rPadcc063242e5: Fix a PHP 8.1 string strictness issue in repositories

Summary

Ref T13588. This is a little too strict, as $line may be an integer.

Test Plan

Will deploy

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley requested review of this revision.Apr 25 2022, 7:28 PM

epriestley created this revision.

Harbormaster completed remote builds in B25668: Diff 51911.Apr 25 2022, 7:28 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 25 2022, 7:30 PM

Closed by commit rPadcc063242e5: Fix a PHP 8.1 string strictness issue in repositories (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPadcc063242e5: Fix a PHP 8.1 string strictness issue in repositories.

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

storage/

PhabricatorRepository.php

1 line

Diff 51912

View Options

src/applications/repository/storage/PhabricatorRepository.php

Fix a PHP 8.1 string strictness issue in repositoriesClosedPublicActions