From 28618034b8ce05730b2adeb5023554b7e8c03dce Mon Sep 17 00:00:00 2001
From: Simon Seyock <8100558+simonseyock@users.noreply.github.com>
Date: Wed, 11 Sep 2024 18:24:26 +0200
Subject: [PATCH] feat: add version parameter to WMSFacade provider (#1806)

---
 docs/source/data-publishing/ogcapi-maps.rst |  9 +++++----
 pygeoapi/provider/wms_facade.py             | 12 ++++++++----
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/docs/source/data-publishing/ogcapi-maps.rst b/docs/source/data-publishing/ogcapi-maps.rst
index 30e12b1d1..03846eeaa 100644
--- a/docs/source/data-publishing/ogcapi-maps.rst
+++ b/docs/source/data-publishing/ogcapi-maps.rst
@@ -51,7 +51,7 @@ Currently supported style files (`options.style`):
 .. code-block:: yaml
 
    providers:
-       - type: map 
+       - type: map
          name: MapScript
          data: /path/to/data.shp
          options:
@@ -59,7 +59,7 @@ Currently supported style files (`options.style`):
              layer: foo_name
              style: ./foo.sld
          format:
-            name: png 
+            name: png
             mimetype: image/png
 
 WMSFacade
@@ -71,14 +71,15 @@ required.  An optional style name can be defined via `options.style`.
 .. code-block:: yaml
 
    providers:
-       - type: map 
+       - type: map
          name: WMSFacade
          data: https://demo.mapserver.org/cgi-bin/msautotest
          options:
              layer: world_latlong
              style: default
+             version: 1.3.0
          format:
-               name: png 
+               name: png
                mimetype: image/png
 
 
diff --git a/pygeoapi/provider/wms_facade.py b/pygeoapi/provider/wms_facade.py
index 8771b6395..fa3ffd064 100644
--- a/pygeoapi/provider/wms_facade.py
+++ b/pygeoapi/provider/wms_facade.py
@@ -84,7 +84,9 @@ def query(self, style=None, bbox=[-180, -90, 180, 90], width=500,
 
         self._transparent = 'TRUE'
 
-        if crs in [4326, 'CRS;84']:
+        version = self.options.get('version', '1.3.0')
+
+        if crs in [4326, 'CRS;84'] and version == '1.3.0':
             LOGGER.debug('Swapping 4326 axis order to WMS 1.3 mode (yx)')
             bbox2 = ','.join(str(c) for c in
                              [bbox[1], bbox[0], bbox[3], bbox[2]])
@@ -106,12 +108,14 @@ def query(self, style=None, bbox=[-180, -90, 180, 90], width=500,
         if not transparent:
             self._transparent = 'FALSE'
 
+        crs_param = 'crs' if version == '1.3.0' else 'srs'
+
         params = {
-            'version': '1.3.0',
+            'version': version,
             'service': 'WMS',
             'request': 'GetMap',
             'bbox': bbox2,
-            'crs': CRS_CODES[crs],
+            crs_param: CRS_CODES[crs],
             'layers': self.options['layer'],
             'styles': self.options.get('style', 'default'),
             'width': width,
@@ -128,7 +132,7 @@ def query(self, style=None, bbox=[-180, -90, 180, 90], width=500,
         else:
             request_url = '?'.join([self.data, urlencode(params)])
 
-        LOGGER.debug(f'WMS 1.3.0 request url: {request_url}')
+        LOGGER.debug(f'WMS {version} request url: {request_url}')
 
         response = requests.get(request_url)