[Fofredux-devel] Re: unknown column feed.latitude (upgrade issues)

[Benjamin S. <bs...@cr...>]

So the next problems I was having were because my category and
categoryXfeed tables had not been created.  This may have been because I
didn't add the table defitions to my config.php before running install,
but I have not followed up on this possibility.

Regardless, we should handle the case of the idiot who does not read
documnetion when trying to upgrade.

Any insights as to what the problem might have been before I back up and
dig into that?

--Benjy

On Sun, Nov 27, 2005 at 03:06:15PM -0800, Benjamin Stewart wrote:
> So I just checked out off the head and copied my config.php over to try
> to use my existing FOF database;  It correctly asks "cache dir not found
> / writable whatever have you run install?" so I ran install, and hit the
> panel, and got this error:
> 
> [select feed.id, feed.url, feed.title, feed.link,
> feed.description,feed.latitude,feed.longitude, feed.category_id,
> cat.name from px_feeds feed, FOF_CATEGORY_TABLE cat WHERE cat.id =
> feed.category_id order by feed.title]: 0.0022
> Cannot query database. Have you run install.php? MySQL says: Unknown
> column 'feed.latitude' in 'field list'
> 
> The problem is fairly obvious;  I guess we just need to make sure
> upgrades from FOF work seemlessly, and ideally are backwards compatible
> for now?   (This mainly relies on stevem not having any select *'s or
> insert into table select * from other table in his code, but we should
> also be good and add new columns to the end to give any select *s a
> *chance* of working)
> 
> So uh, i tried altering my tables with the below sql, and I'm still
> getting some errors.  I'll follow up when I isolate exactly what else
> will be need for an upgrade.
> 
> (must be run in order at the moment:)
> 
> alter table px_feeds add column category_id int(11) default NULL;
> alter table px_feeds add column latitude float default '0.0';
> alter table px_feeds add column longitude float default '0.0';
> alter table px_items add column flag tinyint(4) default NULL 
> alter table px_items add column latitude float default '0.0';  
> alter table px_items add column longitude float default '0.0';
> 
> (Oh, this does bring up the question of whether or not we want to change
> the name of the 'read' column at some point, so we can stop confusing
> mysql)
> 
> --Benjy