Closed (fixed)
Project:
Mollom
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Dec 2010 at 22:05 UTC
Updated:
24 Apr 2014 at 17:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sunThanks for reporting! Attached patch should resolve the problem.
Comment #2
geerlingguy commentedFixed the upgrade to D7 for me... now getting an error on #7008 :-/
Update #7008
Failed: DatabaseSchemaObjectExistsException: Cannot add index <em class="placeholder">session_id</em> to table <em class="placeholder">mollom</em>: index already exists. in DatabaseSchema_mysql->addIndex() (line 433 of ~/public_html/includes/database/mysql/schema.inc).Comment #3
sunThanks for testing! Attached patch should fix that remaining failure. :)
Overall, sorry, the upgrade path to D7 hasn't been tested yet, and it's especially complex, because most of the new features in D7 have been backported to D6 already, so the module updates need to cater for both D7->D7 as well as D6->D7 update paths. :-/
However, I'm relatively sure that we should be done now with the attached patch.
Comment #4
geerlingguy commentedPerfect - all updates applied perfectly.
Comment #5
sunThanks for reporting, reviewing, and testing! Committed to HEAD.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.