6.0.0-beta1
9/24/25

[#4051] Kolab.php issues errors when public, shared folders are present
Summary Kolab.php issues errors when public, shared folders are present
Queue Kolab
Type Bug
State Not A Bug
Priority 1. Low
Owners
Requester wrobel (at) gentoo (dot) org
Created 06/16/2006 (7040 days ago)
Due
Updated 08/04/2006 (6991 days ago)
Assigned 06/21/2006 (7035 days ago)
Resolved 08/04/2006 (6991 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
08/04/2006 10:21:21 PM Jan Schneider Comment #6
State ⇒ Not A Bug
Reply to this comment
Great news.
08/04/2006 10:16:03 PM wrobel (at) gentoo (dot) org Comment #5 Reply to this comment
No since I rewrote the Kolab driver. I finished the first draft this 
week but I still wanted to fix a few minor things before I submit it 
for revision by the horde people. Should happen next week though. This 
ticket can be closed.
08/04/2006 10:10:44 PM Jan Schneider Comment #4 Reply to this comment
Are you going to update your patch?
06/21/2006 08:24:11 AM wrobel (at) gentoo (dot) org Comment #3 Reply to this comment
Yes I realized that my patch also creates problems in other areas of 
the Kolab driver. I'll reconsider it and attach an updated patch onve 
I get it fixed properly.



But the regex needs fixing since it currently distinguishes only 
"user/..." and "INBOX/..." mailboxes and does not know anything about 
"shared...".
06/21/2006 07:42:56 AM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
The patch doesn't look quite correct. I didn't look at the original 
code, but the original regexp seems to be looking explicetly for the 
INBOX folder, not for any subfolders of INBOX. Maybe it did this by 
accident, but "INBOX/" was definitely not matched by the expression 
before, but it is now with your patch.
06/16/2006 11:57:09 AM wrobel (at) gentoo (dot) org Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Kolab.php issues errors when public, shared folders are present
Queue ⇒ Kolab
New Attachment: package-framework_kolab-sync-shares.patch Download
State ⇒ Unconfirmed
Reply to this comment
The _synchShares function in Kolab.php uses the regular expression



(INBOX|user/([^/]+))/([^@]+)(@.*)?



which does not include folders that start with "shared." The presence 
of shared folders leads to errors in the subsequent code



A suggested patch is attached. It is no real fix since the Kolab.php 
script needs in general some more work to support shared folders in a 
better way, but the proposed fix at least prevents the script from 
issuing errors.


Saved Queries