Posted by Darren Oh on December 12, 2006 at 10:40pm
Jump to:
| Project: | Drupal core |
| Version: | 4.7.x-dev |
| Component: | system.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
The system settings page allows the GD toolkit to be selected even if it is not installed. The error is detected when the form is next loaded, rather than when it is submitted. The form error prevents any valid settings from being submitted.
The attached patch causes validation to be performed before the form is submitted rather than after. It patches both system.module and image.inc.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| gd-toolkit.patch | 2.83 KB | Ignored: Check issue status. | None | None |
Comments
#1
An equivalent patch for Drupal 5 is available in issue 102987, so this fix will not need to be ported.
#2
@Darren Oh - I'm curious about how to test this. Basically we need to have an installation with gd disabled and imagemagick enabled. Is that right?
#3
I have an install without GD and no image toolkit is offered.
#4
Strange: Once I copy image.imagick.inc to the includes directory, I get both GD and imagick offered. If I remove it again then there is no choice offered.
#5
ok, I understand now why I don't get a selection offered with only one toolset present. However I already get a validation error without this patch.
#6
ok, some testing revealed that this indeed makes the form much better behaved.
#7