Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Server WFS-T 1.1.0 wrong case for totalUpdated, totalInserted, totalDeleted #38558

Closed
elpaso opened this issue Sep 2, 2020 · 0 comments · Fixed by #38764
Closed

Server WFS-T 1.1.0 wrong case for totalUpdated, totalInserted, totalDeleted #38558

elpaso opened this issue Sep 2, 2020 · 0 comments · Fixed by #38764
Assignees
@elpaso
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server

Comments

@elpaso
Copy link
Contributor

elpaso commented Sep 2, 2020 •
edited
Loading

The server generated transaction respose uses a capital initial letter, while the specifications and the reference implementation use an initial lower case letter.

wrong

 <TransactionResponse xsi:schemaLocation="http://www.opengis.net/wfs http://schemas.opengis.net/wfs/1.1.0/wfs.xsd" version="1.1.0" xmlns:ogc="http://www.opengis.net/ogc" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://www.opengis.net/wfs">
 <TransactionSummary>
  <TotalUpdated>1</TotalUpdated>
 </TransactionSummary>
</TransactionResponse>

correct

 <TransactionResponse xsi:schemaLocation="http://www.opengis.net/wfs http://schemas.opengis.net/wfs/1.1.0/wfs.xsd" version="1.1.0" xmlns:ogc="http://www.opengis.net/ogc" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://www.opengis.net/wfs">
 <TransactionSummary>
  <totalUpdated>1</totalUpdated>
 </TransactionSummary>
</TransactionResponse>
@elpaso elpaso added Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server labels Sep 2, 2020
@elpaso elpaso self-assigned this Sep 14, 2020
elpaso added a commit to elpaso/QGIS that referenced this issue Sep 15, 2020
@elpaso
Fix server WFS-T 1.1.0 wrong case in transaction response
aa79a84 
Fixes qgis#38558
@elpaso elpaso mentioned this issue Sep 15, 2020
Merged
nyalldawson pushed a commit to nyalldawson/QGIS that referenced this issue Oct 5, 2020
@elpaso @nyalldawson
Fix server WFS-T 1.1.0 wrong case in transaction response
4fec0d3 
Fixes qgis#38558

(cherry picked from commit aa79a84)
nyalldawson pushed a commit that referenced this issue Oct 23, 2020
@elpaso @nyalldawson
Fix server WFS-T 1.1.0 wrong case in transaction response
6d17ac8 
Fixes #38558

(cherry picked from commit aa79a84)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elpaso elpaso

Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix server WFS-T 1.1.0 wrong case in transaction response elpaso/QGIS
1 participant
@elpaso