Closed (fixed)
Project:
Image javascript crop
Version:
7.x-1.x-dev
Component:
PHP
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2011 at 05:14 UTC
Updated:
16 Nov 2014 at 17:13 UTC
Jump to comment: Most recent
Comments
Comment #1
nils.destoop commentedThe 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?
Comment #2
dboulet commentedI’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.
Comment #3
nils.destoop commentedDo 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?
Comment #4
dboulet commentedYes, 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.
Comment #5
nils.destoop commentedAlso 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.
Comment #6
yevgeny.ananin commentedMy 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?
Comment #7
nils.destoop commentedThe 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?
Comment #8
yevgeny.ananin commentedTurning '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.
Comment #9
nils.destoop commentedOk, i'll close this one then :)
Comment #10
Chris2u commentedApparently 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!)
Comment #11
ianthomas_ukWhat version are you using? This is marked fixed in -dev, but not in 7.x-1.0-rc3
Comment #12
ianthomas_ukReturning to closed (fixed), but please reopen if you are actually using the latest -dev version.