Re: Connected Callback Errors Ignored

2013-11-15 Thread Tim Bunce
On Thu, Nov 14, 2013 at 02:07:41PM -0800, David E. Wheeler wrote: On Nov 14, 2013, at 9:17 AM, David E. Wheeler da...@justatheory.com wrote: The error about foo not existing is gone, overridden by the second error about bar missing. This can lead to hard-to-find bugs. What if the second

Re: Connected Callback Errors Ignored

2013-11-15 Thread David E. Wheeler
On Nov 15, 2013, at 3:12 AM, Tim Bunce tim.bu...@pobox.com wrote: You'd need to write the callback code in the way you'd naturally write it when not using RaiseError. Which typically means something like: $dbh-do('SET search_path = ?', undef, 'foo') or return; That will prevent

Re: Connected Callback Errors Ignored

2013-11-15 Thread Tim Bunce
On Fri, Nov 15, 2013 at 09:14:58AM -0800, David E. Wheeler wrote: On Nov 15, 2013, at 3:12 AM, Tim Bunce tim.bu...@pobox.com wrote: You'd need to write the callback code in the way you'd naturally write it when not using RaiseError. Which typically means something like: $dbh-do('SET

Re: Connected Callback Errors Ignored

2013-11-15 Thread David E. Wheeler
On Nov 15, 2013, at 10:24 AM, Tim Bunce tim.bu...@pobox.com wrote: When the method call (which fired the callback) returns, the error recorded on the handle will trigger RaiseError etc. Only on 1.630 and higher. I *always* use RaiseError = 1 (or HandleError). Never ever check return