The cropping is not working for me, every time that I try I get this error in the watchdog:

The file http:‍//example.com/sites/default/files/styles/_imagecrop_temp/public/photos/image.jpg?time=1320987164 was not deleted, because it does not exist.

Comments

nils.destoop’s picture

The error in watchdog is now fixed. But the error had nothing to do with cropping that's not working.
What is exactly not working? The javascript selection, or the saving?

dboulet’s picture

I’ll look into this some more. I think what is happening is that the uncropped image is not being deleted so the newly cropped image cannot be saved in its place.

Everything seems to work fine except that the newly cropped image does not get displayed.

nils.destoop’s picture

Do you mean when viewing the image on the frontend? Then it's probably browser cache. Can you view the image in your browser and do a hard refresh?

dboulet’s picture

Yes, that’s what I mean: the new image does not show up in the front end, even with a refresh. The newly cropped image is not saved over the old one.

nils.destoop’s picture

Also if you do a hard refresh of that image, or clear your cache?
If it's shown ok in the imagecrop overview. Then the image is cropped succesfully.

yevgeny.ananin’s picture

My image javascript crop was working smoothly before. The image style I created that used jscript crop is called crop_image_jscript. Now when I upload an image and then crop it, I get the same error as above, as well as: public://styles/crop_image_jscript/public/slide_images/image.png was not deleted, because it does not exist. What happened?

nils.destoop’s picture

The 2 errors should be gone now.

public://styles/crop_image_jscript/public/slide_images/image.png was idd not deleted, because you probably had 'skip preview' enabled?
Got this error also, and it's now fixed on dev.

The error on temp crop should also be gone.

However, this should not change anything on crop not working. If you open the image on your system, is it then correct size?

yevgeny.ananin’s picture

Turning 'skip preview' off took care of the two error messages. However, the bigger problem I was having was that the cropped image wasn't being written to the javascript crop image style folder. I then upgraded from the December dev version to the January dev version. It's weird, but now everything works fine.

nils.destoop’s picture

Status: Active » Closed (fixed)

Ok, i'll close this one then :)

Chris2u’s picture

Issue summary: View changes
Priority: Critical » Major
Status: Closed (fixed) » Needs review

Apparently this is still an issue when using private storage, at least if "skip previews" is enabled. I'm still getting the same "... was not deleted" watchdog message with the latest version when using private storage together with "skip previews" and cropping does not work.

Thanks for the great module (by far the best of all alternatives!)

ianthomas_uk’s picture

Status: Needs review » Needs work

What version are you using? This is marked fixed in -dev, but not in 7.x-1.0-rc3

ianthomas_uk’s picture

Status: Needs work » Closed (fixed)

Returning to closed (fixed), but please reopen if you are actually using the latest -dev version.