backdrop-contrib / smart_ip

Backdrop module to identify visitor's geographical location (longitude/latitude), country, region, city and postal code based on the IP address of the visitor
GNU General Public License v2.0
0 stars 1 forks source link

Module does not work on php 5.3 #5

Open jenlampton opened 5 years ago

jenlampton commented 5 years ago
( ! ) Fatal error: Can't use method return value in write context in backdrop/modules/contrib/smart_ip/smart_ip.module on line 282
jenlampton commented 5 years ago

Hm, I just fixed the smart_ip module to work on PHP 5.3, but now I can see in the status report: 5.3.29 less Your PHP installation is too old. Smart IP requires at least PHP 5.4.

Is this a limitation of the ip2location php library? If so, we should adjust the module (and the documentation) so that it will work (and not throw a status report error) for all the geolocation options where it does work.

We should also update the documentation to reference the Backdrop modules as requirements, and no libraries on their own.