On fresh install, drupal has already checked the availability of GD Library at status report page (admin/reports/status).
When I enable imageapi, imagecache and GD Library support, it adds another check in status report. So there are two checks on GD Library.
When I try to disable GD Library support ships with imageapi, imageapi complains that it needs GD Library installed, while at the same, at the status report page drupal has already confirmed that GD Library is correctly installed.
I have no idea how this can happen and any possible problem it may cause. It just doesn't sound right when two checks are there CMIIW.
Any hint and suggestion would be very much appreciated.
Thanks
Comment | File | Size | Author |
---|---|---|---|
#19 | imageapi_537834.patch | 1.03 KB | drewish |
#18 | imageapi_537834.patch | 2.22 KB | drewish |
Comments
Comment #1
stefanwray CreditAttribution: stefanwray commentedI'm noticing the same thing. GD Library is checked twice.
Comment #2
Weka CreditAttribution: Weka commentedI am experiencing the same issue where GD library is listed twice on the Status report page.
This is a new 6.13 install and I wonder if this is something to worry about?
Comment #3
scifisi CreditAttribution: scifisi commentedYup me too. I'm having an amazing amount of grief with imagecache not working so I'm looking at every conceivable possibility. I've followed every step I've found so far and this double listing for GD library is about the only possible thing I can see that might cause a problem.
I'm concerned that it might be messing imagecache up, I'm not sure but ANY evidence of flakeyness is not a good thing - Anyone found a way around this?
Comment #4
Weka CreditAttribution: Weka commentedYou need to describe in more detail than "imagecache not working".
Although GD library is listed twice on the Status report page, I am not experiencing any problems with ImageCache at all.
Comment #5
drewish CreditAttribution: drewish commentedi don't think it's totally accurate to say it's double listed. one of them is from core and one is from imageapi. if gd is enabled it's used to warn about insufficient memory and other conditions that might cause the library to not work as expected. perhaps we just need to clarify the labeling.
Comment #6
ExTexan CreditAttribution: ExTexan commented@drewish... that sounds like a good solution. When might that be done?
I know it's not a *real* problem (not functionally at any rate) but it does seem to be causing more than a few people to go searching for the solution to a *perceived* problem - which is, in fact, a waste of time.
Comment #7
Weka CreditAttribution: Weka commentedAs not to waste Andrew's time on something trivial as this, a short comment on the project page would be sufficient for now. 8-)
Comment #8
ExTexan CreditAttribution: ExTexan commentedHmmm... wouldn't it take the same amount of time (or perhaps even more) to add a comment on the project page rather than just changing the label as he suggested?
I understand that he probably has better things to do than make such a trivial change - but that's exactly my point... we *all* have better things to do than go searching for a problem that isn't really a problem. We only see a few comments left here in this thread, but that doesn't tell us how many others (tens?... hundreds?) have wasted *their* valuable time on this. Compare that to one person's time to make a small quick change.
Comment #9
Weka CreditAttribution: Weka commentedYou are right, changing label does not take much time and I bet that the change has been made in dev already. However, there are other issues that need to be resolved before the next version is released.
Comment #10
jeffpurtle CreditAttribution: jeffpurtle commentedSame thing here too.
Any answers?
Comment #11
verta CreditAttribution: verta commentedI am seeing this in Drupal 6.16 and ImageAPI GD2 6.x-1.8 and it looks like an error to have two identical status checks on the status page.
The status report is there to give admins confidence that their installation is correct and information to help them correct errors. The current situation erodes that confidence.
Lower the priority of the issue if it is trivial, that's what it's there for. Just 2 cents.
Comment #12
tomsm CreditAttribution: tomsm commentedsame here... subscribing
Comment #13
dpatte CreditAttribution: dpatte commentedThe label is still broken in 6.17. Closing in on a year now.
Comment #14
klonosyep! *not* fixed in latest 6.x-1.x-dev (2010-Jul-11) either as guessed in #9.
I think that changing the label to something more distinguishing/meaningful is a nice temporary measure, but the real solution would be to eventually merge any status message with core status.
Comment #15
bryancasler CreditAttribution: bryancasler commentedSubscribe, same problem. What is going on?
Comment #16
philbar CreditAttribution: philbar commentedsubscribe
Comment #17
klonos...can we have an update on this one from the maintainer(s) please?
Comment #18
drewish CreditAttribution: drewish commentedI'm really not seeing this as a big deal. If you have ImageAPI enabled with no toolkits it complains that you need to enable one. That's a good thing.
If you have Drupal's Color module and ImageAPI GD module enabled you'll see two messages. ImageAPI GD does additional checks that Color does not need because it uses additional functionality. I've got no idea why that would concern people but to make everyone happy here's a patch that doesn't stick the check that duplicates color module's check.
Comment #19
drewish CreditAttribution: drewish commentedWent ahead and committed the attached patch.
Comment #20
klonosThanx again ;)
Comment #21
verta CreditAttribution: verta commentedYay, thanks - confirmed that it only has one line now on admin/reports/status for the GD Library. Really appreciate this, attention to little details can make a big difference.
Comment #22
dpatte CreditAttribution: dpatte commentedI too can confirm that the fix works for me
Comment #23
klonos@Andrew, #18:
...well see? Big a deal or not, you already have three happy customers ;)