[dev] [cvs] commit: framework/Prefs Prefs.php framework/Share Share.php
Chuck Hagenbuch
chuck at horde.org
Mon Jul 13 23:43:14 UTC 2009
Quoting Michael M Slusarz <slusarz at horde.org>:
> slusarz 2009-07-13 19:34:11 EDT
>
> Modified files:
> Prefs Prefs.php
> Share Share.php
> Log:
> Some more Horde_Registry changes.
>
> Revision Changes Path
> 1.206 +8 -5 framework/Prefs/Prefs.php
> 1.178 +19 -23 framework/Share/Share.php
>
> Chora Links:
>
> http://cvs.horde.org/diff.php/framework/Prefs/Prefs.php?rt=horde&r1=1.205&r2=1.206&ty=u
>
> http://cvs.horde.org/diff.php/framework/Share/Share.php?rt=horde&r1=1.177&r2=1.178&ty=u
Unless we are very sure that we're never going to need those
exceptions, let's not trap them silently. I think that we should
probably bubble up legit exceptions from callhook, and not throw one
if the hook simply doesn't exist.
-chuck
More information about the dev
mailing list