Page MenuHomeDevCentral

Refactor URL encode/decode
Needs ReviewPublic

Authored by dereckson on Apr 18 2026, 13:00.
Tags
None
Referenced Files
F57136636: D4077.diff
Sun, Nov 15, 05:49
F57129874: D4077.id.diff
Sun, Nov 15, 03:28
F57127562: D4077.id11039.diff
Sun, Nov 15, 02:41
F57114650: D4077.id10640.diff
Sat, Nov 14, 21:58
F57114501: D4077.id11038.diff
Sat, Nov 14, 21:50
F57113708: D4077.id11039.diff
Sat, Nov 14, 21:11
F57098843: D4077.id10640.diff
Sat, Nov 14, 14:36
F57095570: D4077.id.diff
Sat, Nov 14, 12:28
Subscribers

Details

Reviewers
Duranzed

Diff Detail

Repository
rKERUALD Keruald libraries development repository
Lint
Lint Passed
Unit
Tests Passed
Branch
follow-up-for-doc-iii
Build Status
Buildable 6712
Build 7000: arc lint + arc unit

Event Timeline

dereckson created this revision.
Duranzed requested changes to this revision.May 18 2026, 23:15
Duranzed added a subscriber: Duranzed.

Harbormaster reports unit test failures on omnitools/tests/HTTP/URLTest.php, also the exception message changed in beautifyQuery() from "Unexpected encoding value" to "Unexpected value", while normalizeQuery() keeps "Unexpected encoding value".

This revision now requires changes to proceed.May 18 2026, 23:15

Restore full exception message

Move constantes to a backed enum, so we can get autocomplete from IDEs
and exhaustive matching.

_toString() -> (string) cast

Exception can be removed using an PHP 8.1 enum instead of in-class constants.