search

IDgis / geoide-commons

0 stars 0 forks source link

LegendGraphicUrl url encoden #274

Open HermanAssink opened 7 years ago

HermanAssink commented 7 years ago

Gaat fout bij bijv. java.net.URISyntaxException: Illegal character in query at index 139: http://odru.gispubliek.nl/deegree-webservices/services/wms_archeologie?request=GetLegendGraphic&version=1.1.1&service=WMS&layer=Archeologie Bunnik&style=archeologie_bunnik&format=image/png

HermanAssink commented 7 years ago

(spatie tussen Archeologie en Bunnik) Kan deze fout ook met andere url's optreden? Graag ontwikkeltest met de services van ODRU

kad-velsl commented 7 years ago

Nog even bespreken of een legend plaatje direct door de client moet worden opgevraagd of via de Image Controller (dus via de play applicatie). In eerste geval ws geen of slecht gedeeltelijk encoding nodig in het tweede geval wel volledige encoding.

HermanAssink commented 7 years ago

Waarom zou je de verantwoordelijk voor / plek waar het legenda plaatje wordt opgevraagd wijzigen nav url-encoden?

kad-velsl commented 7 years ago

De vraag is andersom. De wijze waarop de url opgevraagd moet worden is afhankelijk van de manier/plek waar het plaatje wordt opgevraagd.

HermanAssink commented 7 years ago

Wat is erop tegen om altijd url encoding toe te passen?

kad-velsl commented 7 years ago

Bij encoden wordt de space vervangen door %20, dit wordt verderop in de programmatuur weer gelezen als een space. Encoding lost het probleem niet op. Het probleem is weg als de space wordt vervangen door een +.

kad-velsl commented 7 years ago

Eind conclusie: de legendGraphicUrl is fout, we gaan geen programmatuur inbouwen om foutieve url's op te knappen. Wel een alt tekst als het legenda plaatje niet beschikbaar is.

HermanAssink commented 7 years ago

Het is verwarrend dat je in de Composer wel een plaatje krijgt, maar de viewer meldt "Legenda niet beschikbaar". Kan de melding in de viewer worden gewijzigd in "Fout [foutcode] bij ophalen legenda"?

kad-velsl commented 7 years ago

Ik heb de alternatieve tekst als alt code in de html getypt, op dat punt heb ik geen informatie over de fout die is opgetreden.

kad-velsl commented 7 years ago

Voorlopig allen melding Fout bij ophalen legenda met andere stijl

HermanAssink commented 7 years ago

Een duidelijker melding naar gebruikers is gewenst, want het is verwarrend dat de legenda wel in de Composer, maar niet in de Viewer wordt getoond. Of dat de Composer wordt aangepast zodat het gedrag in Composer en Viewer hetzelfde zijn

HermanAssink commented 7 years ago

Blijft open in milestone Uitbreidingen.