Ah, yeah this stuff I just moved without changing.
On Dec 19, 2013 11:00 AM, "Cory Zue" <notifications@github.com> wrote:

> In couchforms/getters.py:
>
> > +    attachments = {}
> > +    if request.META['CONTENT_TYPE'].startswith('multipart/form-data'):
> > +        # it's an standard form submission (eg ODK)
> > +        # this does an assumption that ODK submissions submit using the form parameter xml_submission_file
> > +        # todo: this should be made more flexibly to handle differeing params for xform submission
> > +        try:
> > +            instance = request.FILES[MAGIC_PROPERTY].read()
> > +        except MultiValueDictKeyError:
> > +            instance = MULTIPART_FILENAME_ERROR
> > +        else:
> > +            for key, item in request.FILES.items():
> > +                if key != MAGIC_PROPERTY:
> > +                    attachments[key] = item
> > +    else:
> > +        #else, this is a raw post via a j2me client of xml (or touchforms)
> > +        #todo, multipart raw submissions need further parsing capacity.
>
> it doesn't bother me, but i'm surprised to see non-pep8 in your code
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/couchforms/pull/76/files#r8473339>
> .
>