I am not experiencing this all the time but occasionally the first time that I view the node I am seeing the original image. Refresh the page and then I see the cropped image.

Has anyone else experienced this?

CommentFileSizeAuthor
#3 imagefield_crop-original_image-1483980-3.patch2.48 KBnlisgo
#2 imagefield_crop-original_image-1483980-2.patch545 bytesnlisgo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nlisgo’s picture

nlisgo CreditAttribution: nlisgo commented

I can confirm that on some occasions the original image is being displayed when the post is being viewed for the first time. I think this may have something to do with the fact that the cropped image overwrites the original image and if the node is viewed before this action is done then we will see the original image.

nlisgo’s picture

nlisgo CreditAttribution: nlisgo commented
FileSize
imagefield_crop-original_image-1483980-2.patch545 bytes

Fixed. This was a caching issue. basically because the cropped image is stored using the same filename as the original image. Cache will inevitably return the original image sometimes. I get round this by changing the location of the cropped file.

nlisgo’s picture

nlisgo CreditAttribution: nlisgo commented
FileSize
imagefield_crop-original_image-1483980-3.patch2.48 KB

Here is an improved patch that renames the file for updates not just inserts.

Peter Feddo’s picture

Peter Feddo CreditAttribution: Peter Feddo commented
Version: 7.x-2.x-dev » 7.x-1.0

I applied the patch and can't seem to get the cropped image to display. The image that gets stored and rendered is the original uncropped version.

nlisgo’s picture

nlisgo CreditAttribution: nlisgo commented
Version: 7.x-1.0 » 7.x-2.x-dev

I had set the version to be 7.x-2.x-dev but you have switched it to 7.x-1.0

Would you prefer that I prepare a patch for the 7.x-2.0 branch? I am not using the 7.x-1.0 branch.

akalam’s picture

akalam CreditAttribution: akalam commented
Issue summary: View changes
Status: Active » Needs review

Thanks for the patch !
I ported it to the 1.x branch

https://www.drupal.org/node/2412183

kcolwell’s picture

kcolwell CreditAttribution: kcolwell commented

I found that desarrollo2.0's patch was very helpful and should be rolled into the next update.

Ken Colwell

joetsuihk’s picture

joetsuihk CreditAttribution: joetsuihk commented

Hi @kcolwell, you mean you tested the patch for 1.x? do you mind go over https://www.drupal.org/node/2412183 and state you had tested that patch? I will commit that to 1.x dev if so.

Thanks.

  • ram4nd committed d48534d on 7.x-2.x authored by nlisgo 
    Issue #1483980 by nlisgo: Original image displayed when the node is...
ram4nd’s picture

ram4nd CreditAttribution: ram4nd as a volunteer commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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