[dev] [commits] Horde branch develop updated. 90cc999e6173bc827b2387090249ec7e8f3e48eb

Michael M Slusarz slusarz at horde.org
Thu May 31 18:06:06 UTC 2012


Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael M Slusarz <slusarz at horde.org>:
>
>> 0c01370 [mms] Import API call now throws an ObjectExists exception  
>> when a duplicate is found.
>
> We have similiar exceptions in the other groupware application APIs,  
> so this go into the Horde_Exception package. If for some reason this  
> isn't possible, this exception should extend Turba_Exception instead.

I don't agree with the former: ObjectExists is pretty generic (does it  
refer to a Turba_Object?  Or maybe a Turba_Widget_Foo_Object?), so it  
necessarily needs to be defined in each application depending on the  
data a particular application is dealing with.

The latter is correct though - this should be extending  
Turba_Exception.  I will fix this.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]



More information about the dev mailing list