Closed (fixed)
Project:
Ubercart Discounts (Alternative)
Version:
6.x-1.0-beta14
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
20 Apr 2009 at 18:19 UTC
Updated:
5 May 2009 at 20:00 UTC
it looks like this might be a quick fix. upgrade to beta 14 generates this error:
The following queries were executed
uc_discounts module
Update #2
* ALTER TABLE {uc_discounts} ADD has_qualifying_amount_max tinyint NOT NULL default 0 AFTER qualifying_amount
* ALTER TABLE {uc_discounts} ADD qualifying_amount_max float NOT NULL default 0 AFTER has_qualifying_amount_max
* Failed: ALTER TABLE uc_discounts MODIFY has_role_filter tinyint NOT NULL
The fix is in 'uc_discounts.install': lines 423 - 425
<?php
//Fixes update bug that existed in uc_discounts_update_1 (but is fixed now)
// the following line should have '{' and '}' brackets for those like me who use a drupal prefix
// $queries[] = update_sql("ALTER TABLE uc_discounts MODIFY has_role_filter tinyint NOT NULL");
$queries[] = update_sql("ALTER TABLE {uc_discounts} MODIFY has_role_filter tinyint NOT NULL");
break;
?>
Comments
Comment #1
ryangroe commentedOh, that was sloppy of me. Thanks gtfourdreams. I'll release a fixed version tonight.
Comment #2
ryangroe commentedThis has been fixed in CVS and the fix will be part of beta 15 to be released tonight.
Comment #3
gtfourdreams commented