feat(metadata): use uri RFC 3986 on PHP8.5

[SVillette]

Q	A
Branch?	main
Tickets	-
License	MIT
Doc PR	api-platform/docs#...

This PR replaces the usage of parse_url() method by the new rfc 3986 URI API on PHP8.5.
cc @soyuka this is the subject we talk about at the SymfonyCon hackday.

Still in draft as it's missing some tests.

[soyuka]

could you maybe rebase this?

[SVillette]

@soyuka PR is rebased.
However these changes break a test in ApiPlatform\Metadata\Tests\Util\IriHelperTest as http:/// is considered valid with the new Uri\Rfc3986\Uri class. WDYT about it?

[VincentLanglet]

I feel like this work could be simpler by exposing the require method in a BCHelper class or by using a polyfill like https://github.com/thephpleague/uri-polyfill

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[alexislefebvre]

Suggested change

	public function testHelpers(): void
	#[RequiresPhp('>= 8.5.0')]
	public function testHelpers(): void

and the import:

use PHPUnit\Framework\Attributes\RequiresPhp;

This is supported by PHPUnit: https://docs.phpunit.de/en/12.5/writing-tests-for-phpunit.html#skipping-tests-using-attributes

Then the check on PHP_VERSION_ID can be removed.

---

Note: updated with full version to avoid an issue: symfony/symfony#63987

[alexislefebvre]

Suggested change

	public function testHelpersWithNetworkPathAndRFC3986(): void
	#[RequiresPhp('< 8.5.0')]
	public function testHelpersWithNetworkPathAndRFC3986(): void

[alexislefebvre]

I feel like this work could be simpler by exposing the require method in a BCHelper class or by using a polyfill like https://github.com/thephpleague/uri-polyfill

I like these ideas.

@SVillette Would you like to update your PR with this idea?

I’m not sure if adding a dependency for this is worth it. WDYT @soyuka ?