[Geoserver-devel] [jira] Created: (GEOS-4017) Simple patch for ColorMapLegendCreator.java

Simple patch for ColorMapLegendCreator.java
-------------------------------------------

                 Key: GEOS-4017
                 URL: http://jira.codehaus.org/browse/GEOS-4017
             Project: GeoServer
          Issue Type: Improvement
          Components: WMS
    Affects Versions: 2.0.2
            Reporter: Andres Gonzalez
            Assignee: Andrea Aime
         Attachments: ColorMapLegendCreator.java.patch

I have been patching ColorMapLegendCreator.java since version 2.0.0 for CIESIN at Columbia University. We simply wanted to get rid of the footer on the legends, since aren't much use to us, and generate larger than necessary legends. This closes ticket GEOS-2849, allowing users to disable the footer.

In addition to this I found a minor mistake in the logic to determine the width of the legend. I am attaching a patch, from a checkout earlier today from trunk.

*Please note* originally I had added a new custom parameter called 'addFooter', but I now see that 'bandInfo' was added. According to the documentation, 'bandInfo' is supposed to take care of this, but it seems to still be showing the footer, and does not seem to make any change to the rendered legends. Because of this I just used the 'bandInfo' property, instead of adding a new one. Also, my feeling is this should be set to 'false' by default, but I left it as-is. Please be sure to read my comments.

CIESIN is planning to contribute additional code to the legends in the future, and already have some proof of concept for JSON response for WMS getFeatureInfo (based on older code) -- so eventually we would like to look into collaborating with development, and hopefully get commit access in the future.

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira