Skip to content
This repository was archived by the owner on Oct 1, 2024. It is now read-only.

Conversation

@ajthinking
Copy link
Contributor

Return type of Point jsonSerialize needs updating because:

PHP Deprecated: Return type of MStaack\LaravelPostgis\Geometries\Point::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /Users/anders/Code/propertysearch/vendor/mstaack/laravel-postgis/src/Geometries/Point.php on line 103

Fixes #162

Let me know if this is good I can do the other types as well.

@shayneoneill
Copy link

Whats the status on this? Doesnt seem like its fixed.

@ajthinking
Copy link
Contributor Author

Hi @shayneoneill, this was fixed in fork: https://github.com/ajthinking/laravel-postgis (v6.0.4)

See #163

@joelmellon
Copy link

joelmellon commented Mar 26, 2022

@ajthinking Did you close this because you fixed this in your repo?

@ajthinking
Copy link
Contributor Author

I think I closed it by accident. If you want feel free to reopen it.
But since #163 was merged prematurely without needed dependency upgrades I think this repo is now totally broken

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecated notice on PHP 8.1.1

3 participants