MyFaces Core
  1. MyFaces Core
  2. MYFACES-2370

'invokeOnComponent' method in UIData does not properly process <h:column> header/footer facets

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.7
    • Fix Version/s: 1.2.8
    • Component/s: General
    • Labels:
      None

      Description

      A bug that was originally reported to the RichFaces project here (includes simple test case)...

      https://jira.jboss.org/jira/browse/RF-7700

      ...was identifed by the RichFaces team as being caused by failure to process <h:column> header/footer facets in 'invokeOnComponent'. The RichFaces team reported this to the Mojarra guys...

      https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1308

      ...who fixed it in their implementation (as of 1.2_14)

      I then put together a MyFaces version of the same test case (attached as suggestionMyFaces.zip)...

      https://jira.jboss.org/jira/browse/RF-7700

      ...and it appears MyFaces (as of 1.2.7) has the same bug?

      1. uidata.patch
        7 kB
        Jakob Korherr

        Activity

        Hide
        Kennard Consulting added a comment -

        Terrific. Thanks for the quick turnaround guys. Look forward to trying it.

        Show
        Kennard Consulting added a comment - Terrific. Thanks for the quick turnaround guys. Look forward to trying it.
        Hide
        Leonardo Uribe added a comment -

        Thanks to Jakob Korherr for provide this patch

        Show
        Leonardo Uribe added a comment - Thanks to Jakob Korherr for provide this patch
        Hide
        Jakob Korherr added a comment -

        Here is my patch for this issue, including a test case, in which invokeOnComponent looks for a Facet of a h:column child of UIData.

        I also tested suggestionMyFaces.zip with my implementation and it worked!

        Show
        Jakob Korherr added a comment - Here is my patch for this issue, including a test case, in which invokeOnComponent looks for a Facet of a h:column child of UIData. I also tested suggestionMyFaces.zip with my implementation and it worked!
        Hide
        Leonardo Uribe added a comment -

        It is obvious that MyFaces has the same bug, because it is a problem in the definition of the algorithm of UIData.invokeOnComponent. Note this method definition is part of the spec, but theory, if ri change this behavior, we could do the same. The only problem is if solve this one makes fail the TCK test related to this method, so if we solve this one and some problem happens on next release, we could skip this one.

        Show
        Leonardo Uribe added a comment - It is obvious that MyFaces has the same bug, because it is a problem in the definition of the algorithm of UIData.invokeOnComponent. Note this method definition is part of the spec, but theory, if ri change this behavior, we could do the same. The only problem is if solve this one makes fail the TCK test related to this method, so if we solve this one and some problem happens on next release, we could skip this one.

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Kennard Consulting
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development