gdi-by / downloadclient

GDI-BY DownloadClient
Apache License 2.0
2 stars 0 forks source link

Attribut matchAction="Any" im WFS-Filter korrigieren #107

Closed dstenger closed 6 years ago

dstenger commented 6 years ago

Das Attribut matchAction="ANY" in einem durch den DLS generierten WFS-Filter wird mit diesem Pull Request auf matchAction="Any" korrigiert. Somit sind Requests mit den WFS-Filter Operationen "PropertyIsEqualTo", "PropertyIsNotEqualTo", "PropertyIsLessThan", "PropertyIsGreaterThan", "PropertyIsLessThanOrEqualTo" und "PropertyIsGreaterThanOrEqualTo" nun, in Hinsicht auf das Attribut "matchAction", korrekt und es wird die erwartete Response zurückgegeben, wenn beispielsweise die deegree Demo [1] zum Testen verwendet wird.

Fehler wurde in https://github.com/gdi-by/downloadclient-dev/issues/1#issuecomment-410657215 beschrieben.

Der Fehler resultiert aus einem Bug in GeoTools. Dieser wurde dem GeoTools Projekt gemeldet: https://osgeo-org.atlassian.net/browse/GEOT-6092

[1] http://demo.deegree.org/utah-workspace/services/wfs?service=WFS&request=GetCapabilities

gdi-by commented 6 years ago

Hallo Herr Friebe, Wir können leider keinen Merge durchführen, da der Travis-CI-Build fehlerhaft ist. Bitte nachprüfen und nochmal committen.

Ferner ist der Wfs http://geoserv.weichand.de:8080/geoserver/wfs derzeit nicht erreichbar, weswegen einige Unit-Tests momentan nicht funktionieren werden. Wir versuchen das Problem schnellstmöglich zu beheben und werden dann nochmal Rückmeldung geben, wenn der Wfs wieder erreichbar ist.

viele Grüße, Tobias Rösgen

dstenger commented 6 years ago

Die beiden Tests schlagen fehl, da folgende URL nicht erreichbar ist: http://geoserv.weichand.de:8080/geoserver/wfs?service=wfs&acceptversions=2.0.0&request=GetCapabilities

Bitte geben Sie uns Bescheid, wenn die URL wieder erreichbar ist. Dann können wir den Travis CI Build neu starten.

gdi-by commented 6 years ago

Der WFS ist wieder erreichbar und damit sollte der Travis CI Build wieder erfolgreich durchlaufen

dstenger commented 6 years ago

Alle Tests laufen nun erfolgreich durch.