When TilesListener throws IllegalArgumentException, it should wrap the original
cause.
--------------------------------------------------------------------------------------
Key: TILES-237
URL: https://issues.apache.org/struts/browse/TILES-237
Project: Tiles
Issue Type: Improvement
Components: tiles-core
Affects Versions: 2.0.5
Reporter: Mike Calmus
Priority: Minor
The contextInitialized method of org.apache.tiles.web.startup.TilesListener
throws an IllegalStateException when a TilesException is thrown. This exception
should wrap the original TilesException so the underlying exception is not lost.
patch (likely won't apply due to some syntax issues):
Index:
src/tiles-core/src/main/java/org/apache/tiles/web/startup/TilesListener.java
===================================================================
---
src/tiles-core/src/main/java/org/apache/tiles/web/startup/TilesListener.java
(revision ????)
+++
src/tiles-core/src/main/java/org/apache/tiles/web/startup/TilesListener.java
(working copy)
@@ -61,? +61,? @@
TilesAccess.setContainer(servletContext, container);
} catch (TilesException e) {
LOG.fatal("Unable to retrieve tiles factory.", e);
- throw new IllegalStateException("Unable to instantiate container.");
+ throw new IllegalStateException("Unable to instantiate container.",
e); }
}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
