Bug Report: Smtp Password When Editing Settings

Smtp Password When Editing Settings

I thought I had reported this before, but maybe it was for a different app. If using SMTP mail transport, if you go back into settings to edit something (could be anything, unrelated to SMT

SS

Scott Smith

Reported

I thought I had reported this before, but maybe it was for a different app.

If using SMTP mail transport, if you go back into settings to edit something (could be anything, unrelated to SMTP), when you try to save the settings there’s an error because the SMTP password is null.

This because the password is not passed into the form, there for the field is null, but then the validation fails on save.

There needs to be something so settings can be changed w/o having to supply the SMTP password again.

Personally, I would just move SMTP settings to a new item under the Settings menu. However it’s accomplished, it is an annoying thing that needs a solution.

  • MS

    Mian Saleem

    Answered

    Hello,

    Thank you for reporting this.

    You can comment the line around 782

    $this->form_validation->set_rules('smtp_pass', lang('smtp_pass'), 'required');
    

    Thank you

  • SS

    Scott Smith

    Answered

    That should stop the validation error.

    However, when the settings are saved, I think this might result in a null string being saved as the SMTP password.

    I will test later today and let you know.

  • SS

    Scott Smith

    Answered

    It works ok. I see in the code the password is preserved, only not set in the form field.

    Ok, thanks!

  • Login to Reply