[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