Uploaded image for project: 'Magnolia Form Module'
  1. Magnolia Form Module
  2. MGNLFORM-354

Allow configuration of attachment disposition from mail form processor

    XMLWordPrintable

Details

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

    Description

      Currently AbstractEMailFormProcessor uses default configuration of disposition setting when generating list of attachments. This could be easily made configurable together with mime multipart settings to make the output more compatible with target mail system. 

      Steps to reproduce

      1. Create page with form component on it. (default form component from magnolia forms module)
      2. Configure your form to send an email after submission.  
      3. As one of the fields in the form pls add formFile component. 
      4. When you are submitting your form, upload to created file field for example jpg image. 
      5. Check Content Disposition for that attachment

      Expected results

      A chance of switching between DISPOSITION_INLINE and DISPOSITION_ATTACHMENT so the attachment can be view / downloaded

      Actual results

      When info.magnolia.module.mail.templates.MailAttachment.MailAttachment(URL url, String name) is called, the content disposition is hardcoded to INLINE which causes in some cases (like using Outlook Web App) that the attached image is completely ignored and there is no possibility to view it or even download.

      Workaround

      N/A

      Development notes

      info.magnolia.module.mail.templates.MailAttachment class already has the

      public static final String DISPOSITION_ATTACHMENT = "attachment"; 

      option, but every time is hardcoded to DISPOSITION_INLINE

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              Unassigned Unassigned
              ccantalapiedra Carlos Cantalapiedra
              AuthorX
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:

                Checklists

                  Task DoD