Yurik has uploaded a new change for review.
https://gerrit.wikimedia.org/r/275731
Change subject: Disable geojson double-sanitization
......................................................................
Disable geojson double-sanitization
Disable mapbox's internal sanitization
after geojson has passed through the MW sanitizer.
Bug: T129189
Change-Id: Ic244ea0253b9980303fc02da3260a48496f4a7ef
---
M modules/kartographer.js
1 file changed, 9 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Kartographer
refs/changes/31/275731/1
diff --git a/modules/kartographer.js b/modules/kartographer.js
index a0eb7b7..0e318d0 100644
--- a/modules/kartographer.js
+++ b/modules/kartographer.js
@@ -115,6 +115,14 @@
return map;
};
+ mw.kartographer.dataLayerOpts = {
+ // Disable double-sanitization by mapbox's internal sanitizer
+ // because geojson has already passed through the MW internal
sanitizer
+ sanitizer: function (v) {
+ return v;
+ }
+ };
+
/**
* Create a new GeoJSON layer and add it to map.
*
@@ -123,7 +131,7 @@
*/
mw.kartographer.addDataLayer = function ( map, geoJson ) {
try {
- return L.mapbox.featureLayer( geoJson ).addTo( map );
+ return L.mapbox.featureLayer( geoJson,
mw.kartographer.dataLayerOpts ).addTo( map );
} catch ( e ) {
mw.log( e );
}
--
To view, visit https://gerrit.wikimedia.org/r/275731
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic244ea0253b9980303fc02da3260a48496f4a7ef
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Kartographer
Gerrit-Branch: master
Gerrit-Owner: Yurik <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
