Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 1.0
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      152

      Description

      Don't laugh but I am attempting to utilize datatable in a recursive manner. I
      actually got it to work but it required a minor tweak to UIData. It would be
      nice if this tweak could get incorperated into the JSF-RI so I don't have to
      extend it and create my own setRowIndex() implementation.

      The tweak only involves making UIData a better component citizen by restoring
      the value that previously existed in "var" to the request once UIData is done
      with "var". Whether or not you believe recursive datatables are a bad idea or
      not I believe preserving "var" once UIData is done with it could be argued as
      useful functionality in and of itself. I've included a patch.

      Mike

        Activity

        Hide
        youngm added a comment -

        Created an attachment (id=79)
        Patch to preserve var attribute

        Show
        youngm added a comment - Created an attachment (id=79) Patch to preserve var attribute
        Hide
        youngm added a comment -

        Changing to "ENHANCEMENT" since it seems it was being ignored as a "PATCH".

        Show
        youngm added a comment - Changing to "ENHANCEMENT" since it seems it was being ignored as a "PATCH".
        Hide
        jayashri added a comment -

        Transferring ownership to Ed

        Show
        jayashri added a comment - Transferring ownership to Ed
        Hide
        Ed Burns added a comment -

        Seems to make sense. I'd make the ivar transient. Let me try it out and see if
        it breaks anything.

        Show
        Ed Burns added a comment - Seems to make sense. I'd make the ivar transient. Let me try it out and see if it breaks anything.
        Hide
        Ed Burns added a comment -

        Fix checked in.

        Show
        Ed Burns added a comment - Fix checked in.
        Hide
        youngm added a comment -

        Is this fix going to be added to JSF_1_1_ROLLING?

        Show
        youngm added a comment - Is this fix going to be added to JSF_1_1_ROLLING?
        Hide
        Manfred Riem added a comment -

        Closing out issue

        Show
        Manfred Riem added a comment - Closing out issue

          People

          • Assignee:
            Ed Burns
            Reporter:
            youngm
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: