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

        Mike Calmus created issue -
        Antonio Petrelli made changes -
        Field Original Value New Value
        Priority Minor [ 4 ] Major [ 3 ]
        Fix Version/s 2.0.6 [ 21862 ]
        Assignee Antonio Petrelli [ brenmcguire ]
        Antonio Petrelli made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Antonio Petrelli made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Antonio Petrelli made changes -
        Workflow Struts - editable closed status [ 44806 ] Struts - editable closed status (temporary) [ 45853 ]
        Antonio Petrelli made changes -
        Workflow Struts - editable closed status (temporary) [ 45853 ] Struts - editable closed status [ 49637 ]
        Antonio Petrelli made changes -
        Workflow Struts - editable closed status [ 49637 ] Simplified [ 53391 ]
        Jeff Turner made changes -
        Project Import Mon Feb 01 01:30:28 UTC 2010 [ 1264987828110 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development