[ http://issues.apache.org/jira/browse/MYFACES-160?page=comments#action_62150 ] sean schofield commented on MYFACES-160: ----------------------------------------
I have been thinking about this some more. At no point should getNodeId() return null. It would appear this fix is unecessary. If this value is ever null at some point we need to understand why. During the rendering of tree2 this value is set and incremented properly (as near as I can tell.) The rendering of tree *depends* on this being the case (similar to dataTable and the rowIndex property.) > HtmlTree.isNodeSelected doesn't check for null node id > ------------------------------------------------------ > > Key: MYFACES-160 > URL: http://issues.apache.org/jira/browse/MYFACES-160 > Project: MyFaces > Type: Bug > Versions: 1.0.9 beta > Environment: All > Reporter: Rob Decker > Assignee: sean schofield > Priority: Minor > > Apparently UITreeData can return a null node id when getNodeId() is called. > HtmlTree.isNodeSelected() doesn't check for a null id and dumps a long strack > trace in the log. It would be nicer for the log file if checked for a null > value: > return (getNodeId() != null) ? getNodeId().equals(_selectedNodeId) : false; -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira
