Incorrect JS *.map files

Created on 8 November 2023, about 1 year ago
Updated 16 November 2023, about 1 year ago

I think your map files need to be regenerated. Perhaps these were from a previous version ?

In leaflet.js.map it references "sources":["../src/core/Util.js" which does not exist.
The /src folder does not exist, only the /dist folder.

This caused an issue for me when I was trying to do some debugging, as chrome dev tools was pointing me to a files that do not exist.
Once these map files were deleted, I could debug my issue correctly.

Probably not a common issue. Marking as minor.

🐛 Bug report
Status

Closed: works as designed

Version

10.2

Component

Code

Created by

🇬🇧United Kingdom 8bitplateau

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @8bitplateau
  • 🇬🇧United Kingdom 8bitplateau

    Have just download leaflet Leaflet 1.9.4 and I see the map files are in there, so these are from the library itself, and are correct when downloading from github. Still, misleading when debugging, not sure why they would be in their dist package.
    Feel free to close :)

  • 🇩🇰Denmark ressa Copenhagen

    Great that it works, and thanks for reporting back @8bitplateau. You should be able to close the issue yourself :-)

  • Status changed to Needs review about 1 year ago
  • 🇮🇹Italy itamair

    thanks @8bitplateau ... well it seems that Drupal Leaflet module indeed need only the Leaflet Js files mapped from the leaflet.libraries.yml file, so we could remove all those others not mapped in the list folder.
    Could you test & QA the attached patch if it could work for you and for this issue as a whole?

  • Status changed to Closed: works as designed about 1 year ago
  • 🇮🇹Italy itamair

    Let's close this, as not relevant.

Production build 0.71.5 2024