Closed (fixed)
Project:
Ad Flash
Version:
6.x-2.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
1 Jun 2010 at 20:50 UTC
Updated:
4 Oct 2010 at 21:20 UTC
Jump to comment: Most recent file
There was an error when running the update.php after installing the 2.6 update from 2.4. Here are the errors:
* warning: Invalid argument supplied for foreach() in /usr/local/apache2/vhosts/mysite.com/htdocs/includes/database.mysql-common.inc on line 109.
* 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 1 query: ALTER TABLE ad_flash ADD PRIMARY KEY () in /usr/local/apache2/vhosts/mysite.com/htdocs/includes/database.mysql-common.inc on line 374.
* warning: Invalid argument supplied for foreach() in /usr/local/apache2/vhosts/mysite.com/htdocs/includes/database.mysql-common.inc on line 109.
* 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 1 query: ALTER TABLE ad_flash ADD INDEX aid () in /usr/local/apache2/vhosts/mysite.com/htdocs/includes/database.mysql-common.inc on line 434.
The following queries were executed:
ad_flash module
Update #6000
* ALTER TABLE {ad_flash} DROP KEY aid
* Failed: ALTER TABLE {ad_flash} ADD PRIMARY KEY ()
* Failed: ALTER TABLE {ad_flash} ADD INDEX aid ()
Ads seem to still be displaying and functioning correctly afterward. Are these errors to be worried about? Why did they occur?
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 815326.patch | 565 bytes | svendecabooter |
Comments
Comment #1
svendecabooterI have the same problem.
Attached is a patch that fixes it.
Comment #2
gregarios commentedVerified. The patch in the previous comment by svendecabooter fixes the issue completely.
Comment #3
Alex Andrascu commentedwill do
Comment #4
ball.in.th commentedi just ran into the same problem while upgrading from 2.4 to 2.6 .
Comment #5
danyg commenteduse the patch above, or manually type the SQL-query into phpmyadmin or console.
Btw "Primary key" and "Index" use the same field, I think one of the is unnecessary.
Comment #6
tonku commentedI think the created index should be fid instead of aid (like in the module's clean install database). So should the batch be updated?
Comment #7
Alex Andrascu commentedSorry for not beeing arround for awhile now.
This patch is now ported in the dev version. Please review and comment.
@jollydiver - If you are so kind to commit a patch against the head i'll happy commit it.
Thanks svendecabooter for this patch
Comment #8
Alex Andrascu commented