The Ubercart product kit install file's schema has a bracket and comma in the wrong place, causing the 'synchronised' field not to be created.

CommentFileSizeAuthor
#7 synch_fix.patch5.53 KBIsland Usurper
#1 400720.patch466 bytesScheepers de Bruin

Comments

Scheepers de Bruin’s picture

StatusFileSize
new466 bytes
rszrama’s picture

Status: Active » Fixed

Ahh, thanks for posting the fix! Committed.

(Quick side note about patches - create your patches from the root ubercart directory instead of from the directory where the actual module sits. This will help testers, esp. if you ever get into patching files from multiple directories.)

Status: Fixed » Closed (fixed)

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

j0rd’s picture

Just ran into this myself. Glad to see it's fixed. Will apply patch.

hebhansen’s picture

Is it possible for someone to post the fixed file as is? Patch included. Would be a big help

tr’s picture

Version: 6.x-2.0-beta5 » 6.x-2.0-beta6
Status: Closed (fixed) » Active

This patch did not include a hook_update_N() to correct flawed beta5 installations. As a result, anyone who upgrades from beta5 to beta6 continues to receive this error. (A fresh install of beta6 works, however.)

The solution, for beta6 and above, is to disable and UNINSTALL the product_kit module, then re-enable it to rebuild the DB table with the correct schema.

At this point, I think an update function is still needed which checks the presence of the 'synchronized' column and creates it if not present. Otherwise, this error will continue to rear its ugly head when people upgrade from beta5 to *any* higher version. I suggest creating a function uc_product_kit_update_6003() which is simply a copy of uc_product_kit_update_6002().

Island Usurper’s picture

Assigned: Scheepers de Bruin » Island Usurper
Status: Active » Needs review
StatusFileSize
new5.53 KB

It never fails. Something always comes up on the day of a release. Hopefully this one won't last too long. In the meantime, let's bookmark this thread and point people to this patch.

rszrama’s picture

Status: Needs review » Fixed

This was "accidentally" committed with another patch. : P

Status: Fixed » Closed (fixed)

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