Add parsing of Attribution, KeywordList and MetadataURL to WMSCapabilities

MetadataURL, Attribution and KeywordList parsing

Thanks for working on this, Andreas.

For me, tests fail in IE7.

 Format/WMSCapabilities/v1_1_1.html: fail 1 ok 1 (detailed: fail 0 ok 13)
    test_read ok 13
        ok object contains capability property
        ok getmap formats parsed
        ok getmap href parsed
        ok layers parsed
        ok correct number of layers parsed
        ok [2] correct layer name
        ok [2] correct layer title
        ok [2] correct layer abstract
        ok [2] correct layer bbox 
        ok [2] correct styles length
        ok [2] correct style name
        ok [2] correct legend url
        ok [2] correct queryable attribute

    test_ogc planned 11 assertions but got 0; fail 0 ok 0
        exception: : object: 'null' is null or not an object

They pass in Safari though.

satisfy IE - removed DTD with invalid URL

The new patch fixes tests in IE. The problem was that the DTD of the OGC example points to an invalid URL.

Hi Andreas, why not use the official location for the DTD instead?

http://schemas.opengis.net/wms/1.1.1/capabilities_1_1_1.dtd

use correct DTD url, which also satisfies IE and is a more appropriate solution

Thanks for the suggestion Bart. New patch uses official DTD url.

Andreas, I can't say commit, but this looks good to me, no further comments.

wmscapabilities enhancements

This looks good.

A couple changes in the latest patch:

• Looks like read_cap_LegendURL wasn't previously parsing the legend format. This method became readImageInfo in the patches above. The format element is now parsed for LogoURL and LegendURL.

• Though both LogoURL and LegendURL have width and height attributes, I think it is a bit clearer to do the parsing in the read_alias_NodeName methods. If you don't mind, I'd rather not have readImageInfo (the next step in improving this is to use readChildNodes and other methods on the XML format).

If you feel strongly about adding readImageInfo, I'm not strongly opposed. In either case, please commit. Tests pass in FF3 and IE6.

Tim: my aim was to avoide code duplication, but since our xml parsers are modular, I see the advantage of better code readibility if everything is done in the block where it belongs to. So I am in favor of your changes.

closed with r9499: added parsing of Attribution, KeywordList and MetadataURL to WMSCapabilities. Thanks tschaub for the improvements to my original patch. r=tschaub