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

Initial version of OGC-Capabilities handling rewrite #796

Merged
merged 23 commits into from
Feb 3, 2022

Conversation

ZakarFin
Copy link
Member

@ZakarFin ZakarFin commented Jan 28, 2022

The idea is moving the capabilities xml-parsing etc to service-capabillities and likely get rid of saving the XML in the database in the process. Currently everything related to WMTS-layers outside of the printing functionality should use the new service included here. Next steps are moving capabilities related code for WMS and WFS layers to the same pipeline and possibly add a parser for ArcGIS json listing for layers.

Also the ViewHelper is now used to get a list of system projections in the migration for example. That might lead to problems later so we might want to fix it asap. Luckily it's a Java-migration so migration checksums shouldn't be an issue for later mofication.

Note! Also adds a dependency-free drop-in'ish replacement for fi.nls.oskari.util.XmlHelper as org.oskari.xml.XmlHelper

TODO cleanup (addressed in #799):

  • We might be able to modify V2_5_1__update_WMTS_capabilities to do nothing
  • service-map has a lot of duplicated code currently for WMTS-parsing
  • service-admin has references to the old code in LayerCapabilitiesHelper

@ZakarFin ZakarFin changed the title Initial version of OGC(and other services) Capabilities handling Initial version of OGC-Capabilities handling rewrite Jan 28, 2022
@ZakarFin ZakarFin marked this pull request as draft January 28, 2022 12:48
@ZakarFin ZakarFin changed the base branch from develop to capa-rewrite February 3, 2022 15:32
@ZakarFin ZakarFin added this to the 2.7.0 milestone Feb 3, 2022
@ZakarFin ZakarFin marked this pull request as ready for review February 3, 2022 15:33
@ZakarFin ZakarFin merged commit e08f140 into oskariorg:capa-rewrite Feb 3, 2022
@ZakarFin ZakarFin deleted the capabilities branch February 3, 2022 15:41
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.

1 participant