Use of drupal_realpath is deprecated because it breaks stream wrappers, e.g., using Amazon S3 as the filesystem with imagecrop causes it to break.
See this issue for more info for a similar issue that was with the xml sitemap module: http://drupal.org/node/1460594

Patch attached. Alternative to simply removing, you could maybe replace with file_create_uri() as replacement for drupal_realpath(), however it seemed that on most systems the uri functioned fine without any function call needed on it at all.

CommentFileSizeAuthor
imagecrop-no-realpath.patch690 bytescoderintherye

Comments

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
commented
Issue summary: View changes
Status: Needs review » Fixed

This change has already been made in a fix for #1272162: Page Not Found when cropping with Media Module but was not in 7.x-1.x-rc3.

http://drupalcode.org/project/imagecrop.git/blobdiff/70ed56d53655b3beed9...

Status: Fixed » Closed (fixed)

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

1kenthomas’s picture

1kenthomas commented
Title: Remove use of deprecated function drupal_realpath » Make compatible with stream wrappers
Status: Closed (fixed) » Active

Current -dev fails with at least Amazon S3 as filesystem.

Note: fails to read S3 files. Self-assigning for now... will try to get to work.

1kenthomas’s picture

1kenthomas commented
Assigned: Unassigned » 1kenthomas
1kenthomas’s picture

1kenthomas commented
Status: Active » Closed (fixed)

Will open a new issue.