There is no browscap_update_6000() function to handle upgrading from D5 to D6.

This results in lots of unknown column serialized errors.

Here is a patch to add this update function.

CommentFileSizeAuthor
#2 browscap_update_6000_2.patch724 bytesAgileware
browscap_update_6000.patch657 bytesAgileware
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented

I believe that if someone installed a 6.x Drupal site and then installed Browscap for that and then we add this code to the 6.x branch it will try to execute and will cause an error.

I think we should do something to test for the existince of that column before trying to create it.

Agileware’s picture

Agileware CreditAttribution: Agileware commented
FileSize
browscap_update_6000_2.patch724 bytes

Oops. Here is one that does the check.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, thanks! I'm inclined to commit this since it can't really break anything and will be useful to others.

I'd like to leave it in the queue for a bit in case Rob or Mike want to provide feedback.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Reviewed & tested by the community » Fixed

Now fixed http://drupal.org/cvs?commit=415010

Thanks!

Status: Fixed » Closed (fixed)

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