[TILES-237] When TilesListener throws IllegalArgumentException, it should wrap the original cause.

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.5
• Fix Version/s: 2.0.6
• Component/s: tiles-core
• Flags: Patch
• Assignee: Antonio Petrelli
• Reporter: Mike Calmus
• Created: 04/Jan/08 17:07
• Updated: 27/May/08 10:03
• Resolved: 07/Jan/08 19:56

Description

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); }

}