You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Aditya Sharma (JIRA)" <ji...@apache.org> on 2019/03/09 07:35:00 UTC
[jira] [Commented] (OFBIZ-10759) Replace document.write()
occurrences with some legitimate code
[ 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)