From: Andrew T. <ajt...@hi...> - 2005-12-14 16:18:54
|
I made the lat/lon fields editable since many feeds don't include their location, but it is sometimes discernable. For example, my weatherunderground.com feed does not have a location information, but I *know* where it's located. Perhaps just adding a test that if there is no lat/lon in the feed, check the db for a 'stored' location. I'm already adding functionality in a local version for displaying all this location stuff on a map. Also, Kevin, what did you do with the RSS image stored in the DB when you refactored how feeds are stored? Andy On 12/14/05, Kevin <ke...@dr...> wrote: > Andrew Turner wrote: > > >Wow, that's a pretty big change. You don't forsee any other > >repurcussions of not storing this metadata in the database? I do see > >the benefits, in terms of not having to do updates of the feed, etc. > >Let the feed & cache be your storage mechanism. > > > >I did a quick test on my current DB and everything appeared to work > >well. I have yet to test an "upgrade" of a FoF v0.19 database. > > > >Also - what is the status of the RSS image Katie? Is this work > >complete, in progress for v0.2, or pushed back to v0.3? I wasn't sure > >if it changed. > >Thanks for the work. > >Andy > > > > > > > I haven't seen any problems with it either. Part of my testing was with > an empty cache directory. The feed list simply shows "version > unavailable" and "description unavailable" for feeds with no cache > file. Same as it shows "never" in the last update column. For > enriching items for view.php, I cache the rss info in memory for each > feed, so it's not having to read in the cache file for each > item.(although the performance wasn't that bad without the in-memory cach= e) > > As far as an upgrade from FoF, I don't see how it could be a problem. > FoF has a description column on the feed table, it simply won't be used > in FoFr and won't be populated for feeds added after the upgrade. The > column doesn't have a "not null" restriction, so that shouldn't be an iss= ue. > > I see one case where this could be a problem. On the feed edit page, > the lat and long are editable fields. Under my new design, these > attributes are read only and not able to be modified by the user. > > > The problem with feed image is localized to one particular feed where > they publish an image URL but do not allow foreign referrals. We could > either: 1) do nothing. 2) cache the feed image locally. #2 can be a > deferred to a future release, if we want to do it at all. > > --Kevin > > > ------------------------------------------------------- > This SF.net email is sponsored by: Splunk Inc. Do you grep through log fi= les > for problems? Stop! Download the new AJAX search engine that makes > searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! > http://ads.osdn.com/?ad_id=3D7637&alloc_id=3D16865&op=3Dclick > _______________________________________________ > Fofredux-devel mailing list > Fof...@li... > https://lists.sourceforge.net/lists/listinfo/fofredux-devel > -- Andrew Turner ajt...@hi... 42.4266N x 83.4931W http://highearthorbit.com Northville, Michigan, USA |