A fully-updated D6 version has already created this table, so this update can probably be removed.

If for some reason it can not be removed, it should first check that the table does not exist.

CommentFileSizeAuthor
#1 honeypot-users-table-update-1792396-1.patch1.91 KBgeerlingguy

Comments

geerlingguy’s picture

geerlingguy commented
Status: Active » Needs review
StatusFileSize
new honeypot-users-table-update-1792396-1.patch1.91 KB

How does this look? The reason for that update is the later addition of the table in the 7.x-1.13 version of Honeypot; existing (pre-1.13) users of the D7 version don't have the table already, so this update is necessary to make sure it's created for them.

drumm’s picture

drumm
he/him
Location NY, US
commented

This looks okay.

geerlingguy’s picture

geerlingguy commented
Status: Needs review » Reviewed & tested by the community

I'll commit when I get a chance.

drumm’s picture

drumm
he/him
Location NY, US
commented

Sorry to bug you about this, but it would be ideal to get this in the next few hours so we can get it to our codebase and likely get our nightly upgrades working again. We always like running un-hacked modules.

geerlingguy’s picture

geerlingguy commented

Sure thing! I'll push up the patch now.

geerlingguy’s picture

geerlingguy commented
Status: Reviewed & tested by the community » Fixed

http://drupalcode.org/project/honeypot.git/commit/7ca23da

Status: Fixed » Closed (fixed)

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