Hi

I've just installed this module and i found a misspell in the uc_uplog table creation

A comma is missing after this line

discount 			INT(4) NOT NULL DEFAULT '0' 

Here is the code fixed

     db_query("CREATE TABLE {uc_uplog} (
          uplog_id      INT     NOT NULL AUTO_INCREMENT,
          uid           INT(10) NOT NULL DEFAULT '0',
          oid						INT(10) NOT NULL DEFAULT '0',
          points        INT(10) NOT NULL DEFAULT '0',
				  referrer			tinyint(4) NOT NULL DEFAULT '0',
					discount 			INT(4) NOT NULL DEFAULT '0',
          PRIMARY KEY (uplog_id)
        ) /*!40100 DEFAULT CHARACTER SET utf8 */;");

Comments

korvus’s picture

I just wanted to confirm that I saw the same bug (SQL error when creating the tables on install), and adding that comma fixes it.

bmagistro’s picture

Assigned: Unassigned » bmagistro
Status: Active » Fixed

i thought i committed the fix after the first time...oops. i have reupdated the cvs and it should be good once the packaging script runs again.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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