Looks like there's still some dev code in the .module file. Line 13:

drupal_set_message("hi");

Comments

e2thex’s picture

e2thex commented

Thanks for spotting that, I removed it.

crookednumber’s picture

crookednumber commented

I found some more things -- mostly typos -- late last night, as I was nodding off. This module -- great, by the way -- is almost doing what I want it to do. I'm importing large feeds with geo:lat and geo:long data. It's grabbing the data ... but the long/lat data is getting jumbled across items. Trying to figure out if this is happening at the exhaustive_parser level. Needless to say, I'll keep you posted.

Anyway, I'm more than happy to shoot back minor fixes. Is an issue pointing to a line OK? Or do you prefer patches? And do you mind if I roll multiple typo fixes in a single patch?

e2thex’s picture

e2thex commented

Any of the three is fine,i f it is just cosmetic. If you have any functional change, I would appreciate a patch for that one functional change.
Thanks for digging in.

crookednumber’s picture

crookednumber commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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