If you have two scheduled backups (files and database) and they both send to e-mail it craps. The first one runs and the second gets a redeclaration of the class error.

If cron is run again then it runs the second one fine. Moving the class outside of the function should fix issue, from my understanding conditionally defining things is not usually a good idea.

CommentFileSizeAuthor
#1 588430-mail-class.patch4.36 KBboombatower

Comments

boombatower’s picture

boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 588430-mail-class.patch4.36 KB

This patch makes it work...but I would recommend not defining things conditionally as, to my understanding, it is considered a bad practice...and this shows one of the reasons.

Otherwise great module...I'm running the dev of this and files with the patch applied and things seem to work well.

ronan’s picture

ronan commented

Awesome, thanks for the patch. Truth be told, I don't use this functionality myself so I just ported it straight across from greggle's 1.x patch and haven't paid much attention to it since. I kept meaning to clean that up so thanks again for the patch. I'll get it committed as soon as I have a free minute.

ronan’s picture

ronan commented
Status: Needs review » Fixed

Applied.

Thanks again for the patch.

Status: Fixed » Closed (fixed)

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