Skip to content

Commit

Permalink
Mark this as deprecated.
Browse files Browse the repository at this point in the history
The calendar parameter was never implemeneted correctly. The API method
doesn't take a calendar parameter, and the parameter that is *expected*
to be passed here is the $range parameter. Can't remove the method
parameter for BC reasons, but at least don't pass the incorrect value
for $range.
  • Loading branch information
mrubinsk committed Jan 21, 2016
1 parent faaef46 commit 3156014
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions framework/Core/lib/Horde/Core/ActiveSync/Connector.php
Expand Up @@ -211,11 +211,11 @@ public function calendar_replace($uid, Horde_ActiveSync_Message_Appointment $con
* Delete an event from Horde's calendar storage
*
* @param string $uid The UID of the event to delete
* @param string $calendar The calendar id. @since 2.12.0
* @param string $calendar The calendar id. @since 2.12.0 @deprecated (unused).
*/
public function calendar_delete($uid, $calendar = null)
{
$this->_registry->calendar->delete($uid, null, $calendar);
$this->_registry->calendar->delete($uid);
}

/**
Expand Down

0 comments on commit 3156014

Please sign in to comment.