Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update upload_file.rst for handling FileException #10382

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

Closed
wants to merge 2 commits into from
Closed

Update upload_file.rst for handling FileException #10382

wants to merge 2 commits into from

Conversation

LS05
Copy link
Contributor

@LS05 LS05 commented Sep 23, 2018

#9642
Handled FileException in coding examples. We need to discuss how to handle specific file exceptions ;)

FileException handled in examples. This is for bc with 2.8. We need to discuss how to handle specific file exceptions ;)
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

I like this! I'll wait for more opinions before merging it. Thanks!

Typo fixed :)
@LS05
Copy link
Contributor Author

LS05 commented Sep 26, 2018

@javiereguiluz I have fixed a typo in there! :)

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

This is really nice, thanks for this contribution!

@HeahDude HeahDude added this to the 2.8 milestone Sep 27, 2018
@LS05
Copy link
Contributor Author

LS05 commented Sep 27, 2018

Thank you @HeahDude ! Looking forward to do more :)

javiereguiluz added a commit that referenced this pull request Sep 28, 2018
This PR was submitted for the master branch but it was squashed and merged into the 2.8 branch instead (closes #10382).

Discussion
----------

Update upload_file.rst for handling FileException

#9642
Handled FileException in coding examples. We need to discuss how to handle specific file exceptions ;)

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

0a5e663 Update upload_file.rst for handling FileException
@javiereguiluz
Copy link
Member

@LS05 this is now merged! We merged it in 2.8 branch (and we'll merge in the other branches automatically later). By the way, we slightly modified this when merging to add the missing import of the FilexException class. Cheers!

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.

4 participants