geopython / pywps

PyWPS is an implementation of the Web Processing Service standard from the Open Geospatial Consortium. PyWPS is written in Python.
https://pywps.org
MIT License
177 stars 117 forks source link

fix cdata deserialzation #555

Closed cehbrecht closed 3 years ago

cehbrecht commented 3 years ago

Overview

This PR fixes issue #553. It removes the CDATA tag when loading an input from json.

Related Issue / Discussion

553

Additional Information

https://github.com/roocs/rook/pull/72

Contribution Agreement

(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.04%) to 75.484% when pulling 86f96a0c3c7875df0c5c43a67279c347d2bde4af on cehbrecht:fix-553-json-with-cdata into 9fa56cc17bf545a9a583271d83affa5b479d3373 on geopython:pywps-4.2.

cehbrecht commented 3 years ago

Note that another, maybe cleaner option, would be to set the CDATA tag into the xml template, instead of the json representation.

I might have tried to do that originally but could not get all the kinks out. It might be more difficult though to code the necessary logic inside the template.

I have opened an issue for this #557.