ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacques Le Roux (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OFBIZ-9465) Replace Inline Javascript with External Javascript
Date Mon, 04 Dec 2017 12:41:00 GMT

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

Jacques Le Roux commented on OFBIZ-9465:
----------------------------------------

One way to check inline Javascript to migrate is to use the browser console and see where
CSP complains about inline script, for me in French it says:
bq. Les paramètres de la page ont empêché le chargement d’une ressource à self
so could be someting like
bq. Page settings have prevented the loading of a resource to self
in English. Anyway it's pretty obvious when you look into console



> Replace Inline Javascript with External Javascript 
> ---------------------------------------------------
>
>                 Key: OFBIZ-9465
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9465
>             Project: OFBiz
>          Issue Type: Improvement
>            Reporter: Aditya Sharma
>            Assignee: Aditya Sharma
>             Fix For: Upcoming Release
>
>         Attachments: OFBIZ-9465.patch, OFBIZ-9465.patch, OFBIZ-9465.patch, OFBIZ-9465.patch,
OFBIZ-9465_example.patch, OFBIZ-9465_example.patch
>
>
> As discussed in http://ofbiz.markmail.org/thread/ti54omjdt4fy662m inline javascript for
various macros like renderLookupField, makeHyperlinkString,shortDateInput etc in HtmlFormMacroLibrary.ftl
should be replaced with a generic code written as external javascript. Generic Javascript
code should be written in OfbizUtil.js and appropriate classes or identifiers should be used
for applying that. Any additional data can be passed with HTML 5 custom data attribute (https://www.w3.org/TR/2011/WD-html5-20110525/elements.html#embedding-custom-non-visible-data-with-the-data-attributes).
> As suggested by Jacques in  https://s.apache.org/y8P1 will introduce require.js for the
split javascript files in a sub-task.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message