Hi, looking forward to trying this module more. With MySQL5 the install of the module fails and this error appears after enabling it. I looked into it and the "ads" table was not created. I assume that's somewhat critical ;)
user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 25 query: CREATE TABLE ads ( aid INT(10) UNSIGNED NOT NULL UNIQUE DEFAULT '0', uid INT(10) UNSIGNED NOT NULL DEFAULT '0', gid INT(10) UNSIGNED NOT NULL DEFAULT '0', adstatus VARCHAR(255) NOT NULL DEFAULT '', adtype VARCHAR(255) NOT NULL DEFAULT '', redirect VARCHAR(255) NOT NULL DEFAULT '', autoactivate INT UNSIGNED NOT NULL DEFAULT '0', autoactivated INT UNSIGNED NOT NULL DEFAULT '0', autoexpire INT UNSIGNED NOT NULL DEFAULT '0', autoexpired INT UNSIGNED NOT NULL DEFAULT '0', activated INT UNSIGNED NOT NULL DEFAULT '0', maxviews INT UNSIGNED NOT NULL DEFAULT '0', maxclicks INT UNSIGNED NOT NULL DEFAULT '0', expi in /var/www/html/includes/database.mysql.inc on line 121.

Comments

racakg’s picture

racakg CreditAttribution: racakg commented
Version: 4.7.x-1.0-beta7 » 5.x-1.x-dev

Same problem here with MySQL 5.0.27 and PHP 5.1.6.

teleted’s picture

teleted CreditAttribution: teleted commented

There's a missing closing parenthesis, line 43 of ad.install:

KEY (autoexpire)");
should be
KEY (autoexpire))");

Jeremy’s picture

Jeremy CreditAttribution: Jeremy commented
Status: Active » Fixed

Whoops, I got sloppy there when moving notifications into their own module. I checked the fix into both the 4.7.x and 5.x development branches, but won't be releasing a new beta until the notification stuff is working. Hopefully in the next couple of days.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)