Closed (fixed)
Project:
GeoNames
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 May 2011 at 16:08 UTC
Updated:
3 Jun 2011 at 01:51 UTC
There is 4 references to split which needs replacing with preg_split - http://php.net/manual/en/function.split.php
geonames.module:67: $object = array_combine($field_names, split("\t", $line));
geonames.module:90: $lines = split("\n", $file->data);
geonames.module:93: $a = split("\t", $line);
geonames.module:94: $b = split("\.", $a[0]);
Comments
Comment #1
7wonders commentedexplode is a better option than preg_split I think.
Comment #2
7wonders commentedSadly I havent figured out creating patches yet but here is the required changes to get the latest d7 version working which includes adding the check for geonameid not equalling 0 and use of explode instead of split.
Comment #3
lyricnz commentedReplaced split() with explode() in D6 and D7 branches. Thanks.