[jira] Commented: (TOMAHAWK-882) missing setNodeId(null) in processDecodes()

["Grant Smith (JIRA)" <de...@myfaces.apache.org>]

    [ https://issues.apache.org/jira/browse/TOMAHAWK-882?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12496937 ] 

Grant Smith commented on TOMAHAWK-882:
--------------------------------------

Tobias,

This issue is marked as "Patch Available", yet there is no patch attached. Was that an oversight, or should the status be changed ?

> missing setNodeId(null) in processDecodes()
> -------------------------------------------
>
>                 Key: TOMAHAWK-882
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-882
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: Tree2
>    Affects Versions: 1.1.5-SNAPSHOT
>            Reporter: Tobias Haubner
>            Priority: Critical
>
> At the end of processDecodes() in HtmlTree, nodeId is not set back to null. 
> This leads to the tree returning the id of a node, not the id of the tree, when calling getClientId() after processDecodes().
> This e.g. affects AJAX calls from within the tree.
> processUpdates() and processValidators() are correct.
> To avoid such errors in the future it would be even better to change TreeWalkerBase to set nodeId=null at the end of its walk (i.e. in the next() method if next() is false).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.