Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Send copy of ContentFile.file to upload_fileobj. #376

Closed
wants to merge 3 commits into from
Closed

Send copy of ContentFile.file to upload_fileobj. #376

wants to merge 3 commits into from

Commits on Aug 6, 2017

  1. Send copy of ContentFile.file to upload_fileobj. 

    This will stop upload_fileobj from modifying the internal file. This is
important because the default storage mixin for hashed static files that
comes in Django expects files to be open after they are saved, and
upload_fileobj closes them.
    @jpmelos
    jpmelos committed Aug 6, 2017
    Copy the full SHA
    d67586d View commit details
    Browse the repository at this point in the history

  2. Use BytesIO from django.utils.six.

    @jpmelos
    jpmelos committed Aug 6, 2017
    Copy the full SHA
    1be3c02 View commit details
    Browse the repository at this point in the history

  3. Rewind file before reading. 

    This code is run twice per file, so we don't know if the file has been
read before.
    @jpmelos
    jpmelos committed Aug 6, 2017
    Copy the full SHA
    747d698 View commit details
    Browse the repository at this point in the history