[jbosstools-issues] [JBoss JIRA] (JBIDE-9932) Fix VpeDomMapping.getNodeMappingAtVisualNode to use HashMap.get instead of iteration

[Denis Maliarevich (Commented) (JIRA)]

    [ https://issues.jboss.org/browse/JBIDE-9932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12650853#comment-12650853 ] 

Denis Maliarevich commented on JBIDE-9932:
------------------------------------------

Now the code is the following:
{code}
public VpeNodeMapping getNodeMappingAtVisualNode(nsIDOMNode visualNode) {
VpeNodeMapping nodeMapping = null;
/*
 * https://issues.jboss.org/browse/JBIDE-9932
 * Replaced with map.
 */
if (visualNode != null) {
nodeMapping = visualMap.get(visualNode);
}
return nodeMapping;
}
{code}
                
> Fix VpeDomMapping.getNodeMappingAtVisualNode to use HashMap.get instead of iteration
> ------------------------------------------------------------------------------------
>
>                 Key: JBIDE-9932
>                 URL: https://issues.jboss.org/browse/JBIDE-9932
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: Visual Page Editor core
>    Affects Versions: 3.3.0.M4
>            Reporter: Denis Golovin
>            Assignee: Denis Maliarevich
>             Fix For: 3.3.0.M5
>
>         Attachments: JBIDE-9932-Draft1.patch, JBIDE-9932-Draft2-wTests.patch, JBIDE-9932-Draft3.patch, JBIDE-9932-Final.patch
>
>
> nsIDOMNode.appendChild(nsIDOMNode child) actually adds copy of child provided and returns it as result. VPE saves in map a child passed as parameter instead of actually added node returned from appendChild method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira