javaserverfaces-spec-public
  1. javaserverfaces-spec-public
  2. JAVASERVERFACES_SPEC_PUBLIC-729

Bug in jsf2.0 final sepc draft 20090625 -> 7.5.2 Defautl ViewHanlder Impl

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0 Rev a
    • Component/s: Components/Renderers
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      729
    • Status Whiteboard:
      Hide

      cat1 frame changelog

      Show
      cat1 frame changelog

      Description

      Hello Guys,
      While going through the spec found a bug/typo mistake...

      7.5.2 Default ViewHanlder Implementation:
      Page no. 236

      [P1-start renderView() requirements] The renderView() method must obtain a
      reference to the
      ViewDeclarationLanguage for the viewId of the argument viewToRender and call its
      ViewDeclarationLanguage.createView() method, returning the result and not
      swallowing any exceptions
      thrown by that method.[P1-end]
      [P1-start restoreView() requirements]The restoreView() method must obtain a
      reference to the
      ViewDeclarationLanguage for the viewId of the argument viewToRender and call its
      ViewDeclarationLanguage.createView() method, returning the result and not
      swallowing any exceptions
      thrown by that method.[P1-end]

      Here you can see the renderView() and restoreView() mentions to call
      ViewDeclarationLanguage.createView() instead it should call
      ViewDeclarationLanguage.renderView() and ViewDeclarationLanguage.restoreView()
      respectively.

      Regards,
      Amar Pandav

        Activity

        Hide
        Ed Burns added a comment -

        cat1

        Show
        Ed Burns added a comment - cat1
        Hide
        Ed Burns added a comment -

        switch to renderers

        Show
        Ed Burns added a comment - switch to renderers
        Hide
        Ed Burns added a comment -

        These are valid 2.0 Rev a issues

        Show
        Ed Burns added a comment - These are valid 2.0 Rev a issues
        Hide
        Ed Burns added a comment -

        Take ownership

        Show
        Ed Burns added a comment - Take ownership
        Hide
        Ed Burns added a comment -

        Fix committed.

        Show
        Ed Burns added a comment - Fix committed.
        Hide
        Ed Burns added a comment -

        Actually mark it fixed.

        Show
        Ed Burns added a comment - Actually mark it fixed.
        Hide
        rogerk added a comment -

        changelog

        Show
        rogerk added a comment - changelog
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

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

            Dates

            • Created:
              Updated:
              Resolved: