6.0.0-beta1
7/4/25

[#12591] Kolab driver: Fix searching through folders in getByUid()
Summary Kolab driver: Fix searching through folders in getByUid()
Queue Nag
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester thomas.jarosch (at) intra2net (dot) com
Created 08/20/2013 (4336 days ago)
Due
Updated 02/09/2016 (3433 days ago)
Assigned 08/23/2013 (4333 days ago)
Resolved 02/09/2016 (3433 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
02/09/2016 04:51:23 PM Michael Rubinsky Comment #5
Assigned to Michael Rubinsky
Taken from Jan Schneider
State ⇒ Resolved
Reply to this comment
01/27/2016 05:03:06 PM Jan Schneider Comment #4 Reply to this comment
Ping
08/23/2013 10:08:12 AM Jan Schneider Comment #3
State ⇒ Feedback
Reply to this comment
This could be easier (or better) done without the helper method. 
Instead, unset $this->_data when changing $this->_tasklist. 
Alternatively, check if $this->_data was cached for the correct 
$this->_tasklist in _getData().
08/22/2013 08:00:40 PM Michael Rubinsky Comment #2
State ⇒ Assigned
Assigned to Jan Schneider
Reply to this comment
Patch looks good to me but since I'm not as familiar with Kolab as 
Jan, assigning to him to be sure.
08/20/2013 04:26:27 PM Thomas Jarosch Patch ⇒ Yes
New Attachment: 0001-Kolab-driver-Fix-searching-through-folders-in-getByU.patch Download
 
08/20/2013 04:25:11 PM Thomas Jarosch Comment #1
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Queue ⇒ Nag
Summary ⇒ Kolab driver: Fix searching through folders in getByUid()
Type ⇒ Bug
Priority ⇒ 1. Low
Reply to this comment
From the patch:

getByUid() was only searching the first tasklist it found.
The intention of the code was to search all tasklists,
but there was no code to change the current folder.

Also restore a sane state when leaving the function.

Bonus points: If the task is found in the first tasklist,
it does not cost any extra performance.

Saved Queries