Geonovum / pdokkaart

API and the wizard for PDOKKaart
http://geonovum.github.com/pdokkaart/
14 stars 18 forks source link

Is PDOK Kaart backwards compatible? #170

Open J-kad opened 9 years ago

J-kad commented 9 years ago

Goedemiddag,

Is de laatste versie van PDOK Kaart backwards compatible met de eerder versie van PDOK Kaart (versie van PDOK)?

Alvast bedankt!

rduivenvoorde commented 9 years ago

Hoewel er niet specifiek op gelet is, verwacht ik het wel.

Laat ajb weten wanneer het niet zo is, dan kunnen we dat eventueel nog fixen, zodat dat wel zo blijft.

J-kad commented 9 years ago

Wij zullen er naar kijken en laten weten als het niet zo is. Bedankt voor je antwoord!

J-kad commented 9 years ago

Het lijkt er helaas op dat de laatste versie van PDOK Kaart niet backward compatible is. De test is als volgt uitgevoerd:

Wij hebben de nieuwe versie op een test-omgeving gezet. Op de huidige versie (kaart.pdok.nl) hebben wij daarna een kaart gegenereerd met 1 extra laag (natura2000). De URL die gegenereerd wordt is aangepast naar de URL van de testversie. Vervolgens zie je wel een kaartje van NL, maar zonder zoommogelijkheid en keuze voor kaartlagen.

Kan dit aangepast worden?

Alvast bedankt!

ndkv commented 9 years ago

Zie je dit: http://geonovum.github.io/pdokkaart/api/api.html?zoom=2&baselayer=BRT&loc=142500%2C%20470000&pdoklayers=BRT%2CBRTTIJDELIJK%2CBRTGRIJSTIJDELIJK%2CBRTPASTEL%2CLUFO%2CNATURA2000_WMS&markersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-markers.js&layersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-layers.js

On 5/22/2015 2:33 PM, Kaart01 wrote:

Het lijkt er helaas op dat de laatste versie van PDOK Kaart niet backward compatible is. De test is als volgt uitgevoerd:

Wij hebben de nieuwe versie op een test-omgeving gezet. Op de huidige versie (kaart.pdok.nl) hebben wij daarna een kaart gegenereerd met 1 extra laag (natura2000). De URL die gegenereerd wordt is aangepast naar de URL van de testversie. Vervolgens zie je wel een kaartje van NL, maar zonder zoommogelijkheid en keuze voor kaartlagen.

Kan dit aangepast worden?

— Reply to this email directly or view it on GitHub https://github.com/Geonovum/pdokkaart/issues/170#issuecomment-104649006.

J-kad commented 9 years ago

Ja dat krijg je inderdaad te zien, zonder extra (kaart) mogelijkheden.

ndkv commented 9 years ago

En welke commit heb je in testomgeving gezet?

On 26 May 2015 09:10:13 CEST, Kaart01 notifications@github.com wrote:

Ja dat krijg je inderdaad te zien, zonder extra (kaart) mogelijkheden.


Reply to this email directly or view it on GitHub: https://github.com/Geonovum/pdokkaart/issues/170#issuecomment-105420954

Sent from my Android device with K-9 Mail. Please excuse my brevity.

J-kad commented 9 years ago

De commit van 31 maart 2015

rduivenvoorde commented 9 years ago

Ok, dit is een keiharde bug, die gewoon nooit is opgemerkt...

Gefixed in: 70c3d0abd6d0a8686572cbaca6fb9c252b3fdf8c

Ik heb uitgerold op http://zuidt.nl/pdokkaart. Graag retest

JeroenBaltussen commented 9 years ago

Compliment voor de fix. Hopelijk hiermee release ready for use.

J-kad commented 9 years ago

Mooi om te horen, wij zullen dit gaan testen.

J-kad commented 9 years ago

Ik zie in de laatste versie overigens enkel de PDOK achtergrondkaart als standaardlaag in de laagfunctionaliteit staan. In de huidige (productie) versie staan ook nog andere lagen. Kunnen deze ook getoond worden? Of moet ik hiervoor een nieuw issue aanmaken?

rduivenvoorde commented 9 years ago

ik heb een nieuw issue aangemaakt:

https://github.com/Geonovum/pdokkaart/issues/174

en zal deze afsluiten. Nieuwe issues graag gewoon melden.

J-kad commented 9 years ago

@rduivenvoorde @JeroenBaltussen helaas is de laatste commit getest en is PDOK kaart nog niet backwards compatible.. Bevindingen van de ontwikkelaar zijn:

rduivenvoorde commented 9 years ago

@beheerPDOK @JeroenBaltussen

er moet een duidelijker omschrijving worden gegeven of reproduceerbare scenario's van bugs.

WAT werkt niet bij het kopieren van de kaart? Een snelle test bij werkt hier gewoon. WELKE browsers/versies worden gebruikt. Liefst met gebruikte code of screendumps.

J-kad commented 9 years ago

@rduivenvoorde @JeroenBaltussen

Was inderdaad wat kort omschreven, excuses :) Hierbij een duidelijkere omschrijving:

