Page MenuHomePhorge

Unify type-checking for `setHref()` type methods
ClosedPublic

Authored by avivey on Jul 24 2023, 13:23.

Details

Summary

Adopt PhutilURI::checkHrefType() to unify type-check of some PHUI objects.

Ref T15316. Depends on D25356.

Test Plan

In production.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey requested review of this revision.Jul 24 2023, 13:23
avivey retitled this revision from Unify type-checking for `setHref()` type methods to (phorge) Unify type-checking for `setHref()` type methods.Jul 24 2023, 13:25
src/view/phui/PHUITagView.php
106
valerio.bozzolan retitled this revision from (phorge) Unify type-checking for `setHref()` type methods to Unify type-checking for `setHref()` type methods.Jul 26 2023, 06:58
valerio.bozzolan edited the summary of this revision. (Show Details)

+1 with just that inline suggestion

Thanks

This revision is now accepted and ready to land.Aug 2 2023, 16:34