After CMS and PHP upgrades error in saving a record - _request_sanitize

I went to update the Geocoder plugin to the latest version.  In doing that and uploading the new version, I didn't enter my Google Maps API.

When I have an empty value in the Google maps API here:

$GLOBALS['GEOCODER_GOOGLE_API_KEY']         = '';

I can save the record.

Obviously, we use this field to automatically add the lat/long of a property.

Once I add my API key, that's when I start getting that error.

Also just want to say that my API key is valid, as I used the built in test.

So something in that function in CMS Builder is not liking the format of my key.

Hi LM, 

Thanks for reporting this.  Can you try this fix, open /lib/common.php and search for "function _request_sanitize".

Replace this

function _request_sanitize(array|string|null $input): array|string|null {

With this (adding "|float")

function _request_sanitize(array|string|float|null $input): array|string|null {

Let me know if that works for you.

Thanks!

Hi Dave,

When replace that and use our API key, I get this error:

str_contains(): Argument #1 ($haystack) must be of type string, float given

The raw log:

Array
(
    [logType] => exception
    [errno] => TypeError
    [errstr] => str_contains(): Argument #1 ($haystack) must be of type string, float given
    [errfile] => /home2/user/public_html/cmsAdmin/lib/common.php
    [errline] => 1011
    [exceptionCode] => 0
    [exceptionClass] => TypeError
    [exceptionTrace] => 
#0 /home2/user/public_html/cmsAdmin/lib/common.php(1011): str_contains(33.9919159, '\x00')
#1 /home2/user/public_html/cmsAdmin/lib/common.php(993): _request_sanitize(33.9919159)
#2 /home2/user/public_html/cmsAdmin/lib/Fields/BaseField.php(74): request('latitude')
#3 /home2/user/public_html/cmsAdmin/lib/common.php(495): Itools\CMSB\Fields\BaseField->getRequestValue(false)
#4 /home2/user/public_html/cmsAdmin/lib/menus/default/save.php(15): _getRecordValuesFromFormInput()
#5 /home2/user/public_html/cmsAdmin/lib/menus/default/actionHandler.php(68): include('/home2/user...')
#6 /home2/user/public_html/cmsAdmin/admin.php(59): require_once('/home2/user...')
#7 {main}
    [prevExceptionTrace] => 

    [externalFilePath] => /home2/user/public_html/cmsAdmin/lib/common.php
    [externalLineNum] => 1011
)

I can privately send you the API key if you want (if that matters).

There is something about this specific combination of characters.

Hi LM, 

Can you send in a second level support request?  https://www.interactivetools.com/support/request/

We'll take a look at this ASAP and get you a fix.

Second level support request submitted.

Let me know if that's all you need for now.

Dave,

Couple of notes for your testing:

- The Google Maps API key is saved in the Geolocator.php file as a commented line.

- You can test this functionality using the Property Listings menu item in the CMS.  I would just edit one of the current Property Listings entries and add a "2" onto the end of the name and try to save.  This is where the error occurs with the API key active.

Hi LM, 

I'll post here for anyone else with the same issue.  As discussed over email, GeoCoder needed to be upgraded to work with the latest CMS and the more strict variable typing rules we've been implementing for PHP 8.

I've uploaded the latest version to your server and made it available here as well: 
https://www.interactivetools.com/plugins/geocoder/

Thanks again for reporting this and helping us resolve it.