Skip to content
New issue

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

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

Already on GitHub? # to your account

[Imaging Uploader] Use Utility::getMaxUploadSize() instead of ini_get #4367

Conversation

davidblader
Copy link
Contributor

Brief summary of changes

Makes error message consistent with upload size limit that's displayed on the front-end in the case where post_max_size != upload_max_filesize

To test this change...

  • Try uploading a file greater in size than the limit displayed in the upload form. Error message should specify that the file size cannot be larger than the limit displayed on the upload form

@johnsaigle johnsaigle added Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Cleanup PR or issue introducing/requiring at least one clean-up operation [branch] bugfix labels Feb 27, 2019
@driusan driusan merged commit 291c5d9 into aces:bugfix Feb 27, 2019
@ridz1208 ridz1208 added this to the 20.2.1 milestone Mar 14, 2019
kchatpar pushed a commit to kchatpar/Loris that referenced this pull request Apr 15, 2019
…aces#4367)

Makes error message consistent with upload size limit that's displayed on the front-end in the case where `post_max_size != upload_max_filesize`
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Cleanup PR or issue introducing/requiring at least one clean-up operation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants