Uploaded image for project: 'Magnolia Mail Module'
  1. Magnolia Mail Module
  2. MGNLMAIL-183

Add Reply-To support in templates package

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Unresolved
    • Neutral
    • None
    • 5.6.0
    • None
    • None

    Description

      Currently we don't make use of the reply-to support offered in javax.mail.internet.MimeMessage. The class offers a method to take in an array of email addresses for Reply-To. See https://www.w3.org/Protocols/rfc822/

      /**
           * Set the RFC 822 "Reply-To" header field. If the address 
           * parameter is <code>null</code>, this header is removed.
           *
           * @exception	IllegalWriteException if the underlying
           *			implementation does not support modification
           *			of existing values
           * @exception	IllegalStateException if this message is
           *			obtained from a READ_ONLY folder.
           * @exception	MessagingException for other failures
           */
          @Override
          public void setReplyTo(Address[] addresses)
      

      Add support by:

      • Adding getter/setter to info.magnolia.module.mail.templates.MgnlEmail
      • Adding getter/setter to info.magnolia.module.mail.templates.impl.FreemarkerEmail

      Update

      • After a closer look I do see that MgnlEmail does provide setReplyToList() which appears to take a "list" (String) delimited by \n characters. This might need to be expanded to handle a list delimited by commas OR maybe the dialog should have a text field with a rows property set so users get the intuition they can have multiple emails.
      • For FreemarkerEmail we need a setReplyTo() method but it's not quite clear how to handle processFreemarkerString() when the list is provided as a parameter. If the list is delimited by \n then I would guess it would make sense to change those out for commas.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                Unassigned Unassigned
                rgange Richard Gange
                Votes:
                1 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:

                  Checklists

                    Task DoD