javaeetutorial
  1. javaeetutorial
  2. JAVAEETUTORIAL-20

Advanced Composite Components chapter has error, needs enhancement

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.0.6
    • Fix Version/s: 7.0.2
    • Component/s: doc
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

      Description

      Reported to the Java EE Documentation mailing list from a user.

      Error:
      I think there is a mistake in chapter 13 of the Java EE 6 Tutorial (Advanced Composite Components), within the fourth section of that chapter (The compositecomponentlogin Example Application), in the first subsection of that section (The Composite Component File): I think there is a composite:attribute clause missing (<composite:attribute name="myloginBean" required="true"/>, or something like that).

      Enhancement:
      Moreover, from my point of view the contents of the next subsection (The Using Page) would be clearer if the subsection The Composite Component File of Chapter 13 started saying something like "The composite component file is an XHTML file called LoginPanel.xhtml." and a reference was added to the Composite Components section of Chapter 5 (Introduction to Facelets), where the use of Facelet libraries and prefixes for composite components is introduced.

        Activity

        Hide
        jendrock added a comment -

        Removed name of user who reported the issue.

        Show
        jendrock added a comment - Removed name of user who reported the issue.
        Hide
        jendrock added a comment -

        The "error" was fixed quite some time ago. The required="true" portion is not included, as I believe it is the default. The verbiage requested in the "enhancement" section was included as well.

        The cross reference back to the Composite Components section in the Introduction to Facelets chapter would be helpful. We'll add that in for the 7.0.0 release.

        Show
        jendrock added a comment - The "error" was fixed quite some time ago. The required="true" portion is not included, as I believe it is the default. The verbiage requested in the "enhancement" section was included as well. The cross reference back to the Composite Components section in the Introduction to Facelets chapter would be helpful. We'll add that in for the 7.0.0 release.
        Hide
        Ian Evans added a comment -

        Reassigning Chinmayee's issues.

        Show
        Ian Evans added a comment - Reassigning Chinmayee's issues.
        Hide
        jendrock added a comment -

        We will defer this fix until the next update, just prior to JavaOne. For that release, we are looking at replacing the compositecomponentlogin example with a more appropriate example.

        Show
        jendrock added a comment - We will defer this fix until the next update, just prior to JavaOne. For that release, we are looking at replacing the compositecomponentlogin example with a more appropriate example.
        Hide
        Kim Haase added a comment -

        Actually, there is a cross-reference to the Facelets chapter intro to composite components, right at the beginning of he Advanced Composite Components chapter. I don't think another one is needed.

        So I think this issue can be closed now.

        Show
        Kim Haase added a comment - Actually, there is a cross-reference to the Facelets chapter intro to composite components, right at the beginning of he Advanced Composite Components chapter. I don't think another one is needed. So I think this issue can be closed now.
        Hide
        jendrock added a comment -

        Actually, Kim is right. I will close the issue and open a new issue for replacing the compositecomponentlogin example with a more appropriate composite component example.

        Show
        jendrock added a comment - Actually, Kim is right. I will close the issue and open a new issue for replacing the compositecomponentlogin example with a more appropriate composite component example.

          People

          • Assignee:
            jendrock
            Reporter:
            jendrock
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: