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

	Unknown Object (File)
	Tue, Jun 30, 3:04 PM

	Unknown Object (File)
	Tue, Jun 30, 2:31 PM

	Unknown Object (File)
	Jun 29 2026, 12:20 PM

	Unknown Object (File)
	Jun 28 2026, 1:08 PM

	Unknown Object (File)
	Jun 24 2026, 7:06 AM

	Unknown Object (File)
	Jun 19 2026, 7:45 PM

	Unknown Object (File)
	Jun 18 2026, 12:25 AM

	Unknown Object (File)
	Jun 4 2026, 2:01 PM

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