If you try to upload an image without filling in the description field, the upload will fail without telling you why. It would be a good idea to make the description field required so this doesn't happen.

Comments

swentel’s picture

swentel commented
Status: Active » Fixed

Better descriptions are now provided:

Empty description: You need to enter a description for the image.
Empty image: The image is empty.
Image with errors (eg wrong extension): The image is invalid. Only JPG, PNG and GIF images are allowed.

Status: Fixed » Closed (fixed)

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

mrharolda’s picture

mrharolda commented
Status: Closed (fixed) » Needs work

After uploading a PNG file, I get the error "The image is invalid. Only JPG, PNG and GIF images are allowed." which didn't help me discovering public://sweaver wasn't writable ;)

Sweaver has the same error message for all errors:

        if (!empty($errors)) {
          sweaver_session(t('The image is invalid. Only JPG, PNG and GIF images are allowed.'));
        }

I suggest changing that to:

        if (!empty($errors)) {
          sweaver_session(implode("<br/>", $errors));
        }

(sorry, don't have time to create a patch on the -git version)