improve usability for Mailer module #548

Closed
ifs-net opened this Issue Oct 22, 2012 · 4 comments

Comments

Projects
None yet
3 participants

ifs-net commented Oct 22, 2012

when testing mailer module and the sending fails the fields of the form are all empty and have to be refilled manually again.
when sending fails the fields should not be reset!

Owner

cmfcmf commented Nov 16, 2012

I see two ways to do that:

  1. Put testconfig() and sendmessage() in one function, which checks weather to send an email or to display the input fields. Then it could also assign the old input values to the template.
  2. Add a form handler.

I think a form handler is better because of highlighting incorrect fields and it's error messages.

Member

jusuff commented Nov 17, 2012

Form handler seems to be right choice.

jusuff was assigned Nov 17, 2012

@jusuff jusuff added a commit to jusuff/core that referenced this issue Nov 18, 2012

@jusuff jusuff Use Zikula_Form for Mailer settings view (refs #548) 776f512

@jusuff jusuff added a commit to jusuff/core that referenced this issue Nov 18, 2012

@jusuff jusuff Use Zikula_Form for Mailer test config view (refs #548) 6bedbb1
Member

jusuff commented Nov 18, 2012

@ifs-net Made some changes to Mailer module. Please let me know it this what you wanted.

ifs-net commented Feb 6, 2013

@ifs-net Made some changes to Mailer module. Please let me know it this what you wanted.

yes, testet with 1.3.5, seems to work fine

ifs-net closed this Feb 6, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment