Inline scripts (like for maps) are included in the rendered HTML from the WA API and need to be stripped, either upstream or downstream.

[cswendrowski]

In GitLab by @vbfischer on Jun 13, 2020, 13:49

Imported article that contained map. Map does not display.

Seeing the following error in console:

jquery.min.js:2 Uncaught ReferenceError: L is not defined
    at <anonymous>:2:17
    at b (jquery.min.js:2)
    at Pe (jquery.min.js:2)
    at S.fn.init.append (jquery.min.js:2)
    at JournalSheet._injectHTML (foundry.js:4018)
    at JournalSheet._render (foundry.js:3869)
    at async JournalSheet._render (foundry.js:4455)

Presumably related to the following source in the HTML of the article:

var labelIcon = new L.Icon({
    iconUrl: '/images/markers/marker-icon-transparent.png',
    iconSize: [150, 35],
    iconAnchor: [75, 15],
    popupAnchor: [0, -20],
})
...

[cswendrowski]

In GitLab by @vbfischer on Jun 13, 2020, 18:07

Presumably its because it doesn't include the leaflet lib

[cswendrowski]

In GitLab by @worldanvil on Jun 13, 2020, 23:34

It feels like trying to include support for maps from WA into FVTT is a bit of an overkill - there is a lot of logic there - but we certainly need to find a way to handle their display / or lack thereof gracefully.

[cswendrowski]

I didn't realize that WA articles could contain inline scripts like this. I'll definitely need to strip those out on the Foundry side - although perhaps they should not be included in the return from the API in the first place.

[cswendrowski]

In GitLab by @vbfischer on Jun 15, 2020, 11:14

Maybe a link to the map? (note, not the article itself, but directly to the map?)