Tickets :: [#8258] Catch exists() errors

6.0.0-beta1

7/26/25

Summary	Catch exists() errors
Queue	Horde Base
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	duck (at) obala (dot) net
Created	05/08/2009 (5923 days ago)
Due
Updated	07/28/2009 (5842 days ago)
Assigned	06/22/2009 (5878 days ago)
Resolved	07/28/2009 (5842 days ago)
Milestone
Patch	Yes

07/28/2009 11:07:41 PM	Michael Slusarz	Comment #7 State ⇒ Resolved	Reply to this comment
Cleaned up to new Horde_Auth/Horde 4 standards (PEAR_Errors are bad - use Exceptions instead) and committed to horde-git.

07/09/2009 05:09:00 AM	Chuck Hagenbuch	Comment #6 Assigned to Michael Slusarz Taken from Jan Schneider	Reply to this comment
Assigning over to Michael S. since he just refactored this package.

06/22/2009 03:15:32 PM	Chuck Hagenbuch	State ⇒ Assigned Assigned to Jan Schneider

06/22/2009 12:06:38 PM	Duck	Comment #5 New Attachment: Signup.diff	Reply to this comment
I'm sorry, I forgot about this ticket when I refactored queueSignup(). Can you please update your patch to what we have in CVS now?

06/16/2009 01:22:01 PM	Jan Schneider	Comment #4 State ⇒ Feedback	Reply to this comment
I'm sorry, I forgot about this ticket when I refactored queueSignup(). Can you please update your patch to what we have in CVS now?

05/26/2009 03:50:47 AM	Chuck Hagenbuch	State ⇒ Accepted

05/20/2009 08:01:43 AM	Duck	Comment #3 New Attachment: Sigun.diff	Reply to this comment
Since we have duplicate code in 3 different places now, this should rather be consolidated into a single exists() method. Added _preSignup() method to do common tasks in queue.

05/12/2009 02:30:42 PM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
Since we have duplicate code in 3 different places now, this should rather be consolidated into a single exists() method.

05/08/2009 09:58:37 AM	Duck	Comment #1 New Attachment: Auth.diff State ⇒ New Patch ⇒ Yes Milestone ⇒ Queue ⇒ Horde Base Summary ⇒ Catch exists() errors Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
- Catch errors from exists calls (ex: an pear error about missing table results as an existing user detection) - Don't need to check queue signup entries, if queue is disabled