Any progress?

Also needed for email servers such as spam filtering service. Ours white lists when we send an email.

We'll probably need to allow config to define multiple SMTP server profiles. Helpdesk Setup would pick a default outgoing SMTP, and Groups/Buckets could choose a different SMTP to use. This would prevent us needing to input the full SMTP details (host, port, encryption, user/pass, etc) multiple times.

Since it looks like we're going to allow a "From:" address per bucket, this will likely be at the bucket level, and cascade to group or global as needed.

http://forums.cerb4.com/showthread.php?p=11645#post11645

Well doing thing it would be great if you could do this one at the same time since it is just a minor UI tweak.

[Mail Setup] Add an option to disable a mail server on the add screen. It is already on the edit screen.
http://www.wgmdev.com/jira/browse/CHD-1273

Thanks
Robert

dobesv in the forums.
http://www.cerb4.com/forums/showpost.php?p=13871&postcount=10

Branch with latest Cerb5 patch

http://github.com/rmiddle/cerb5/tree/jira/chd-374

Thanks
Robert

Updated patch. Don't really need 2 fuctions when you can do it in one.

Branch updated as well.

Update MZSupport in the forums.
http://www.cerb4.com/forums/showpost.php?p=14016&postcount=2

Updated the patch to fix a few typo's that didn't show up in initial testing.

Thanks
Robert

There are a few more errors in the most recent patch cerb5-chd-374_3.patch causing an unexpected T_CONSTANT_ENCAPSED_STRING in /home/www/rosehost/cerberus-gui/api/app/Mail.php when applied to cerb 5.0.0

The following line in the patch is missing a comma so instead of:

+ 'auth_user' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_AUTH_USER''),

should be

+ 'auth_user' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_AUTH_USER,''),

and the following line in the patch has one comma too many so instead of:

+ 'enc' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_ENC,,'None'),

should be

+ 'enc' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_ENC,'None'),

Bobi R,

Thanks for the updates. I have been meaning to update those patches. thanks for pointing out my mistakes.

Thanks
Robert

Updated patch thanks to Bobi R, who fixed a few typos I had left it the previous patch.

Thanks
Robert

There is something seriously wrong with this patch. It simply does not work, emails all ticket replies through the default SMTP server even though there are group SMTP servers configured and tested. I don't know the code very well, but looking at this snippet:

                } else { // regular mail sending
                        try {
                                @$group_smtp = DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_IS_ENABLED, 0);
                                // objects
                                $mail_service = DevblocksPlatform::getMailService();
                                if ($group_smtp) {
                                        $mailer = $mail_service->getMailer(CerberusMail::getMailerDefaults());
                                } else {
                                        $mailer = $mail_service->getMailer(CerberusMail::getMailerDefaults($team_id));
                                }
                                $email = $mail_service->createMessage();

Isn't this wrong? Shouldn't the logic for the if statement be opposite of what it is? I mean, if there is $group_smtp then $mailer should be set to getMailer(CerberusMail::getMailerDefaults($team_id), not the other way around, right? This same logic is seen across the entire patch, so I may be wrong. Has this patch ever been tested?

Thanks.

BR

one more error I just found:

                                'max_sends' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_MAX_SENDS,20),

should be

                                'max_sends' => DAO_GroupSettings::get($team_id, DAO_GroupSettings::SETTING_SMTP_MAX_SENDS,20),

Please note that this is unrelated to the reversed logic problem in the if statements mentioned in the comment above. The problem with the reversed logic still persists.

BR,

I had to look at that several times to see you were right I did very limited testing on this patch I see you jumped into the campfire for more debugging. I will be there today(5/23) Sunday until around Midnight and as a general rule Monday - Friday 9:00 AM to 10:00 PM EST.

Thanks
Robert

This should be easier to implement with the Reply-To features in Cerb5 now.