Make app compatible with NC32

[rakekniven]

Actually the app is not listed as ready for NC32

[tflidd]

And I stumbled on this error when using it with NC 32:
#1469

[isdnfan]

I hit this issue while sync calendar shared from one user to another. SSO login with OIDC (and many other actions) failed with "Undefined constant OCP\\Share::SHARE_TYPE_USER in file '/var/www/html/custom_apps/maps/lib/Hooks/FileHooks.php' line 154" - solved after disabling maps app

[ant0nwax]

Thank you @isdnfan one more reason for me, not to upgrade to NC32 yet

[ricokulzk-dotcom]

Wir nutzen Nextcloud-AIO im Docker und hier wird zwar schon NC 32.0.2 angezeigt, wegen inkompatible Software ( Maps) geht das wohl aber noch nicht..., Bitte aktualisieren auf die Version 32. Habe auch schon versucht per Email kontakt aufzunehmen.

[rakekniven]

Please give a thumbs up instead of "me too" or "+1" comments. Issues can be sorted by upvotes.

[hayvan96]

Hi, the PR aiming to close this issue is ready for a month now -- is something holding back its being merged, and if so, how can we help? Thank you for your hard work

[tflidd]

is something holding back its being merged, and if so, how can we help?

Yes, pull requests need to be tested and reviewed, you cannot just merge them without someone approving it.

[RainerEmrich]

One of the issues is the lack of compatibility with php8.4.

[Szwendacz99]

One of the issues is the lack of compatibility with php8.4.

Maps are not compatible with php8.4? Weird since for quite some time now I have php8.4 on my nextcloud which is on version 31, and maps work there very well.

[RainerEmrich]

One of the issues is the lack of compatibility with php8.4.

Maps are not compatible with php8.4? Weird since for quite some time now I have php8.4 on my nextcloud which is on version 31, and maps work there very well.

I have a NC32 instance runinng with the recommended php8.4. If you try to enable maps, you get a failure saying that it needs php version lower than 8.3!