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

6.0.0-beta6

4/10/26

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	5/8/09 (6181 days ago)
Due
Updated	7/28/09 (6100 days ago)
Assigned	6/22/09 (6136 days ago)
Resolved	7/28/09 (6100 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