Ik stuur je zo je via de mail twee bestanden waarmee de embedded kaart op onze test-omgeving mee is getest en versie van onze productie-omgeving. De versie van productie werkt wel, maar op test zie je de eerder gemelde problemen. In een iframe zie je geen laagselectie en werkt zoomen niet. Via object en broncode werkt de hele kaart niet.

De rechtstreekse link die is gebruikt is: http://test.kaart.pdok.nl/api/api.html?zoom=7&baselayer=BRT&loc=193713.96%2C%20465749.6&pdoklayers=BRT%2CBRTTIJDELIJK%2CBRTGRIJSTIJDELIJK%2CBRTPASTEL%2CLUFO&markersdef=http%3A%2F%2Ftest.kaart.pdok.nl%2Fapi%2Fjs%2Fpdok-markers.js&layersdef=http%3A%2F%2Ftest.kaart.pdok.nl%2Fapi%2Fjs%2Fpdok-layers.js&mloc=194160.27%2C465870.5&titel=Werkplek&tekst=Op%20de%206e%20verdieping&mt=mt14

Hier werkt de laagselectie en zoomen niet.

rduivenvoorde commented 9 years ago

@raymondnijssen

ik denk dat ik zie wat het probleem is....

in de nieuwe versie zijn er nieuwe checkboxen (EN daarbij parameters) bijgekomen: &showzoom=true <= wel of niet de blauwe zoomknoppen +/- tonen(!) &shownavigation=true <= wel of niet een navigation control aanmaken(!)

Probleem hierbij: 1) Die laatste is wat mij betreft een naamgevingsprobleem: het is niet SHOWnavigation, maar 'navigation': het bepaalt niet of iets wel of niet zichtbaar is, maar of navigatie (== pannen en zoomen) wel werkt of niet. 2) blijkbaar is zowel showzoom als shownavigation standaard 'False', dus ze werken/zijn pas zichtbaar als je ze expliciet op true zet. Dit breekt idd backwards compatibilitiet. Ze zouden standaard op True moeten staan, en je kunt ze expliciet uitzetten.

Beide lijken kleine fixes, ik kijk even of en wanneer ik tijd heb

rduivenvoorde commented 9 years ago

De fout met de mij toegestuurde html is het niet juist hebben gekopieerd van de HTML: er is een oude html file gebruikt met een niet volledige head-deel

rduivenvoorde commented 9 years ago

@beheerPDOK @milovanderlinden @raymondnijssen plz retest

ik heb die pdokextenders file overal uitgesloopt en de 3 classen die erin zitten uiteindelijk toch geappend aan de api zelf (zodat ze niet VOOR de api zitten).

ik heb hier rws apps, pdokkaart en vegetatielegger getest. Laat ajb weten wanneer ik dit issue kan sluiten.

J-kad commented 9 years ago

@rduivenvoorde n.a.v. ons gesprek van vrijdag hebben wij opnieuwe getest op de laatste commit (van vrijdag). De kaart wordt getoond en het zoomen werkt. De laagselectie wordt echter nog niet weergegeven na het openen van de link ( in iframe, link en object en broncode). Link is:

http://test.kaart.pdok.nl/api/api.html?zoom=8&baselayer=BRT&loc=158869.92%2C%20438200.96&pdoklayers=BRT%2CBRTTIJDELIJK%2CBRTGRIJSTIJDELIJK%2CBRTPASTEL%2CLUFO&markersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-markers.js&layersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-layers.js&features=%3Ckml%20xmlns%3D%22http%3A%2F%2Fearth.google.com%2Fkml%2F2.0%22%3E%3CFolder%3E%3CPlacemark%3E%3Cname%3E%26amp%3Bnbsp%3B%3C%2Fname%3E%3Cdescription%3E%26amp%3Bnbsp%3B%3C%2Fdescription%3E%3CPoint%3E%3Ccoordinates%3E5.419236305233392%2C51.92768141165513%3C%2Fcoordinates%3E%3C%2FPoint%3E%3CExtendedData%3E%3CData%20name%3D%22styletype%22%3E%3Cvalue%3Emt0%3C%2Fvalue%3E%3C%2FData%3E%3C%2FExtendedData%3E%3C%2FPlacemark%3E%3CPlacemark%3E%3Cname%3E%26amp%3Bnbsp%3B%3C%2Fname%3E%3Cdescription%3E%26amp%3Bnbsp%3B%3C%2Fdescription%3E%3CPoint%3E%3Ccoordinates%3E5.459089495171703%2C51.92271085488218%3C%2Fcoordinates%3E%3C%2FPoint%3E%3CExtendedData%3E%3CData%20name%3D%22styletype%22%3E%3Cvalue%3Emt0%3C%2Fvalue%3E%3C%2FData%3E%3C%2FExtendedData%3E%3C%2FPlacemark%3E%3C%2FFolder%3E%3C%2Fkml%3E

rduivenvoorde commented 9 years ago

@beheerPDOK dit was een foutje in de defaultwaarden. Ik heb die nu zo aangepast dat de url wel zou moeten werken. Plz retest