Page MenuHomePhorge

isSelfURI: fix anchors and relative URIs (that are "self" indeed)
ClosedPublic

Authored by valerio.bozzolan on Mar 9 2024, 22:30.
Tags
None
Referenced Files
F2166656: D25555.id.diff
Sat, Apr 27, 07:03
Unknown Object (File)
Mon, Apr 22, 22:33
Unknown Object (File)
Mon, Apr 22, 22:32
Unknown Object (File)
Mon, Apr 22, 19:09
Unknown Object (File)
Mon, Apr 22, 18:33
Unknown Object (File)
Wed, Apr 17, 18:26
Unknown Object (File)
Wed, Apr 17, 07:08
Unknown Object (File)
Wed, Apr 10, 01:28

Details

Summary

Example cases that were wrongly considered external URLs:

  • "#foo"
  • "/foo"

Note that Phorge usually does not change stable things. In fact this thing
was not documented even inline, and was not even in the unit tests.

So this is a fix, and not a breaking change. Probably no one had ever tried it.

Closes T15182

Test Plan

The already existing unit tests still work.

The added ones make sense.

There are no problems doing other random things.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable