[jira] [Comment Edited] (OFBIZ-5255) Error in fieldlookup.js

["Jacques Le Roux (JIRA)" <ji...@apache.org>]

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

Jacques Le Roux edited comment on OFBIZ-5255 at 1/18/15 11:29 AM:
------------------------------------------------------------------

Thanks Pranay,

Skip w/o news from you in a week I will close, thanks!


was (Author: jacques.le.roux):
Skip w/o news from you in a week I will close, thanks!

> Error in fieldlookup.js
> -----------------------
>
>                 Key: OFBIZ-5255
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-5255
>             Project: OFBiz
>          Issue Type: Bug
>          Components: ALL APPLICATIONS
>    Affects Versions: Release Branch 12.04
>         Environment: All
>            Reporter: Skip Dever
>            Priority: Minor
>
> On line 221 in fieldlookup.js is the statement:
> if (ajaxurl != "" && showDescription != "")
> This logic is incorrect because showDescription is normally a boolean and if you pass showDescription="false" to the macro, showDescription != "" evaluates to false.
> I am not sure what the author was trying to test for, but I modified it to either comment out showDescription != "", or change it to:
> if (ajaxurl != "" && !(showDescription === ""))
> The author needs to test this with a macro that passes showDescription="false".  For example, in ordermgr\entry\checkinits.ftl, modify line 104 and add showDescription="false" as a macro argument.  
> Doing so will disable the autocompletion until a modification is made to fieldlookup.js as above.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)