Issue status update for http://drupal.org/node/18093 Project: Drupal Version: cvs Component: system.module Category: bug reports Priority: normal Assigned to: Anonymous Reported by: Mad Maks Updated by: Junyor -Status: active +Status: patch Attachment: http://drupal.org/files/issues/junyor.system.patch (927 bytes) OK, the problem seems to be that image_gd_settings() returns TRUE. So, if there's only one toolkit installed, you'll just see "1" in the image handling section on the settings page. I had thought that "image_toolkit_invoke('settings')" was supposed to show settings for an image toolkit, not verify that the toolkit is installed. Here's a new patch that corrects the problem in the first patch. If only one toolkit is installed and working, it prints a message saying the toolkit is ready for use. If no toolkits exist, the same message as in the previous patch is printed. If no message should be printed if no settings are available (only one toolkit is installed or no toolkits are installed), I can roll another patch. Junyor Previous comments: ------------------------------------------------------------------------ February 28, 2005 - 10:28 : Mad Maks i have installed the cvs version yesterday (upgrade from 4.5.2) and now i see at administration => settings a block with the titel image handeling. But in that block there is nothing except a "1" . Is this a now bug or did i do something wrong? greetings MM ------------------------------------------------------------------------ March 1, 2005 - 09:24 : Mad Maks nobody? ------------------------------------------------------------------------ March 9, 2005 - 11:41 : Mad Maks BrianH has posted a screenshot in http://drupal.org/node/18588 ------------------------------------------------------------------------ March 19, 2005 - 21:25 : Junyor Attachment: http://drupal.org/files/issues/system_5.patch (976 bytes) I think the problem is that no image toolkits are installed at your host. Here's a patch that should clean up things a bit by adding information if no toolkits are available. ------------------------------------------------------------------------ March 20, 2005 - 10:18 : Mad Maks thanks ------------------------------------------------------------------------ March 21, 2005 - 22:26 : Dries Committed to HEAD. ------------------------------------------------------------------------ March 22, 2005 - 05:29 : TDobes Forgive me if this is totally incorrect, but... A cursory glance at this patch seems to indicate that it's not the correct fix. The image_toolkit_invoke('settings') only gets called if there is more than one toolkit. What if there's only one toolkit installed and it has settings available? Also, the else clause is encountered if the number of toolkits is not greater than one (in other words: less than two). The else clause tells the user that there are NO toolkits. Isn't this message inaccurate if there's only one toolkit? For that matter, won't there always be at least one toolkit because we bundle one in image.inc? If we've decided that there will never be any settings from the core toolkit, perhaps we should just change the text to read "There are no extra toolkits installed, so image manipulation will be done using gd2." or something like that. From a user's perspective, the "no toolkits" message would be interpreted as "you can't manipulate images", which is not the case. Feel free to correct me if I'm wrong here... I haven't yet thoroughally looked over the image handling code. ------------------------------------------------------------------------ March 22, 2005 - 08:49 : Junyor @TDobes: You're correct, it's the wrong fix. I'm looking into it again now.