Download & Extend

Node list shows large images instead of thumbnails

Project:Image
Version:5.x-1.x-dev
Component:image.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Hi all.

I upgraded from 4.7.4 to 5.0 RC1 on Apache running PHP5. When browsing /node/ (or any /views or /taxonomy), the image nodes display full-sized pictures, instead of the thumbnails like it used to, and in my opinion, should.

I marked this as 'critical' because this affects bandwidth usage, and also takes up a lot of space on the screen.

Any newly-created image also shows up full-sized like this.

Attached: Picture of my website, (on /node/) with large images where there used to be thumbnail-sized pictures.

Cheers,
Shadyman@erroraccessdenied.com

AttachmentSizeStatusTest resultOperations
website-bigpictures.PNG347.17 KBIgnored: Check issue status.NoneNone

Comments

#1

I am also having this issue works fine in drupal 4.7x but in drupal 5 both RC1 & RC2 the block of images is in preview sized not thumbnails, I have been trying to think about whether this is an image module issue or a views module issue

#2

Status:active» closed (duplicate)

http://drupal.org/node/99395

#3

Status:closed (duplicate)» active

No, this isn't a duplicate: the previous thread talks about thumbnail images disappearing, in this case its an issue of the display of images (original not thumbs) in a taxonomy view, see for example: http://www.peterjohnchen.com/taxonomy/term/15/feed on a 5.0 install + image.module

#4

sorry about that my bad, i'd grabbed the wrong issue.

#5

Also as seen on any page on http://erroraccessdenied.com.

#6

I was having the same problem so I edited the module to correct it.
The file can be found here
Hope this helps.

-E

#7

here's vatoloco's changes as a patch. i haven't tested it yet.

AttachmentSizeStatusTest resultOperations
vatoloco_image.patch584 bytesIgnored: Check issue status.NoneNone

#8

Status:active» needs review

here's a cleaner version.

AttachmentSizeStatusTest resultOperations
image_teaser.patch686 bytesIgnored: Check issue status.NoneNone

#9

Status:needs review» needs work

The patch makes the teaser text appear twice. See screenshot.

AttachmentSizeStatusTest resultOperations
doubleteaser.PNG159.47 KBIgnored: Check issue status.NoneNone

#10

drewish - THanks for the patch. However - I am seeing the double teaser as well.

#11

I have created a patch that hopefully fixes the problem with the large images in the teasers and the duplicates with the teaser text.
I tried to make the patch around the clean code of drewish but it just wouldn't work.

- E

AttachmentSizeStatusTest resultOperations
vatoloco_image_teaser.patch45.41 KBIgnored: Check issue status.NoneNone

#12

Status:needs work» needs review

Ok, the latest patch works for me. Anyone else?

#13

Priority:critical» normal

Latest patch also works for me.

#14

I open the vatoloco_image_teaser.patch in my text editor but it doesn't recognize the text as PHP code. The patch code lines start with "-" and "+" characters.
I'm a newbie, could someone help me please?
Thanks a lot.

#15

Gianfilippo, see: http://drupal.org/node/60108

#16

Status:needs review» reviewed & tested by the community

#17

I can confirm that it has resolved the issue that I was having.

#18

does any one have the previous version of image.module? This patch is not working for me with the latest dev release of image, as it was written for the previous version. If anyone could direct me to the one version previous, or an already patched file that is currently working, that would be great. Thanks.
sscooterguy

#19

sscooterguy, everything is in CVS take a look at:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/image/image.m...
it'll have every version. the patch should identify which version of the file it was rolled against.

#20

Modified the patch to use LF instead of CR/LF line ending characters. Should apply to current CVS version now.

AttachmentSizeStatusTest resultOperations
vatoloco_image_teaser_mod.patch1.24 KBIgnored: Check issue status.NoneNone

#21

Status:reviewed & tested by the community» fixed

i've committed my patch to HEAD and 5

#22

Status:fixed» closed (fixed)