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 |
Assigned to Michael Rubinsky
Taken from Jan Schneider
State ⇒ Resolved
https://github.com/horde/horde/commit/f1858e1d513ea63a619b719382a0a2d9cbe2d795
State ⇒ Feedback
Instead, unset $this->_data when changing $this->_tasklist.
Alternatively, check if $this->_data was cached for the correct
$this->_tasklist in _getData().
State ⇒ Assigned
Assigned to Jan Schneider
Jan, assigning to him to be sure.
New Attachment: 0001-Kolab-driver-Fix-searching-through-folders-in-getByU.patch
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Queue ⇒ Nag
Summary ⇒ Kolab driver: Fix searching through folders in getByUid()
Type ⇒ Bug
Priority ⇒ 1. Low
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.