I compiled my version of PHP with the --with-gd option on. I do not have the image.module installed. I get the following error in my watchdog logs:

details

Type php
Date Saturday, March 26, 2005 - 20:56
User test
Location /admin/settings
Message The selected image handling toolkit 'gd2' can not correctly process 'image_gd2_settings'.
Severity error
Hostname 192.168.1.6
CommentFileSizeAuthor
#2 image-gd2.patch119 byteswiz-1

Comments

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

I can veriy that this is the case.. Not sure how to fix though..
Another side effect of this is that we can't upload any files, is this only my install or do other people can duplicate the problem?

wiz-1’s picture

wiz-1 commented
Component: system.module » base system
StatusFileSize
new image-gd2.patch119 bytes

Same problem here, I do have gd2 installed. There seems to be a confusion between 'gd' and 'gd2' in image.inc. After applying the patch it works for me again: the built in functions are called image_gd_xxxx, but because of this one line sometimes image_gd2_xxx functions are looked for.

wiz-1’s picture

wiz-1 commented

I'm pretty sure this fix is ok...

Steve Dondley’s picture

Steve Dondley commented

I'm not sure if that patch is the right way to go. I looked into this and traced the problem back to lines 34 and 35 in the includes/image.inc file. These lines set the $toolkit_file variable to includes/image.gd2.inc and then checks for its existence. That file does not exist, however.

Perhaps it does not exist because it is still being written? Can we get someone working on the image module to comment on this?

Steve Dondley’s picture

Steve Dondley commented

wiz, disregard my comment. I was confused about your patch. I thought it was patching the system.module.

Steve Dondley’s picture

Steve Dondley commented
walkah’s picture

walkah commented

hrm. looks like a stray 'gd2' slipped in... this patch is good and should fix.

+1 to commit.

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

Well - maybe it is just me - but isn't it the other way around? Isn't 'gd2' the only good name for that? It is GD2-based code, so code-wise gd2 would be better..

Steven’s picture

Steven commented

Fixed in HEAD.

Anonymous’s picture

(not verified) commented