javaserverfaces
  1. javaserverfaces
  2. JAVASERVERFACES-2179

PartialviewContextImpl doesn't allow custom exception handling

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.3
    • Fix Version/s: 2.1.16, 2.2.0-m07
    • Component/s: ajax
    • Labels:
      None
    • Environment:

      glassfish 3.1

      Description

      PartialViewContextImpl catches any exception and simply logs it in line 253 and 520. This way the exceptionhandler get's never informed about the exceptions. Simply wrap the exceptions in FacesExceptions and rethrow them.

        Activity

        Hide
        rogerk added a comment -

        Changes.

        Show
        rogerk added a comment - Changes.
        Hide
        rogerk added a comment -

        Changes.

        Show
        rogerk added a comment - Changes.
        Hide
        rogerk added a comment -

        Changes for trunk.

        Show
        rogerk added a comment - Changes for trunk.
        Hide
        rogerk added a comment -

        Committed to MOJARRA_2_1X_ROLLING:
        Sending jsf-ri/src/main/java/com/sun/faces/context/PartialViewContextImpl.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component1.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component2.java
        Adding test/agnostic/ajax/src/main/webapp/WEB-INF/taglib.xml
        Sending test/agnostic/ajax/src/main/webapp/WEB-INF/web.xml
        Adding test/agnostic/ajax/src/main/webapp/issue2179-page1.xhtml
        Adding test/agnostic/ajax/src/main/webapp/issue2179-page2.xhtml
        Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue2179IT.java
        Transmitting file data ........
        Committed revision 11044.

        Committed to trunk:
        Sending jsf-ri/src/main/java/com/sun/faces/context/PartialViewContextImpl.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component1.java
        Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component2.java
        Adding test/agnostic/ajax/src/main/webapp/WEB-INF/taglib.xml
        Sending test/agnostic/ajax/src/main/webapp/WEB-INF/web.xml
        Adding test/agnostic/ajax/src/main/webapp/issue2179-page1.xhtml
        Adding test/agnostic/ajax/src/main/webapp/issue2179-page2.xhtml
        Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue2179IT.java
        Transmitting file data ........
        Committed revision 11045.

        Show
        rogerk added a comment - Committed to MOJARRA_2_1X_ROLLING: Sending jsf-ri/src/main/java/com/sun/faces/context/PartialViewContextImpl.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component1.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component2.java Adding test/agnostic/ajax/src/main/webapp/WEB-INF/taglib.xml Sending test/agnostic/ajax/src/main/webapp/WEB-INF/web.xml Adding test/agnostic/ajax/src/main/webapp/issue2179-page1.xhtml Adding test/agnostic/ajax/src/main/webapp/issue2179-page2.xhtml Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue2179IT.java Transmitting file data ........ Committed revision 11044. Committed to trunk: Sending jsf-ri/src/main/java/com/sun/faces/context/PartialViewContextImpl.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component1.java Adding test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2179Component2.java Adding test/agnostic/ajax/src/main/webapp/WEB-INF/taglib.xml Sending test/agnostic/ajax/src/main/webapp/WEB-INF/web.xml Adding test/agnostic/ajax/src/main/webapp/issue2179-page1.xhtml Adding test/agnostic/ajax/src/main/webapp/issue2179-page2.xhtml Adding test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue2179IT.java Transmitting file data ........ Committed revision 11045.
        Hide
        rogerk added a comment -

        Committed.

        Show
        rogerk added a comment - Committed.

          People

          • Assignee:
            rogerk
            Reporter:
            Frank Caputo
          • Votes:
            4 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: