http://drupal.org/update/modules/6/7#install-schema

CommentFileSizeAuthor
#1 761790_schema_auto_handling.patch1.02 KBgreggles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Active » Needs review
FileSize
761790_schema_auto_handling.patch1.02 KB

Status: Needs review » Needs work

The last submitted patch, 761790_schema_auto_handling.patch, failed testing.

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
CreditAttribution: vegantriathlete commented

Before applying the patch I was getting the error message that the table already exists when I attempted to enable comment_notify for the first time.

Also, the uninstall option did not work. So, I had to manually drop the tables.

The patch applied successfully and fixed the problem -- I was able to enable comment_notify.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Assigned: Unassigned » greggles
Status: Needs work » Fixed

Thanks for the review - now fixed: http://drupal.org/cvs?commit=350104

Status: Fixed » Closed (fixed)

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