thats better. I think that throw exceptions on missing resources shouldn't happen in deployment mode anyway i think there is no much use for it a good log statement in the server log should be enough.
i dont find it very dangerous that it shows the body IF it has a body because that you see it on your page you could see [THIS SHOULD BE A KEY] which is very easy to spot. But also the default text which is also fine. (except ofcourse if you think it comes from he db and you change something there and you don't see it, but then you change something or update something and then it should be able to look it up) johan On 11/4/07, Eelco Hillenius <[EMAIL PROTECTED]> wrote: > > On 11/3/07, Al Maw <[EMAIL PROTECTED]> wrote: > > Johan Compagner wrote: > > > Yes we could do that. Just remove that default value. > > > But what i would like to have IF you have a body specified then we > don't > > > throw anything > > > This way we keep old behavior and we have the new one > > > > FWIW, I entirely agree with this. If we just change it for tags that > > have a "default" body then we'll almost certainly break people's sites. > > How about keeping this for the deployment mode (and log an error or > warning once) but throw that exception in development mode? > > Eelco > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > >
