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.

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

Adding information about empty files sent using BinaryFileResponse. #5345

Closed
wants to merge 1 commit into from
Closed

Adding information about empty files sent using BinaryFileResponse. #5345

wants to merge 1 commit into from

Conversation

kherge
Copy link
Contributor

@kherge kherge commented Jun 1, 2015

Updating documentation to account for weird scenario described in #5268.

@javiereguiluz
Copy link
Member

👍 thanks for adding this help note. Maybe our native English speaker wants to do some minor changes. But apart from that, the note is nice. Thanks!

@@ -517,6 +517,14 @@ It is possible to delete the file after the request is sent with the
:method:`Symfony\\Component\\HttpFoundation\\BinaryFileResponse::deleteFileAfterSend` method.
Please note that this will not work when the ``X-Sendfile`` header is set.

.. note::

If a file is created in the same request as when the ``BinaryFileResponse``
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in the same request the BinaryFileResponse is sent

@weaverryan
Copy link
Member

Thanks @kherge!

weaverryan added a commit that referenced this pull request Oct 15, 2015
…ileResponse. (kherge)

This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5345).

Discussion
----------

Adding information about empty files sent using BinaryFileResponse.

Updating documentation to account for weird scenario described in #5268.

Commits
-------

aa2ac55 Adding information about empty files sent using BinaryFileResponse.
@weaverryan weaverryan closed this Oct 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants