Found a minor bug:

If the system module Database logging is not enabled, update 7005 will fail, not allowing the remaining updates to complete. The simple fix is to enable Database logging. Might consider adding a check to see if that module is enabled first when running 7005 because it won't need to remove anything from the watchdog table if the table doesn't even exist to remove anything from.

I realize that the majority will have Database logging enabled but sometimes it gets overlooked (as in my case)...

Files: 
CommentFileSizeAuthor
#2 1871852_ensure_table_exists_before_deleting.patch551 bytesgreggles
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]

Comments

More specifically the check should probably see if the "watchdog" table even exists and if not, skip the 7005 update. As it is, it just errors out saying that it can't find the watchdog table because it doesn't exist and none of the remaining updates seemed to run.

Title:Database Upgrade 7005 ErrorCheck if database logging is enabled before running database upgrade 7005
Status:Active» Needs review
StatusFileSize
new551 bytes
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]

Making the title more explicit with what needs to happen.

#2 Patch worked perfect for me.

Status:Needs review» Reviewed & tested by the community

Gah, thanks greggles.

Once #1798294: Can't edit non-current node revisions is resolved I'll commit this.

Status:Reviewed & tested by the community» Fixed

Status:Fixed» Closed (fixed)

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