[commits] Horde branch master updated. 4bd7428e438a50537882c70e1a71f90a963ff856
Michael J. Rubinsky
mrubinsk at horde.org
Fri Aug 26 16:49:42 UTC 2016
The branch "master" has been updated.
The following is a summary of the commits.
from: 7695fc444ff10bd9950f0332c27c5c5874397d0a
4bd7428 Revert "Bug: 14453 Limit addCoverDates to the $endDate."
Summary: http://github.com/horde/horde/compare/7695fc444ff10bd9950f0332c27c5c5874397d0a...4bd7428e438a50537882c70e1a71f90a963ff856
-----------------------------------------------------------------------
commit 4bd7428e438a50537882c70e1a71f90a963ff856
Author: Michael J Rubinsky <mrubinsk at horde.org>
Date: Fri Aug 26 12:44:23 2016 -0400
Revert "Bug: 14453 Limit addCoverDates to the $endDate."
This reverts commit 7695fc444ff10bd9950f0332c27c5c5874397d0a.
This isn't correct. Setting $endDate to $event->end defeats the purpsose
of this change when polling for alarms but removing that line
will no longer return multi-day events that start before $startDate
when no $endDate is provided.
Not sure what the solution is here since when polling alarms we
don't want an end date - no way of knowing the longest alarm interval
that we have.
Perhaps always limiting to some unrealistic time frame like 10 years
would be sufficient for protecting against memory limits while still
allowing reasonable alarm intervals to be supported?
kronolith/lib/Kronolith.php | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
http://github.com/horde/horde/commit/4bd7428e438a50537882c70e1a71f90a963ff856
More information about the commits
mailing list