[jira] [Commented] (OFBIZ-11882) rename a few map files

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-11882) rename a few map files

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-11882?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17159260#comment-17159260 ]

Aditya Sharma commented on OFBIZ-11882:
---------------------------------------

Hi Jacques,

For most of the libraries we have added/upgraded we simply download the official packages and added/replaced. We tried to ensure that we do least or no changes to original packages. Also, we haven't minified any of the files, most of the minified files are from the official release.

I think for jQuery I have downloaded the files through the links from the official blog and simply replaced them.
[https://blog.jquery.com/2020/05/04/jquery-3-5-1-released-fixing-a-regression/]

But, it seems the path through [download|https://jquery.com/download/] page or [GitHub|https://github.com/jquery/jquery/releases/tag/3.5.1] would have been a much better option.

I noticed that the minified files for jQuery don't come with a sourceMappingUrl as a comment in them
[https://github.com/jquery/jquery/issues/4166]
[|https://blog.jquery.com/2014/01/24/jquery-1-11-and-2-1-released/]
|We've removed the comment on purpose, see [https://blog.jquery.com/2014/01/24/jquery-1-11-and-2-1-released/]. There are many problems with including the comment, one of them being that different CDNs use different file names so the comment would have to be different for different CDNs and that kills security features like Subresource Integrity.
You can still attach the map manually via DevTools, at least Chrome supports that.|
 
We will have to add the sourceMappingUrl to the minified file manually and add the source map file.

> rename a few map files
> ----------------------
>
>                 Key: OFBIZ-11882
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11882
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ecommerce
>    Affects Versions: Release Branch 17.12, Trunk
>            Reporter: Alex Bodnaru
>            Assignee: Jacques Le Roux
>            Priority: Trivial
>         Attachments: ecommerce-js.patch, ecommerce-js2.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> tells a few files to be renamed, to suppress warnings in the browser console.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)