- Issue created by @super_romeo
- Merge request !9Issue #3350345: TypeError: Cannot assign string to property Drupal\smart_ip\SmartIpLocation::$latitude β (Open) created by super_romeo
- Status changed to Needs review
over 1 year ago 12:11pm 19 June 2023 - π¬π§United Kingdom 3li U.K. π¬π§
Worked well for me, created a patch as well.
- Status changed to RTBC
over 1 year ago 1:48pm 9 August 2023 - last update
over 1 year ago 1 pass - π©πͺGermany Anybody Porta Westfalica
Just ran into the same issue. This happens if only calling
$location = \Drupal::service('smart_ip.smart_ip_location');
so can be easily reproduced. MR / patch fixes the issue! - πΊπΈUnited States jmouse888
Will this be merged soon? Trying to upgrade our D9 to D10 and would love to do it without any patch in place. Thank you for your time.
- Status changed to Needs review
12 months ago 5:25pm 19 December 2023 - last update
12 months ago 1 pass - π¬π§United Kingdom jofitz
I suggest this as a more elegant solution.
(no interdiff because it is a completely different approach)
- π¬π§United Kingdom gebiss
Yep, the patch 5 works on D10 with the setting using DB1: Country
Thanks
- πΊπΈUnited States capysara
#9 and #12 didn't fix the issue for me.
The merge request/patch #5 works, but it's described as "a workaround." Is there a better solution? - π³π¬Nigeria chike Nigeria
The MR worked for me. I didn't test any of the other patches.
- πΉπΌTaiwan peterwcm
My site encountered this error after upgrading from 8.x-4.2 to 5.0.0-alpha3
My setup uses Smart IP and Smart IP MaxMind GeoIP2 binary database modulesPatch #5 works for me
Drupal version: 9.5.11
PHP: 8.1.29 - Status changed to Needs work
21 days ago 10:26pm 27 November 2024