Needs work
Project:
Mail System
Version:
7.x-2.x-dev
Component:
Module compatibility
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2011 at 23:39 UTC
Updated:
21 Jul 2015 at 18:33 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
pillarsdotnet commentedWorking on this -- will report back soon.
Comment #2
SolomonGifford commentedany update?
Comment #3
trajedy commentedsubscribe
Comment #4
trajedy commentedprogress report?
Comment #5
kehrin commentedbump
Comment #6
kressin commentedsubscribe
Comment #7
Anonymous (not verified) commentedThe issue here is that when checking that classes still exist, the module automatically restores the default if it encounters a class that does not exist. "mailsystem_theme" is part of the variable that stores mailsystem settings, so when it gets to that key (whose value is never going to be a class) it restores the default.
Comment #8
Anonymous (not verified) commentedEr... maybe not quite. Had this issue on multiple sites and this resolved it, but looking through the rest of the module it looks like "mailsystem_theme" should not even have ever been set within the variable holding the array of classes. So not sure when that issue was introduced.
However, I do think there needs to be some better checking done where the default is reverted in that loop.