support wms version in print spec about mapfish-print HOT 4 CLOSED

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

from mapfish-print.

probably, yes

from mapfish-print.

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.

from mapfish-print.

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

from mapfish-print.