Sounds good... Although I don't like having these huge application
error's  as strings within Turbine.java.  It is already a mile long.

If the resources were already loaded I would say put them in a resource
file... but here you cant do that.

Why not just break it out into an org.apache.util.ErrorMessages class
and have accessors.

This way you keep the ugly (IMO) code out of Turbine.java.

Kevin

Jeff Prickett wrote:
<snip>
---
> >             log("Turbine::init(ServletConfig) - 'properties' Init Parameter" +
> >                    " not found. Please specify a value for the 'properties'" +
> >                    " Init Parameter by editing the servlet zone properties " +
> >                    "file.  This is done by adding the " +
> >                    "following line to the zone properties file: \n" +
> >                    "servlet.Turbine.initArgs=properties=" +
> >                    "/path/to/TurbineResources.properties");
> >
> >             throw new ServletException(
> >                    "Turbine:init(ServletConfig) - 'properties' Init Parameter" +
> >                    " not found. Please specify a value for the 'properties'" +
> >                    " Init Parameter by editing the servlet zone properties " +
> >                    "file.  This is done by adding the " +
> >                    "following line to the zone properties file: \n" +
> >                    "servlet.Turbine.initArgs=properties=" +
> >                    "/path/to/TurbineResources.properties");
> >         }
> >         else if(props.length() == 0)
> >         {
> >             log("Turbine::init(ServletConfig) - 'properties' Init Parameter" +
> >                    " value is empty. The 'properties' Init Parameter exists" +
> >                    " but it is a zero length string. Please specify a value "
<snip>
-- 
Kevin A Burton
Senior Software Engineer
Kendara Inc
http://www.kendara.com
Mobile:  408-910-6145
Linux - The revolution will NOT be televised


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Problems?:           [EMAIL PROTECTED]

Reply via email to