Thanks Mike, I've applied your patch.  FYI, I did a little other work in
the same section as well.

Mike Haberman wrote:
> 
> Here's the patch that deals with the exception only
> 
> Let me know if you want the patch that factors out the
> repetitive code.
> 
> mike
> 
> --- turbine/src/java/org/apache/turbine/Turbine.java    Wed Sep  6 13:05:33 2000
> +++ Turbine.java        Wed Sep  6 13:09:44 2000
> @@ -607,13 +607,23 @@
>          {
>              try
>              {
> +                String msg;
> +                if (data != null)
> +                {
> +                   msg = "Exception: " + data.stackTrace;
> +                }
> +                else
> +                {
> +                   msg = "Exception: " + e;
> +                }
> +
>                  res.setContentType( mimeType );
>                  res.setStatus ( 200 );
> -                res.getWriter().print ("Exception: " + data.stackTrace);
> +                res.getWriter().print (msg);
>              }
>              catch (Exception foob)
>              {
> -                // Ignore.
> +                // ignore
>              }
>              org.apache.turbine.util.Log.error ( reallyScrewedNow );
>          }
> 
> ------------------------------------------------------------
> To subscribe:        [EMAIL PROTECTED]
> To unsubscribe:      [EMAIL PROTECTED]
> Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
> Problems?:           [EMAIL PROTECTED]

-- 

Daniel Rall <[EMAIL PROTECTED]>
http://collab.net/ | open source | do the right thing


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to