[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