torchbox / wagtail / 7788

96%

push

travis-ci

Loop through all multiple value fields when sending form submission by email.

A form field with multiple checkboxes may introduce submissions that have more than one value. The `send_email` function would loop through all form fields and grab only the first item of each field, ignoring fields that may have multiple values selected. This is a side effect of using `QueryDict.get()`, which will only return the last value of a list.

This commit fixes that by first converting the `QueryDict` instance to a regular dict of lists so that `get` will return all values, then joining multiple items if needed.

12 of 12 new or added lines in 2 files covered. (100.0%)

30142 of 31317 relevant lines covered (96.25%)

22.94 hits per line