fix manager transaction evaluation

Author: tomkralidis

pycsw/ogc/api/oapi.py

@@ -31,6 +31,7 @@
 import logging
 
 from pycsw import __version__
+from pycsw.core.util import str2bool
 from pycsw.ogc.api.util import yaml_load
 
 LOGGER = logging.getLogger(__name__)
@@ -429,7 +430,7 @@ def gen_oapi(config, oapi_filepath, mode='ogcapi-records'):
         }
     }
 
-    if config['manager'].get('transactions', False):
+    if str2bool(config['manager'].get('transactions', False)):
         LOGGER.debug('Transactions enabled; adding post')
 
         path['post'] = {
@@ -499,7 +500,7 @@ def gen_oapi(config, oapi_filepath, mode='ogcapi-records'):
         }
     }
 
-    if config['manager'].get('transactions', False):
+    if str2bool(config['manager'].get('transactions', False)):
         LOGGER.debug('Transactions enabled; adding put/delete')
 
         path['put'] = {

pycsw/ogc/api/records.py

@@ -286,7 +286,7 @@ def landing_page(self, headers_, args):
               'type': 'application/json',
               'title': 'Main collection',
               'href': f"{self.config['server']['url']}/collections/metadata:main"
-            },{
+            }, {
               'rel': 'http://www.opengis.net/def/rel/ogc/1.0/ogc-catalog',
               'type': 'application/json',
               'title': 'Record catalogue collection',
@@ -912,7 +912,7 @@ def manage_collection_item(self, headers_, action='create', item=None, data=None
         :returns: tuple of headers, status code, content
         """
 
-        if not self.config['manager']['transactions']:
+        if not str2bool(self.config['manager'].get('transactions', False)):
             return self.get_exception(
                     405, headers_, 'InvalidParameterValue',
                     'transactions not allowed')

tests/functionaltests/suites/oarec/test_oarec_functional.py

@@ -31,6 +31,7 @@
 #
 # =================================================================
 
+from copy import deepcopy
 import json
 from xml.etree import ElementTree as etree
 
@@ -357,6 +358,33 @@ def test_json_transaction(config, sample_record):
 
     assert status == 404
 
+    config2 = deepcopy(config)
+    config2['manager']['transactions'] = False
+
+    api = API(config2)
+    request_headers = {
+        'Content-Type': 'application/json'
+    }
+
+    # insert record
+    headers, status, content = api.manage_collection_item(
+        request_headers, 'create', data=sample_record)
+
+    assert status == 400
+
+    config2['manager']['transactions'] = 'false'
+
+    api = API(config)
+    request_headers = {
+        'Content-Type': 'application/json'
+    }
+
+    # insert record
+    headers, status, content = api.manage_collection_item(
+        request_headers, 'create', data=sample_record)
+
+    assert status == 400
+
 
 def test_xml_transaction(config):
     api = API(config)