[dev] [cvs] commit: framework/Date/Date Recurrence.php

[Jan Schneider]

Zitat von Gunnar Wrobel <wrobel at pardus.de>:

> Jan Schneider <jan at horde.org> writes:
>
>> Zitat von Gunnar Wrobel <p at rdus.de>:
>>
>>> wrobel      2008-07-07 00:40:07 EDT
>>>
>>>   Modified files:
>>>     Date/Date            Recurrence.php
>>>   Log:
>>>   Add a function to return the next active recurrence.
>>>
>>>   Revision  Changes    Path
>>>   1.15      +26 -1     framework/Date/Date/Recurrence.php
>>
>> Why not having nextRecurrence() do this?
>
> You mean with an additional flag like "$active = false" in the
> function call? Sure, should also work. Having the while loop in
> nextActiveRecurrence() seemed appropriate to me but I'm not
> particularly attached to solving it like that.

No, I think nextRecurrence() should unconditionally skip any complete  
recurrences, just like it skips exceptions now.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/