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

Comments

robbiethegeek’s picture

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

drewish’s picture

Status: Active » Closed (duplicate)
peter chen’s picture

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

drewish’s picture

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

shadyman@erroraccessdenied.com’s picture

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

vatoloco’s picture

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

-E

drewish’s picture

StatusFileSize
new584 bytes

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

drewish’s picture

Status: Active » Needs review
StatusFileSize
new686 bytes

here's a cleaner version.

shadyman@erroraccessdenied.com’s picture

Status: Needs review » Needs work
StatusFileSize
new159.47 KB

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

Spacoli’s picture

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

vatoloco’s picture

StatusFileSize
new45.41 KB

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

shadyman@erroraccessdenied.com’s picture

Status: Needs work » Needs review

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

doughnut’s picture

Priority: Critical » Normal

Latest patch also works for me.

Gianfilippo’s picture

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.

drewish’s picture

shadyman@erroraccessdenied.com’s picture

Status: Needs review » Reviewed & tested by the community
robbiethegeek’s picture

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

sscooterguy’s picture

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

drewish’s picture

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.

airblaster’s picture

StatusFileSize
new1.24 KB

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

drewish’s picture

Status: Reviewed & tested by the community » Fixed

i've committed my patch to HEAD and 5

Anonymous’s picture

Status: Fixed » Closed (fixed)