Hi Llewellyn!
Good input. You're probably right in many cases. I did a quick scan
looking for Log.error. Many of them are initialization-related errors
-- perhaps those should be exceptions instead. We just moved the
app-level exceptions to be RuntimeExceptions which gives us more
freedom to throw exceptions without changing the API. (this was one
of the obstacles in the past).
I disagree a bit about reference errors -- I think we should make it
optional whether or not they log a warning or stop page execution.
Right now they always do the former, but I'm working on a event
handler to allow the latter.
I think we've addressed your patches around the uberspector in svn
head at this point.
WILL
On 10/10/06, Llewellyn Falco <[EMAIL PROTECTED]> wrote:
I still would like to put in my vote that sending error's to the log is an
incredibly BAD idea.
If something is not working, it should be LOUDLY shown as an exception.
If it is working I don't really need a log.
The (velocity) log should be there for velocity developers (those
programming the actual velocity code) not users.
I don't ever care to see tomcat's log, I care to see the things I log while
in tomcat.
Most of all, many many many people do not check the log at all, let alone
frequently.
personally I swapped out the logging uberspect over 2 years ago, in favor of
code that actually worked, rather than code that didn't appear broken, but
actually was. but i know the default uberspect is still the logger, i
believe the TestableUberspect is still in the test branch, and a regular
user is probably just confused as to why something isn't working, double or
triple so if the something is in an if or for loop.
Llewellyn.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
Forio Business Simulations
Will Glass-Husain
[EMAIL PROTECTED]
www.forio.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
