7300 - Remove dynamically generated mailsystem classes and convert configuration.

Failed: PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '))' at line 1: DELETE FROM {registry} WHERE (name IN ()) ; Array ( ) in mailsystem_update_7300() (line 100 of /sites/all/modules/mailsystem/mailsystem.install).


joelpittet’s picture

Title:Upgrade to 7300 syntax errror» Upgrade to 7300 syntax error
mrfelton’s picture

Status:Active» Needs review
new651 bytes

This is untested, any maybe there is a reason that $genclasses shouldn;t be empty and so something else actually needs fixing. But this patch should fix the error at hand and allow the update to run smoothly.

mrfelton’s picture

Or rather,

joelpittet’s picture

Yeah that is not a very nice patch... if the condition is empty it erases the registry... :(

mrfelton’s picture

Yes, good point. That will probably explain my registry kept in an aegir migration. I did say it was untested! Ok, here is a better version that wont do that.

ShaneOnABike’s picture

Status:Needs review» Reviewed & tested by the community

I tested this in Aegir and running drush update I no longer had any problems. Thanks for providing the patch upgrading to the latest mailsystem has now resolved another issue with it not finding my tpl files properly yahoo!

pjcdawkins’s picture

The patch in #5 works for me too.

juahoo’s picture

#5 works for me, thanks.

bytecanarias’s picture

Issue summary:View changes

La opción #5 de mrfelton me funciono.

The option #5 mrfelton is good.

  • Les Lim committed 8463cf5 on 7.x-3.x
    Issue #1690078 by mrfelton, fearlsgroove, Les Lim | joelpittet: Fixed...
Les Lim’s picture

Status:Reviewed & tested by the community» Fixed
new996 bytes

Committed the attached patch to 3.x. Thanks, everyone!

joelpittet’s picture

Thanks @Les Lim and @mrfelton!

Status:Fixed» Closed (fixed)

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