Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.3.2
Bug #4949 deleteMailbox() tries to reset ACL on deleted mailbox
Submitted: 2005-07-29 22:38 UTC
From: patrick at coverallcrew dot com Assigned: amistry
Status: Closed Package: Net_Cyrus
PHP Version: 4.3.11 OS: FreeBSD
Roadmaps: (Not assigned)    
Subscription  


 [2005-07-29 22:38 UTC] patrick at coverallcrew dot com
Description: ------------ On line 345 of Net_Cyrus.php, there is a call to _resetAdminPriv() on the mailbox that was just deleted. This line should be moved to line 343, right before a PEAR_Error is returned, as the only time you would want to reset the privileges is if the mailbox could not be deleted.

Comments

 [2006-01-27 22:36 UTC] amistry at php dot net
I know this is trivial, but could you just verify this patch works for you. http://am-productions.biz/docs/Cyrus.patch
 [2006-01-27 22:52 UTC] patrick at coverallcrew dot com
Much better. (This patch changes the code to what I already had changed myself, so I know it fixes it.)
 [2006-02-03 21:18 UTC] amistry at php dot net
Thanks, I'm going to ask for the maintainer hat for this package and will try to update it as soon as I can. It may take about a month or so since I'll be out of town for a few weeks. If you don't hear anything by mid-March send me a reminder.
 [2006-03-22 04:06 UTC] amistry at php dot net (Anish Mistry)
Fixed in the CVS.