Skip to content

[Backport 2023.01.xx] #8167 add serverType option to wms & csw catalog entries (#9048)#9063

Merged
tdipisa merged 2 commits intogeosolutions-it:2023.01.xxfrom
allyoucanmap:issue_7809_2023.01.xx
Mar 29, 2023
Merged

[Backport 2023.01.xx] #8167 add serverType option to wms & csw catalog entries (#9048)#9063
tdipisa merged 2 commits intogeosolutions-it:2023.01.xxfrom
allyoucanmap:issue_7809_2023.01.xx

Conversation

@allyoucanmap
Copy link
Copy Markdown
Contributor

@allyoucanmap allyoucanmap commented Mar 29, 2023

This Backport includes the commit f66f6d1 from the PR #9055

if undefined or set to "geoserver", the behaviour is unchanged. If set to "no-vendor", then TILED query parameter isnt sen't to the remote wms server, and the "tiled" and "localizedLayerStyles" options are hidden in the layer parameters.

more vendor options can be sent/avoided this way. Improves interoperability when displaying and printing layers coming from mapproxy.

closes #7809
closes #7877
closes #7879

Description

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Feature

Issue

What is the current behavior?

#7809, #7877 & #7879

What is the new behavior?
can display and print layers coming from mapproxy, once "no-vendor" is set for the catalog entry.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

Other useful information

landryb and others added 2 commits March 29, 2023 15:28
…ies (geosolutions-it#9048)

 geosolutions-it#7809 failure to print WMS layers coming from mapproxy
 geosolutions-it#7877 set TILED=true only when printing layers from geoserver
 geosolutions-it#7879 set TILED=true only when loading layers from geoserver
---------

Co-authored-by: allyoucanmap <stefano.bovio@geosolutionsgroup.com>
Copy link
Copy Markdown
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok @allyoucanmap, this also contains changes from PR #9055 as mentioned in #9061 (comment)

@tdipisa tdipisa merged commit de8c606 into geosolutions-it:2023.01.xx Mar 29, 2023
@allyoucanmap
Copy link
Copy Markdown
Contributor Author

@tdipisa yes, both are inside this PR
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

3 participants