The coder upgrade didn't upgrade everything, I still have to do this manually.

CommentFileSizeAuthor
#1 uninstall_db_update-1111318-2.patch2.65 KBckng

Comments

ckng’s picture

ckng
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new uninstall_db_update-1111318-2.patch2.65 KB

Fatal error: Call to undefined function db_fetch_object() in [snip]/sites/all/modules/contrib/nodeformsettings/commentformsettings/commentformsettings.install on line 26

Patch
- use st() instead of t()
- updated db api for uninstall

Note:
I believe this in both the *.install files is not needed

// TODO update_sql has been removed. Use the database API for any schema or data changes.
  array() /* update_sql("DELETE FROM {system} WHERE name like '%commentformsettings'") */;
lelizondo’s picture

lelizondo commented
Status: Needs review » Patch (to be ported)

Thanks.. I'll commit it.

askibinski’s picture

askibinski commented
Status: Patch (to be ported) » Reviewed & tested by the community

Patch works fine in latest 7.x-dev.

klidifia’s picture

klidifia commented
Assigned: Unassigned » klidifia
Issue summary: View changes
ram4nd’s picture

ram4nd commented
Assigned: klidifia » Unassigned
Status: Reviewed & tested by the community » Fixed

  • ram4nd committed 2fe36a6 on 7.x-2.x authored by ckng 
    Issue #1111318 by ckng: Uninstall needs to update to the new Database...

Status: Fixed » Closed (fixed)

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