nova-map icon indicating copy to clipboard operation
nova-map copied to clipboard

Replace grimzy/laravel-mysql-spatial by matanyadaev/laravel-eloquent-spatial

Open vesper8 opened this issue 2 years ago • 4 comments

It's become pretty clear that grimzy/laravel-mysql-spatial is abandoned and there's been a migration to matanyadaev/laravel-eloquent-spatial which is actively maintained, does everything that grimzy/laravel-mysql-spatial did and is future proof (supports Laravel 9 and PHP 8.1.

I personally just migrated a rather large project from grimzy/laravel-mysql-spatial to matanyadaev/laravel-eloquent-spatial and am happy to report the migration was very easy and quite painless, taking me only about 1 hour including testing that everything still works.

My only dependence on grimzy/laravel-mysql-spatial now is your package here, which I find very useful and am glad for.

On that note, it would be truly fantastic if you also migrated to matanyadaev/laravel-eloquent-spatial : )

I think you will find it quite easy and simple to do so.

Many thanks for the consideration!

vesper8 avatar Mar 02 '22 17:03 vesper8

Same situation here.

pelmered avatar Apr 04 '22 10:04 pelmered

Thanks for the suggestion.

I am waiting for Nova 4 to launch tomorrow before making any improvements. This package may receive an update shortly.

If you want to PR a update I would be happy to include the change. The ability to remove grimzy would be nice.

D

davidpiesse avatar Apr 04 '22 10:04 davidpiesse

Although it's a required dependency, I don't think the grimzy/laravel-mysql-spatial package is actually used in the current package version? The only reference I can find is a function that is commented out in src/Map.php

image

sutcliffe121 avatar Jun 03 '22 08:06 sutcliffe121

Could you submit a PR with the package removed? It does look like it was only used for a planned editing functionality, which was not yet implemented.

davidpiesse avatar Jun 03 '22 08:06 davidpiesse