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.

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

Update to MapServer 8.0 syntax and add OGC Features API #7

Merged
merged 5 commits into from Jan 19, 2023

Conversation

geographika
Copy link
Member

@geographika geographika commented Sep 21, 2021

  • Remove keywords that have been deprecated and then removed in v8 of MapServer
  • Add layer and web metadata to enable OGC Features API
  • Adds FID columns to all datasets (which is why the DBF files have been modified).

@geographika geographika marked this pull request as ready for review February 22, 2022 16:53
@geographika
Copy link
Member Author

geographika commented Apr 1, 2022

Note - this should be merged when the MapServer 8.0 release comes out, or the demo will no longer work.

Update - it appears the demo site is already using a newer version of MapServer as the following URL is returning an error:

https://demo.mapserver.org/cgi-bin/mapserv?map=%2Fmapserver%2Fapps%2Fitasca_legend%2Fmap%2Fitasca1.map&layers=all&zoomsize=2&zoomdir=1&map_size=600+400

@geographika
Copy link
Member Author

Note to work correctly in MapServer 8 the TEMPLATE parameter needs to be modifiable from a URL.
See MapServer/MapServer#6469 (comment)

We'll likely need to expand run-time substitution a little - web->template would be one, layer->mask is another (there was a user request for that one). I was planning on adding those as a commit to this pull request.

@geographika geographika merged commit 6f406e0 into MapServer:main Jan 19, 2023
@sdlime
Copy link
Member

sdlime commented Jan 19, 2023

Note to work correctly in MapServer 8 the TEMPLATE parameter needs to be modifiable from a URL. See MapServer/MapServer#6469 (comment)

We'll likely need to expand run-time substitution a little - web->template would be one, layer->mask is another (there was a user request for that one). I was planning on adding those as a commit to this pull request.

I did add layer->mask ahead of 8.0, just missed web->template. Have you run into any others? The demo is pretty old-school and should be a good test.

@geographika
Copy link
Member Author

@sdlime - not so far, but I was unable to test much apart from manually updating the TEMPLATE parameter in the Mapfile. I'll hopefully be able to test further on a new OSGeoLive beta, and maybe add a link to the OGC Features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants