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

WMS resource preview with geoview extension not working in CAD and CAT (1.4) #138

Closed
Mbrownshoes opened this issue Jul 29, 2016 · 4 comments
Assignees
Milestone

Comments

@Mbrownshoes
Copy link
Contributor

Mbrownshoes commented Jul 29, 2016

WMS previews are working in prod
https://catalogue.data.gov.bc.ca/dataset/schools-k-12-with-francophone-indicators/resource/d62cc90c-d2d9-44f0-892b-b816454941b1

but the same resource as above isn't working in CAT or CAD
https://cad.data.gov.bc.ca/dataset/wms_test/resource/b7ea8058-ea40-424f-9de4-a6f343cdf823

https://cat.data.gov.bc.ca/dataset/wms_test/resource/596de543-8ac9-4265-a399-c287013b97df

This does work in DEMO (ckan-2.5.2 with leaflet plugin+mapquest tiles)
http://demo.ckan.org/dataset/wms-test/resource/110426c5-9272-42f4-9ae2-0142625be62a

We tried recreating the view and that did not work.

@kfishwick
Copy link
Contributor

kfishwick commented Aug 4, 2016

CKAN updated the demo site with a redeploy of the geoview extension (same version, but newer commit) and it is now broken.

image

The problem is that the geoview is failing when it attempts to parse the url parameters. If you remove the requst=GetCapabilities then the call works.

We are looking into a fix either in the bcgov extension or to fork the geoview repo again & apply a fix there.

@gjlawran
Copy link
Member

gjlawran commented Aug 6, 2016

Confirmed now working in CAD.

@kfishwick
Copy link
Contributor

Just a note, it seems like it was an intentional change in the geoview extension NOT to strip the parameters any more:

ckan/ckanext-geoview@2179cc4

So we won't be able to push this change back into the core extension. We may want to look at redefining the resource urls so that getCapabilities is not included as part of the url.

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

No branches or pull requests

4 participants