support wms version in print spec

[fvanderbiest]

Currently, mapfishapp-print uses WMS version 1.1.1 to query the layers, see https://github.com/mapfish/mapfish-print/blob/master/src/main/java/org/mapfish/print/map/readers/WMSMapReader.java#L131

The French IGN's geoportail layers support WMS version 1.3.0 only, not 1.1.1
Thus, they are not printed.

[sbrunner]

Is it OK If when the version is specified in the 'customParams' we use the provided value ?

[fvanderbiest]

probably, yes

[fvanderbiest]

I might change my mind. Indeed, the version parameter is part of the WMS layer's DEFAULT_PARAMS in OpenLayers.Layer.WMS prototype, see https://github.com/openlayers/openlayers/blob/master/lib/OpenLayers/Layer/WMS.js#L27

... which means that in GeoExt PrintProvider's WMS layer encoder, the version parameter won't go into the customParams, see https://github.com/geoext/geoext/blob/master/lib/GeoExt/data/PrintProvider.js#L651

As a result, I think it's better to have it alongside with the type = "WMS" parameter in the print spec.

[sbrunner]

In booth case you should add the parameter then I don't see what it will change except to do it less generic ...