[jira] [Commented] (OFBIZ-10759) Replace document.write() occurrences with some legitimate code

[Nicolas Malin (Jira)]

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

Aditya Sharma commented on OFBIZ-10759:
---------------------------------------

Removed js code and added Js files in LookupDecorator.
Added Js library files as there were some errors in console logs

Done for trunk at r1855083 and release18.12 at r1855087

Steps to verify:
 # Login to example component.
 # Navigate to [Form Widget Examples|https://localhost:8443/example/control/FormWidgetExamples]
 # Try using Popup lookup field (old way, maybe be useful in some cases, notably if javascript is not enabled)

It should work as expected.

 

Removed document.write.lastFocusedName as code works without it

Done for trunk plugins at r1855084 and release18.12 at r1855092

> Replace document.write() occurrences with some legitimate code
> --------------------------------------------------------------
>
>                 Key: OFBIZ-10759
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10759
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL APPLICATIONS
>    Affects Versions: Trunk
>            Reporter: Aditya Sharma
>            Assignee: Aditya Sharma
>            Priority: Minor
>
> Using document.write() is considered a bad practice.
>  
> Refer:
> [https://blog.dareboost.com/en/2016/09/avoid-using-document-write-scripts-injection/]
> [https://stackoverflow.com/questions/802854/why-is-document-write-considered-a-bad-practice]
>  

--
This message was sent by Atlassian JIRA
(v7.6.3#76005)