Lat/Lng Widget fails to save when no float given

Reproduced on clear drupal 10 standart instalation.
Enviroment: php8.1 and php8.2
Drupal 10.0.9
Drush 11.5.1
drupal/geolocation 3.12.0

Steps to reproduce:
- Install standart Drupal profile
- Enable geolocation module
- Add geolocation field to Article
- Try to create new Article and fill Latitude and Longitude with random string
- Click to Save button
- You will see 500 error on page and in logs

The website encountered an unexpected error. Please try again later.
TypeError: deg2rad(): Argument #1 ($num) must be of type float, string given in deg2rad() 

- went to simplytest.me
- chose Geolocation 8.x-3.x-dev
- waited for ready
- added geolocation field to article
- went to create new article
- added random letters to lat / lng
- got a correct warning (see screenshot)

Unless you can provide a way to reproduce your issue, I really can't do anything.

I think this problem cannot be reproduced on simplytest.me because this test tool uses PHP 7.4. But this problem is reproduced on PHP 8.1 and PHP 8.2

Hi ChristianAdamski,

I also had the same problem on one of our sites using this module.

Drupal version: 10.1.5
PHP version: 8.1.11
Module version: 3.12.0

I got the error after entering a float with a comma as a separator instead of a full stop.

These values (Latitude : 29,9773, Longitude : 31,1325)

Gave the following error (see image):

Only there was a peculiarity, while creating my content the error did happen, but while editing I got the message (NaN) on the field after entering the values. (So after entering it correctly once, it doesn't seem to occur again.)

So validation did happen, but just not during initeal adding of the content?

The patch fixed the problem for me and the field still works. The only thing I am still somewhat unsure about is the message displayed.

In Belgium, this is a valid number (as we work with commas). So perhaps a conversion to full stop in the code or a mention under the field that a full stop is required would be useful, I think?

Just to be sure, I also tried your simplytest.me way, only with the difference that I used the latest release instead of the dev branch.

This is the test site status page.

Perhaps using the release version and not the dev branch caused the PHP version to be chosen correctly. Because in my instance, the site is PHP 8.1.

I fill in the fields with the test values listed above.

After which I am shown the following error on the page and the logs show the same error as on our production site.

I hope this is enough information to take a closer look at this ticket, so that this solution (possibly with a few more tweaks) can be included in to the module.

Greetings,

Fabio

I was facing similar issue with version 8.x-3.12 when I was adding any string in the Lat/Long fields as soon as focus was going out of field input, it was changing to NaN.
In code, parent set value was being called before the custom code execution. After changing the order of execution it started working for me. I have attached my patch below.

The last submitted patch, 14: geolocation-3257751-14.patch, failed testing. View results →

Just to provide more context on above issue, I am only able to reproduce this issue when I use Geolocation Google Maps API - Geocoding and Map in Form display of this Geolocation field.

Created constraint validation on issue https://www.drupal.org/project/geolocation/issues/3236557 📌 Add long / lat range & type validation helper Needs review

This is not reproducible on simplytest.me in v3, not v4.