[jira] [Commented] (OFBIZ-5279) Wrong to use !jQuery('#content-messages') in \framework\images\webapp\images\selectall.js line #354

[Nicolas Malin (Jira)]

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

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

Could you please rather provide a patch based on trunk HEAD?

Except if you confirme that I just have to add the line

jQuery('#content-messages').html('');

above the 2
jQuery('#content-messages').removeClass('errorMessage').fadeIn("fast");
lines

right?
               

> Wrong to use !jQuery('#content-messages') in \framework\images\webapp\images\selectall.js line #354
> ---------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-5279
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-5279
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Wei Zhang
>
> I think we should use
> if(jQuery('#content-messages').length==0)
> insead of
> if(!jQuery('#content-messages'))
> in framework\images\webapp\images\selectall.js line #354.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira