Node list shows large images instead of thumbnails

shadyman@errora... - January 3, 2007 - 07:00
Project:Image
Version:5.x-1.x-dev
Component:image.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

AttachmentSize
website-bigpictures.PNG347.17 KB

#1

robbiethegeek - January 13, 2007 - 02:49

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

drewish - January 18, 2007 - 05:40
Status:active» duplicate

http://drupal.org/node/99395

#3

peter chen - January 18, 2007 - 20:02
Status: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

drewish - January 18, 2007 - 23:14

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

#5

shadyman@errora... - January 19, 2007 - 07:15

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

#6

vatoloco - January 20, 2007 - 01:25

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

drewish - January 20, 2007 - 01:59

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

AttachmentSize
vatoloco_image.patch 584 bytes

#8

drewish - January 20, 2007 - 02:07
Status:active» needs review

here's a cleaner version.

AttachmentSize
image_teaser.patch 686 bytes

#9

shadyman@errora... - January 20, 2007 - 07:07
Status:needs review» needs work

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

AttachmentSize
doubleteaser.PNG 159.47 KB

#10

Spacoli - January 20, 2007 - 16:18

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

#11

vatoloco - January 20, 2007 - 19:44

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

AttachmentSize
vatoloco_image_teaser.patch 45.41 KB

#12

shadyman@errora... - January 21, 2007 - 04:03
Status:needs work» needs review

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

#13

doughnut - January 22, 2007 - 11:02
Priority:critical» normal

Latest patch also works for me.

#14

Gianfilippo - January 22, 2007 - 11:25

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

drewish - January 22, 2007 - 17:35

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

#16

shadyman@errora... - January 22, 2007 - 19:29
Status:needs review» reviewed & tested by the community

#17

robbiethegeek - January 24, 2007 - 01:52

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

#18

sscooterguy - February 3, 2007 - 19:57

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

drewish - February 3, 2007 - 22:32

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

airblaster - February 25, 2007 - 23:29

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

AttachmentSize
vatoloco_image_teaser_mod.patch 1.24 KB

#21

drewish - February 26, 2007 - 01:28
Status:reviewed & tested by the community» fixed

i've committed my patch to HEAD and 5

#22

Anonymous - March 12, 2007 - 02:20
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.