6.0.0-beta1
7/6/25

[#12518] Kolab driver: Fix setting $this->_data twice. Unify duplicated code
Summary Kolab driver: Fix setting $this->_data twice. Unify duplicated code
Queue Turba
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester thomas.jarosch (at) intra2net (dot) com
Created 07/31/2013 (4358 days ago)
Due
Updated 08/06/2013 (4352 days ago)
Assigned
Resolved 08/06/2013 (4352 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
08/06/2013 04:13:41 PM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
 
08/06/2013 04:13:30 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 1f1368f0cd2e5bd33b0af587348350a1250ad843
Author: Thomas Jarosch <thomas.jarosch@intra2net.com>
Date:   Wed Jul 31 17:34:37 2013 +0200

     Kolab driver: Fix setting $this->_data twice. Unify duplicated code.

     _getDataForAddressbook() was only used by _getData().
     It was probably copied over from other Kolab drivers
     which use a similar driver skeleton.

     Signed-off-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
     Signed-off-by: Jan Schneider <jan@horde.org>
     Bug: 12518

  turba/lib/Driver/Kolab.php |   41 +++++++++++++++--------------------------
  1 files changed, 15 insertions(+), 26 deletions(-)

http://git.horde.org/horde-git/-/commit/1f1368f0cd2e5bd33b0af587348350a1250ad843
07/31/2013 03:50:46 PM Thomas Jarosch Comment #1
Priority ⇒ 1. Low
New Attachment: 0001-Kolab-driver-Fix-setting-this-_data-twice.-Unify-dup.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Turba
Summary ⇒ Kolab driver: Fix setting $this->_data twice. Unify duplicated code
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
Hi,

attached patch fixes setting $this->_data twice in the Kolab driver.

It also makes the code path more clear to read by unifying
duplicated code hidden in another function.

Cheers,
Thomas

Saved Queries