Closed (fixed)
Project:
SMTP Authentication Support
Version:
8.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
18 Jan 2016 at 10:21 UTC
Updated:
4 Mar 2016 at 11:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
agialab commentedI have created a simple patch for that - it sets the default system as 'SMTPMailSystem' when the module is enabled and 'php_mail' when it is disabled.
Comment #3
agialab commentedComment #5
agialab commentedComment #6
agialab commentedComment #8
ptmkenny commentedI can confirm this bug. I enabled the module, configured my Mandrill SMTP info, turned SMTP "on" in the Install Options, and then sent a test mail-- which was sent from the server rather than the SMTP module.
Comment #9
estoyausenteI had the same issue. I don't know why I couldn't apply the patch in dev branch, git throw me an error.
I just rebuild the patch uploaded in #5 and now it work correctly and can be apply.
please review it.
Comment #10
ptmkenny commentedComment #11
estoyausenteAdd related issue. This patch seems to fix this other issue too.
Comment #12
estoyausenteComment #13
ptmkenny commentedI can confirm that this module correctly sets SMTP to the default when it is turned on and restores the default to php_mail when it is turned off.
Comment #14
arlina commentedAttached patch builds on the patch from #9 by estoyausente, but instead of restoring to the default "php_mail", it saves it to the config "prev_mail_system", as uninstall routine tried to.
Note: It requires the patch in https://www.drupal.org/node/2653338#comment-10802106 to fix the uninstall issues before testing this patch.
Comment #16
estoyausente@Arlina Maybe we can merge both patchs in order to pass the test and build a commitable patch.
Comment #17
arlina commentedGood idea @estoyausente. Attached patch merges #14 and https://www.drupal.org/node/2653338#comment-10802106 :)
Comment #18
estoyausenteIt seems that not working correctly:
Trying to send testmail using GMAIL as smtp server:
Fatal error: Cannot use object of type Drupal\Core\Config\Config as array in /home/rmx9j/www/sites/default/modules/smtp/src/Form/SMTPConfigForm.php on line 221
Comment #19
arlina commentedJust ran into that as well. Attached is updated version of patch that fixes that, I was able to successfully configure an smtp inbox and send mail though it :)
Comment #20
estoyausenteThe patch is ok although I can't send the mail without apply this another patch: https://www.drupal.org/node/2611210
I hope that maintainer commit asap the 2-3 patch necessaries to use the module. Now it's imposible test patchs in simplytest.me because the module haven't all ok in the dev branch.
Comment #21
mikeegouldingThis patch worked for me as well. Same result as #20. I needed the other patch for mail to actually send.
Comment #22
wundo commentedComment #24
wundo commentedComment #25
wundo commented