Tiles
  1. Tiles
  2. TILES-237

When TilesListener throws IllegalArgumentException, it should wrap the original cause.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.5
    • Fix Version/s: 2.0.6
    • Component/s: tiles-core
    • Labels:
      None
    • Flags:
      Patch

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

      }

        Activity

          People

          • Assignee:
            Antonio Petrelli
            Reporter:
            Mike Calmus
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development