Uploaded image for project: 'MyFaces Tomahawk'
  1. MyFaces Tomahawk
  2. TOMAHAWK-882

missing setNodeId(null) in processDecodes()

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 1.1.5-SNAPSHOT
    • 1.1.8
    • Tree2
    • None

    Description

      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).

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. TOMAHAWK-557 Tree2 renderer (HtmlTreeRenderer) prints invalid clientId when a button/link with "immediate='true'" is fired

          • Major - Major loss of function.
          • Closed

          Activity

            People

              lu4242 Leonardo Uribe
              haubnert Tobias Haubner
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: