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

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

Okay, I'm giving up figuring out what's wrong with upgrading for the
moment and I'm going to work against a clean DB, but I don't think I'd
call it an RC until upgrade / backwards compatability issues are sorted
out.  If we want FOF users to check it out, I think it should be
backwards compatible so FOF users can check it out but switch back to
FOF if they want.

If no one else wants to work on the upgrade issues, I might have time to
work on it in the next week, but am going to integrate keyboard
shortcuts onto a branch today.

--Benjy

On Sun, Nov 27, 2005 at 03:13:31PM -0800, Benjamin Stewart wrote:
> 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