[drupal-devel] [task] GD2-specific functions should be moved to image.gd2.inc

Junyor drupal-devel at drupal.org
Mon Mar 28 22:10:14 UTC 2005


Issue status update for http://drupal.org/node/19611

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     tasks
 Priority:     normal
 Assigned to:  Junyor
 Reported by:  Junyor
 Updated by:   Junyor
 Status:       patch
 Attachment:   http://drupal.org/files/issues/image.imlib.inc (2.07 KB)




Junyor



Previous comments:
------------------------------------------------------------------------

March 28, 2005 - 22:46 : Junyor

Attachment: http://drupal.org/files/issues/system-image.patch (6.8 KB)

The new image handling tools in core have a lot of GD2-specific logic,
making image.inc overly cluttered and complicated.  If gd2 were just
treated like another toolkit (though the only one included by default),
then the image.inc code would be much cleaner and more easily
maintained.
This patch removes all the GD2 stuff from image.inc and puts it into
image.gd2.inc.  The _settings hook is now doing what it sounds like it
should, return settings for the toolkit.  I've added a _check hook
which will tell if the PHP functions needed are available.  If _check
fails, then the toolkit will not be listed on admin > settings and an
error will be logged in watchdog.
I've also update image.gd1.inc, image.imlib.inc, and image.imagick.inc
with the same logic.


------------------------------------------------------------------------

March 28, 2005 - 23:08 : Junyor

Attachment: http://drupal.org/files/issues/image.gd2.inc (3.07 KB)




------------------------------------------------------------------------

March 28, 2005 - 23:09 : Junyor

Attachment: http://drupal.org/files/issues/image.gd1_0.inc (1.8 KB)




------------------------------------------------------------------------

March 28, 2005 - 23:09 : Junyor

Attachment: http://drupal.org/files/issues/image.imagick.inc (2.19 KB)







More information about the drupal-devel mailing list