[Map] Make UX-Map compatible with PHP 8.1 or clarify PHP 8.3 requirement

[JulienGuyenet]

Hi Symfony team,

First, thank you for your amazing work on the UX components!

I'm interested in using ux-map in my project, but I noticed that it requires PHP 8.3. Unfortunately, my application needs to run on a server with PHP 8.1, so I'm trying to understand what makes PHP 8.3 mandatory.

From what I’ve seen, the only feature requiring PHP 8.3 is the use of readonly classes. I tried removing the readonly keyword from the relevant classes in both ux-map and ux-leaflet-map, and everything still seems to work fine—the tests pass and the maps render correctly.

Could you confirm if there's any other reason PHP 8.3 is required? Would you consider relaxing the PHP version constraint, or accepting a PR to make it compatible with earlier versions like PHP 8.1?

Thanks in advance!

[Kocal]

Hi,

Well, that's true, I don't think PHP 8.3 is mandatory for UX Map and its Bridges, except for readonly classes we do not use exclusive features.
IIRC when working on UX Map, we were like "oh, let's use the latest PHP version!", but indeed it has no real benefits.

I would be in favor of:

1. downgrade PHP version to 8.1
2. Wait Symfony UX 3 to update PHP requirement

[JulienGuyenet]

Hi, and thanks again for your reply!

That approach sounds great. If the plan is to revisit the PHP requirement with Symfony UX 3, I’m totally on board with waiting for that version instead of making changes at this stage.

I’ll keep an eye on the next UX release, and I’d be happy to help with the downgrade when the time comes if needed.

Thanks again for your work and support—really looking forward to what’s coming next with Symfony UX!

[Kocal]

I've just merged #2695, it will be available in 2.25 :)