[dev] Re: [cvs] commit: turba/lib Driver.php sam/lib Driver.php orator/lib Slide.php odin/lib Driver.php LogParser.php moment/lib View.php mnemo/lib Driver.php kronolith/lib Driver.php klutz/lib Driver.php ingo/lib Script.php

[Chuck Hagenbuch]

Quoting Jan Schneider <jan at horde.org>:

> Ok, I think it might not always be easy to decide if an error is fatal,
> but it makes sense. Though IIRC Turba (as all other Driver APIs) exits
> fatally if one of the configuration parameters is missing but not if an
> error occured during the initialization of the driver.

Yeah; we should probably have it return a PEAR_Error for those, too,
anywhere that we actually handle that failure.

> What I think we could agree on is to always return at least a PEAR_Error
> if something fails. Just returning false isn't very helpful when trying to
> track why something isn't working. In return each call to factory methods
> needs to check for such an error then.

That sounds like a good goal to me.

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
The alligators were there, too, in a bathtub inside the house.