[#7014] maildir quota driver not working
Summary maildir quota driver not working
Queue IMP
Queue Version 4.2
Type Bug
State Resolved
Priority 2. Medium
Owners Jan Schneider <jan (at) horde (dot) org>
Requester filip (dot) sedlar (at) sendmail (dot) cz
Created 07/02/2008 (57 days ago)
Due 07/03/2008 (56 days ago)
Updated 07/09/2008 (50 days ago)
Assigned 07/06/2008 (53 days ago)
Resolved 07/09/2008 (50 days ago)
Attachments maildirsize Download
Milestone 4.2.1
Patch No

History
07/09/2008 CVS Commit Comment #9 Reply to this comment
Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.699.2.326&r2=1.699.2.327&ty=u
http://cvs.horde.org/diff.php/imp/lib/Quota/maildir.php?r1=1.1.2.6&r2=1.1.2.7&ty=u
http://cvs.horde.org/co.php/imp/lib/tests/fixtures/maildirsize?r=1.1.2.1
http://cvs.horde.org/co.php/imp/lib/tests/quota_maildir.phpt?r=1.1.2.1
07/09/2008 Jan Schneider Comment #8 Reply to this comment
Fixed for IMP 4.2.1.
07/09/2008 Jan Schneider State ⇒ Resolved
Taken from Eric Rostetter
 
07/09/2008 CVS Commit Comment #7 Reply to this comment
Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1159&r2=1.1160&ty=u
http://cvs.horde.org/diff.php/imp/lib/Quota/maildir.php?r1=1.5&r2=1.6&ty=u
http://cvs.horde.org/co.php/imp/lib/tests/fixtures/maildirsize?r=1.1
http://cvs.horde.org/co.php/imp/lib/tests/quota_maildir.phpt?r=1.1
07/06/2008 Jan Schneider State ⇒ Assigned
 
07/06/2008 Jan Schneider Milestone ⇒ 4.2.1
 
07/03/2008 filip (dot) sedlar (at) sendmail (dot) cz Comment #6
New Attachment: maildirsize Download		 Reply to this comment
maildirsize example
07/03/2008 filip (dot) sedlar (at) sendmail (dot) cz Comment #5 Reply to this comment

First: there is a typo in quota.php line:
This has been fixed.
and then I had to change this line:

$line = preg_replace("/[ \t\n\r\0\x0B]/", '', $line);
to this:
$line = preg_replace("/[ \t\n\r", '', $line);
Can you tell me why, and can you upload an example "maildirsize" file?
I got this error with original code:
Warning: preg_replace() 
[http://www.php.net/docs.phpfunction.preg-replace.php]: No ending 
delimiter '/' found (could it be because of PHP version???). Note that 
I am not a PHP programmer, I just administrate freemail server.

07/03/2008 Jan Schneider Comment #4
State ⇒ Feedback		 Reply to this comment
First: there is a typo in quota.php line:
This has been fixed.
and then I had to change this line:

$line = preg_replace("/[ \t\n\r\0\x0B]/", '', $line);
to this:
$line = preg_replace("/[ \t\n\r", '', $line);
Can you tell me why, and can you upload an example "maildirsize" file?
07/03/2008 CVS Commit Comment #3 Reply to this comment
Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/imp/lib/Quota/maildir.php?r1=1.4&r2=1.5&ty=u
07/02/2008 Chuck Hagenbuch Assigned to Eric Rostetter
 
07/02/2008 Chuck Hagenbuch State ⇒ Assigned
Assigned to Jan Schneider
 
07/02/2008 filip (dot) sedlar (at) sendmail (dot) cz Comment #2 Reply to this comment
The file name with bug is lib/Quota/maildir.php
07/02/2008 filip (dot) sedlar (at) sendmail (dot) cz Comment #1
Patch ⇒
Milestone ⇒
Queue ⇒ IMP
Due ⇒ 07/03/2008
Summary ⇒ maildir quota driver not working
Type ⇒ Bug
Priority ⇒ 2. Medium
State ⇒ Unconfirmed		 Reply to this comment
First: there is a typo in quota.php line:
instead of

            foreach ($lines as $line_number => $line) {
                 if ($line_num == 0) {

there should be:
            foreach ($lines as $line_number => $line) {
                 if ($line_number == 0) {

and then I had to change this line:

$line = preg_replace("/[ \t\n\r\0\x0B]/", '', $line);
to this:
$line = preg_replace("/[ \t\n\r", '', $line);

in order to get maildir quotas working

cheers