Issue status update for http://drupal.org/node/19517 Project: Drupal Version: cvs Component: base system Category: bug reports Priority: normal Assigned to: Anonymous Reported by: nysus Updated by: stefan nagtegaal Status: patch 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.. stefan nagtegaal Previous comments: ------------------------------------------------------------------------ March 27, 2005 - 02:58 : nysus 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 Typephp DateSaturday, March 26, 2005 - 20:56 Usertest [1] Location/admin/settings [2] MessageThe selected image handling toolkit '/gd2/' can not correctly process '/image_gd2_settings/'. Severityerror Hostname192.168.1.6 [1] http://drupal.org/user/1\" title=\"View user profile. [2] http://drupal.org//admin/settings ------------------------------------------------------------------------ March 27, 2005 - 13:53 : stefan nagtegaal 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? ------------------------------------------------------------------------ March 27, 2005 - 16:14 : wiz Attachment: http://drupal.org/files/issues/image-gd2.patch (119 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. ------------------------------------------------------------------------ March 27, 2005 - 16:17 : wiz I'm pretty sure this fix is ok... ------------------------------------------------------------------------ March 27, 2005 - 16:21 : nysus 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? ------------------------------------------------------------------------ March 27, 2005 - 16:26 : nysus wiz, disregard my comment. I was confused about your patch. I thought it was patching the system.module. ------------------------------------------------------------------------ March 28, 2005 - 04:05 : walkah hrm. looks like a stray 'gd2' slipped in... this patch is good and should fix. +1 to commit.