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

Feature/allow csw constraints #192

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

knudmoeller
Copy link

@knudmoeller knudmoeller commented Nov 20, 2017

  • extends ckanext.spatial.lib.CswService.getidentifiers() with constraints parameter, which will simply be passed on as the constraints parameter to owslib's get_records2()
  • extends ckanext.spatial.harvesters.CSWHarvester with new get_constraints() function
    • CSWHarvester simply returns an empty list (i.e., no constraint), but sub-classes can overwrite this
    • get_constraints() is being called from gather_stage()
  • Note: the current setup means that constraints can only be set if a sub-class of CSWHarvester is being used. This is exactly what I need. If constraints need to be used from plain CSWHarvester, additional code would be needed to allow entering the constraints as part of the harvester's config.

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

1 participant