search

Piwigo / piwigo-openstreetmap

OpenStreetMap integration for Piwigo
http://piwigo.org/ext/extension_view.php?eid=701
GNU General Public License v3.0
35 stars 35 forks source link

function osm_get_js -> more SSL in baselayerURL #157

Closed 7tonin closed 5 years ago

7tonin commented 5 years ago

Did confirm some https url to replace http ones. In reply to https://github.com/xbgmsharp/piwigo-openstreetmap/issues/137

7tonin commented 5 years ago

Second commit is not related to entitled topic°. (would it have needed a specific branch ?) (° topic title edited) It is a response to a bug, from global map, when opening the link to "see them all". The bug happened when negative lat or lon were asked.

For instance link : osmmap.php?min_lat=42.34&min_lng=-5.33&max_lat=49.55&max_lng=14.61 PHP Fatal error [Hacking attempt] the input parameter "min_lng" is not valid 1 check_input_parameter /home/antonin/yaya/pwg/plugins/piwigo-openstreetmap/include/functions_map.php(147) 2 osm_get_items /home/antonin/yaya/pwg/plugins/piwigo-openstreetmap/osmmap3.php(92) *3 include_once /home/antonin/yaya/pwg/osmmap.php(12)

samwilson commented 5 years ago

That sounds sensible.

Or, does it work if they're protocol-relative URLs? i.e. leave off the bit before the ://.