Hi. I've noticed a confusing situation when image uploading.
If we have an image field and we'll set filefield path restrictions on image size like:
* Minimal dimension NxN
* Maximal dimension NxN
For example N equal to 500 and so we have next dimensions for the image: from 500x500 to 500x500.
If we try to load an image with dimensions MxR where M greater than N and R less than M we'll get confusing output like:
https://dl.dropboxusercontent.com/1/view/36p6hz55jmxxdbu/Apps/Shutter/Se...

Looks like it's wrong because image doesn't upload and status message is extra and confusing.
So looks like it should be reworked to not output status message if any error existed.

CommentFileSizeAuthor
#4 7.43-confusing-file-size-validation-message-2745751-4.patch1.69 KBIRuslan
#2 7.43-confusing-file-size-validation-message-2745751-2.patch1.72 KBIRuslan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Shlyapkin Grigoriy created an issue. See original summary.

IRuslan’s picture

IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commented
Status: Needs work » Needs review
FileSize
7.43-confusing-file-size-validation-message-2745751-2.patch1.72 KB

Fall into the same issue, and there is a patch in the attachment for this.

Status: Needs review » Needs work

The last submitted patch, 2: 7.43-confusing-file-size-validation-message-2745751-2.patch, failed testing.

IRuslan’s picture

IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commented
Status: Needs work » Needs review
FileSize
7.43-confusing-file-size-validation-message-2745751-4.patch1.69 KB

Re-upload patch with proper paths.

Version: 7.43 » 7.x-dev

Core issues are now filed against the dev versions where changes will be made. Document the specific release you are using in your issue comment. More information about choosing a version.