Closed (fixed)
Project:
Ubercart
Version:
6.x-2.0-beta6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
13 Mar 2009 at 10:31 UTC
Updated:
2 May 2009 at 06:20 UTC
Jump to comment: Most recent file
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.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | synch_fix.patch | 5.53 KB | Island Usurper |
| #1 | 400720.patch | 466 bytes | Scheepers de Bruin |
Comments
Comment #1
Scheepers de Bruin commentedComment #2
rszrama commentedAhh, 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.)
Comment #4
j0rd commentedJust ran into this myself. Glad to see it's fixed. Will apply patch.
Comment #5
hebhansen commentedIs it possible for someone to post the fixed file as is? Patch included. Would be a big help
Comment #6
tr commentedThis 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().
Comment #7
Island Usurper commentedIt 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.
Comment #8
rszrama commentedThis was "accidentally" committed with another patch. : P