Hi,

When importing into a node which has location fields in it, the code checks to see if the location fields I'm importing meets certain criteria. In specific - it checks to see if I include longitude/latitude fields and if they are valid.

There's a bug in this check. The first check is if one of the fields is empty. It is being done by multiplying both and if the result is zero - then we are cool. There is a bug here - what if both fields are empty? Such is my case, yet the node import is complaining that both fields need to be populated, or empty (the latter is exactly my case!... :-) ) . See attached screenshot for the error screen.

I've fixed this by applying the attached patch which changes the lon/lat checks code in supported/location.inc, line ~90 (I hope the patch is alright. Its my first...).

CommentFileSizeAuthor
#1 node_import.png37.69 KBboaz_r
location_lon_lat_bug.patch2.1 KBboaz_r

Comments

boaz_r’s picture

StatusFileSize
new37.69 KB

attaching a screenshot - note the third bullet in the error message (the other bullets are not related to this bug).

Robrecht Jacques’s picture

Status: Needs review » Fixed

Fixed. Will be included in next release of node_import (5.x-1.5). Thanks!

Anonymous’s picture

Status: Fixed » Closed (fixed)

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