I attach "image" field to my ctype, and set S3 storage to it.
when I upload the image, the thumbnail is not shown near the upload button, but the image is attached.
when I try to view node after that, if I use original image display, it's rendered without any problems.
but when I set some image style to display, like "thumbnail", the image is not rendered.

in reports I get:

Notice: Use of undefined constant SEET_SET - assumed 'SEET_SET' in AmazonS3StreamWrapper->stream_seek() (line 385 of /home/ufo/domains/ogo2.pixeljets.com/sites/all/modules/amazons3/AmazonS3StreamWrapper.inc).

Error: The file permissions could not be set on s3://_MG_0733.jpg.

Error: The file permissions could not be set on s3://styles/square_thumbnail/s3.

Notice: Unable to generate the derived image located at s3://styles/square_thumbnail/s3/pixeljets_1.png.

in my amazon s3 console I see that folder for styles were created, but there is no image inside.

CommentFileSizeAuthor
#1 fseek.patch503 bytesjustafish

Comments

justafish’s picture

StatusFileSize
new503 bytes

Caused by a typo, fixed in dev and patch attached

justafish’s picture

Status: Active » Needs review
justafish’s picture

If anyone has some time to review this and confirm that it woulds that would be great, thanks!

allella’s picture

The dev branch appears to have gotten rid of the

Notice: Use of undefined constant SEET_SET - assumed 'SEET_SET' in AmazonS3StreamWrapper->stream_seek()

The logs are still showing

The file permissions could not be set on s3

Though, it doesn't seem to be causing any problems.

restyler’s picture

dev version seems to work good for me!

allella’s picture

Regarding the error messages like

"The file permissions could not be set on s3://"...

Would it make more sense to have the function chmod() in AmazonS3StreamWrapper.inc return TRUE? The function currently does nothing and returns FALSE, but returning FALSE appears to generate the error above in the Drupal logs.

justafish’s picture

Closing this as the patch committed to dev has fixed this.

Please see #1311698 for permissions discussion.

justafish’s picture

Status: Needs review » Closed (fixed)