Hi,

Stefan, I don't know how the server ensure backward compatibility, but I think that in OpenJUMP, the version always represent the version given in the request through the version parameter

url?SERVICE=WMS&*VERSION*=1.3.0

This version is chosen by the user in URLWizardPanel and is used by WMService to build the getCapability request, then to  build a MapRequest. But the MapRequest is built with the same version as the WMService and I'm not sure that it makes sense to build the MapRequest with another version. So I'm not sure that MapRequest needs a "version" attribute at all. This is probably the same for WMSLegend and WMSLayer.

@Jukka : as it concerns WMS usage, maybe you can tell us if WMS_1_1_1 is still a good default option. When I connect to our service in France, I know I must use the WMS_1_3_0 protocol, but if I let the WMS_1_1_1 the getCapability will return more layers, including layers I'm not allowed to read (may also be a server side problem)

Michaël


Le 22/02/2019 à 14:51, Stefan Steiniger a écrit :
Uhm... a service may be backward compatible and can therefore support different request versions? - just guessing ;)

On Fri, Feb 22, 2019, 10:42 Michaël Michaud, <m.michael.mich...@orange.fr <mailto:m.michael.mich...@orange.fr>> wrote:

    Ede,

    This value is not used. WMSRequest version is initialized from
    WMService
    version. Not sure why version is an attribute of both WMService and
    WMSRequest.

    I just was adviced from my IDE. Let me have a deeper look. I subclass
    could use the previous default value if it does not use super() in
    constructor, but I can't see any reason to encourage it...

    Michaël

    Le 22/02/2019 à 11:08, edgar.sol...@web.de
    <mailto:edgar.sol...@web.de> a écrit :
    > hey Mike,
    >
    > can you tell me why you removed the default value for 'version'
    below? ..ede
    >
    > On 22.02.2019 08:35, jump-pilot-svn--- via Jump-pilot-devel wrote:
    >> Modified:
    core/trunk/src/com/vividsolutions/wms/AbstractWMSRequest.java
    >> ===================================================================
    >> ---
    core/trunk/src/com/vividsolutions/wms/AbstractWMSRequest.java  
    2019-02-21 07:21:47 UTC (rev 6133)
    >> +++
    core/trunk/src/com/vividsolutions/wms/AbstractWMSRequest.java  
    2019-02-22 07:35:16 UTC (rev 6134)
    > SNIP
    >> @@ -25,12 +21,11 @@
    >>   import com.vividsolutions.jump.util.FileUtil;
    >>   import com.vividsolutions.jump.workbench.Logger;
    >>   import
    com.vividsolutions.jump.workbench.ui.network.ProxySettingsOptionsPanel;
    >> -import
    com.vividsolutions.jump.workbench.ui.plugin.PersistentBlackboardPlugIn;
    >>
    >>   abstract public class AbstractWMSRequest implements WMSRequest {
    >>
    >>     protected WMService service;
    >> -  protected String version = WMService.WMS_1_1_1;
    >> +  protected String version;
    >>     protected HttpURLConnection con = null;
    >
    >
    > _______________________________________________
    > Jump-pilot-devel mailing list
    > Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
    >


    _______________________________________________
    Jump-pilot-devel mailing list
    Jump-pilot-devel@lists.sourceforge.net
    <mailto:Jump-pilot-devel@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel



_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to