Seems there is a small regression error with required-fields.

The #element_validate doesn't contain a 'file_managed_file_validate' anymore.

This was fixed in #1356386: Fields set to "Required" are not validated but doesn't work in the latest version.

Quick fix is to add the 'file_managed_file_validate' to the #element_validate array. (Around line 119 and 323?)

CommentFileSizeAuthor
#1 manualcrop-validation_fix-1835482-1.patch714 bytesspadxiii

Comments

spadxiii’s picture

spadxiii commented
StatusFileSize
new manualcrop-validation_fix-1835482-1.patch714 bytes

Made a small patch, similar to the linked issue.

Note that I didn't add the function to the second part, as this wasn't done in the original commit either
see: http://drupalcode.org/project/manual-crop.git/commitdiff/f1e8069554d54b4...

matthijs’s picture

matthijs
Primary language Dutch
commented
Assigned: Unassigned » matthijs
Status: Active » Fixed

Hi,

Thanks for the patch, I've just committed (again ;-) ) and pushed it!

Matthijs

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.