Skip to main content

Source code revision

svn

summary: Bugdb 16952039 Ensure that Exceptions thrown by Converters during ajax transaction cause a log message including the stack trace.
revision: 12042
author: edburns
date: 2013-07-02 14:31:34 UTC (2 years)
message: Bugdb 16952039 Ensure that Exceptions thrown by Converters during ajax transaction cause a log message including the stack trace.

r=rogerk

SECTION: Modified Files
----------------------------
M jsf-ri/src/main/java/com/sun/faces/context/AjaxExceptionHandlerImpl.java

- Ensure that Exceptions thrown by Converters during ajax transaction
cause a log message including the stack trace.

M test/agnostic/converter/pom.xml

- New test

M jsf-api/src/main/resources/jsf.js

- Writing the test for this exposed a weakness in HtmlUnit's JavaScript
engine. This change works around it while preserving existing
functionality.

A test/agnostic/converter/converterThrowsNPE
A test/agnostic/converter/converterThrowsNPE/nbactions.xml
A test/agnostic/converter/converterThrowsNPE/src
A test/agnostic/converter/converterThrowsNPE/src/test
A test/agnostic/converter/converterThrowsNPE/src/test/java
A test/agnostic/converter/converterThrowsNPE/src/test/java/com
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE
A test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE/VerifyExceptionThrownIT.java
A test/agnostic/converter/converterThrowsNPE/src/main
A test/agnostic/converter/converterThrowsNPE/src/main/java
A test/agnostic/converter/converterThrowsNPE/src/main/java/com
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE
A test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE/ThrowNPEConverter.java
A test/agnostic/converter/converterThrowsNPE/src/main/resources
A test/agnostic/converter/converterThrowsNPE/src/main/webapp
A test/agnostic/converter/converterThrowsNPE/src/main/webapp/index.xhtml
A test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF
A test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF/web.xml
A test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF/glassfish-web.xml
A test/agnostic/converter/converterThrowsNPE/pom.xml

- Content of the new test.

Change Path Actions
M branches/MOJARRA_2_1X_ROLLING/common/ant/maven.xml
M branches/MOJARRA_2_1X_ROLLING/jsf-api/src/main/resources/jsf.js
M branches/MOJARRA_2_1X_ROLLING/jsf-ri/src/main/java/com/sun/faces/context/AjaxExceptionHandlerImpl.java
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/nbactions.xml
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/pom.xml
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE/ThrowNPEConverter.java
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/resources
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/webapp
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF/glassfish-web.xml
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/webapp/WEB-INF/web.xml
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/main/webapp/index.xhtml
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE
A branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/converterThrowsNPE/src/test/java/com/sun/faces/test/agnostic/converter/converterThrowsNPE/VerifyExceptionThrownIT.java
M branches/MOJARRA_2_1X_ROLLING/test/agnostic/converter/pom.xml
 
 
Close
loading
Please Confirm
Close