Great module.

My installation has table_prefixes enabled. All my tables start with D5_... to avoid overwrites.
Database is installed (using modified ip_update.php).
IP Locator block output is correctly on 'view'.

When using the IP Locator block field to check a specific IP, the MySQL statement produces the following error, however:

user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'D5_join ip_locator_country as c on (g.country_code = c.country_code) ' at line 10 query: SELECT g.date, c.country, g.country_code, g.registry, c.latitude, c.longitude FROM D5_ip_locator_geo_ips as g left outer D5_join ip_locator_country as c on (g.country_code = c.country_code) WHERE 3562573844 >= g.ip_from and 3562573844 <= g.ip_to ORDER BY g.date desc, g.registry, g.country_code LIMIT 0, 1 in /...yaddayadda.../includes/database.mysql.inc on line 172.

Notice the "D5_join ip_locator_country as [...]".

The table prefix seems to bee concatenated to the "join" instead of the "ip_locator_country" table name. Other queries use the table prefix correctly, as it seems.

Comments

daniel_h’s picture

daniel_h commented

Also, I would like to add:
ip_update.php worked like a charm. Good code.
I downloaded all ip ranges from the apnic mirror ftp. The other four registrars were available as "delegated_" obviously.
Oh, and though I added more MySQL error feedback, this was not needed. Importing a 10+Mb database via webinterface was more challenging (MySQLdumper to the rescue).

he_who_shall_not_be_named’s picture

he_who_shall_not_be_named commented

Fixed in 5x-5.0. Thanks.

he_who_shall_not_be_named’s picture

he_who_shall_not_be_named commented
Status: Active » Fixed
daniel_h’s picture

daniel_h commented

Lightning fast :)

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

Project: » Lost & found issues

This issue’s project has disappeared. Most likely, it was a sandbox project, which can be deleted by its maintainer. See the Lost & found issues project page for more details. (The missing project ID was 179598)