Re: [Fofredux-devel] not picking up geo tags

[Kevin <ke...@dr...>]

>
>> Could some of you try adding this feed:
>>   http://feeds.feedburner.com/usgs/eqs1day-M25.xml
>> then look in your database to see whether the latitude and longitude
>> columns in fr_items are populated?
>>
>> For me, they aren't, and I'm trying to figure out why not.  Andrew
>> reports that his installation is getting the latitude and longitude,
>> making me suspect a PHP version issue; I'm running PHP Version
>> 4.4.2-1+b1.
>>
>> I tried adding some print statements in init.php, but once the logic
>> gets to the model in lib/model/, I lose track of the flow of
>> control.
>
> This doesn't work for me either.  php 5.0.5
>
> There are really only 3 places that touch lat/long data for an item:
>
> * The getLatitude() and getLongitude() methods of FOF_RssItem.  These a=
re
> responsible for extracting the geo values from the magpierss item hash
> structure.
> * The FOF_ItemModel::save() method.  This pulls the fields out of the
> FOF_Item object and inserts it in the db.
> * fof_add_item() in init.php.  This is the glue that ties the above two
> pieces together.
>
> I suspect the first location.  Just because it has the most code around
> it.    Let me know if you need more help.


My suspicion was right.  The getLatitude() and getLongitude() method were
left broken by *me*.  I've corrected it and committed.  Sorry.


Is the model code really that hard to follow?  It wasn't my intention,
believe me.  If there's anything that should be changed to make it better=
,
please let me know and we'll work it out.

--=20
Kevin