[Freeswitch-users] week day and time condition possible bug

Sun Sep 4 23:05:40 MSD 2016

If you specified 5pm to 5pm it would only fire on the 1 second that is 5 pm as anything else would not match today is not sept 4, 2016 from 00:00:00 until 00:00:00 it is sept 4 2016 from 00:00:00 until 23:59:59 after that we either restarted the day oe went to another day

Sent from my iPhone

> On Sep 4, 2016, at 11:38 AM, Bipin Patel <bipin at xbipin.com> wrote:
> 
> ok so logically in code u dont see it as wrap around so 9pm to 5am would be 0am to 5am and 9pm to 0am and if days 1-2 then on both days because the code evaluate on a per day basis rather than across multiple days and applies the same to the day1 and day2.
> 
> do u still agree the to time needs to be one second shorter than the from time for the code to evaluate it properly or else 5pm-5pm would result in 0 and the way to specify full 24 hrs needs to be 5pm-4:59pm
> 
> 
> Regards,
> Bipin
> 
> 
> -------- Original Message --------
> Subject: Re: [Freeswitch-users] week day and time condition possible bug
> From: Ken Rice <krice at freeswitch.org>
> To: FreeSWITCH Users Help <freeswitch-users at lists.freeswitch.org>
> Date: 9/4/2016, 8:08:31 PM
>> Wrong there is no wrap around time.. There is only ranges of time on a day...
>> 
>> Look back at the code i pasted again.
>> 
>> If start time > end time then we check to see if current time < end time OR > start time again i. Seconds and for symplicity we'll use just 60 of them
>> 
>> If the rand is 45 through 15 then we do the false part of the exp?true:false which is (val >= range_start || val <= range_end)
>> 
>> So now start is 45 and end is 15 so we look for the time to >= 45 or <= 15 again on day 1 or day 2 assuming wday="1-2"
>> 
>> So there is no "wrap around time" there is no wday=1-2" and timeofday="9p-5a" and an expectation of that only working from 9p on day 1 to 9p on day 2 thats day 1 from 00:00 to 05:00 and from 21:00 til 23:59:59 then the same thing on day 2.
>> 
>> 
>> 
>> Sent from my iPhone
>> 
>> On Sep 4, 2016, at 10:33 AM, Bipin Patel <bipin at xbipin.com> wrote:
>> 
>>> thanks for the detailed explanation :)
>>> 
>>> so what i make of this is if u use a wrap around time or u want full 24 hours either in same day or across days then u need to set one second lower so "5:00-5:00" should actually be "5:00:00-4:59:59" which would be 5am from today to 4:59:59am tomorrow and same goes for "00:00:00-00:00:00" which should be "00:00:00-23:59:59"
>>> 
>>> now same if u combine with wday then again u need to specify a second lower in the max time for it to work               properly if the start and end time is set same or it will read it as 0
>>> 
>>> 
>>> Regards,
>>> Bipin
>>> 
>>> 
>>> -------- Original Message --------
>>> Subject: Re: [Freeswitch-users] week day and time condition possible bug
>>> From: Ken Rice <krice at freeswitch.org>
>>> To: FreeSWITCH Users Help <freeswitch-users at lists.freeswitch.org>
>>> Date: 9/4/2016, 7:14:25 PM
>>>> Ok lets try to make sense of this.
>>>> 
>>>> The time-of-day matching in your initial email 
>>>> >>       <condition wday="1-2" time-of-day="00:00:00-00:00:00" break="never">
>>>> 
>>>> happens in 2 stages... stage 1) does wday fall in the range of 1-2
>>>> wday="1-2"
>>>> wdaymin = 1
>>>> wdayamx = 2
>>>> so does if (today >= wday-min || today <= wday-max).... if you set this for 0-0 or 1-1 then only day 0 or only day 1 would match. Now the above example is simplified psuedo code just for the sake of explanation...
>>>> 
>>>> The time-of-day matching works similarly... after the value for the time of day field is parsed the following if                     statement (actually taken from the code) happens
>>>> 
>>>> if (range_start <= range_end ? (val >= range_start && val <= range_end) : (val >= range_start || val <= range_end)) { ... }
>>>> 
>>>> so this allos for the range_end to be a lower value... however the same thing happens does the current time val fall in the range of times... so if you set the start and the end time to 00:00:00 the only time it will match is 00:00:00. lets reduce this for the sake of simplicity from the long date to just seconds of the day... 00:00:00 == second 0 for the day... so 
>>>> 
>>>> 
>>>> range start is <= range_end (in fact its 0) so ... we'll do the true half of the expression?true:false statement there...
>>>> if (val >= range_start && val <= range_end)  where val is the current number of seconds into today... 
>>>> 
>>>> so you have told it that the only range that can match is 0 thru 0 inclusive... how can any other time match... 
>>>> 
>>>> so in your example there are only 2 times that will match 00:00:00 on day 1 and 00:00:00 on day 2... 
>>>> 
>>>> the proper way to include an entire day is 00:00:00-23:59:59 which covers all possible seconds in a range
>>> _________________________________________________________________________
>>> Professional FreeSWITCH Consulting Services: 
>>> consulting at freeswitch.org
>>> http://www.freeswitchsolutions.com
>>> 
>>> Official FreeSWITCH Sites
>>> http://www.freeswitch.org
>>> http://confluence.freeswitch.org
>>> http://www.cluecon.com
>>> 
>>> FreeSWITCH-users mailing list
>>> FreeSWITCH-users at lists.freeswitch.org
>>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
>>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
>>> http://www.freeswitch.org
>> 
>> 
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services: 
>> consulting at freeswitch.org
>> http://www.freeswitchsolutions.com
>> 
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://confluence.freeswitch.org
>> http://www.cluecon.com
>> 
>> FreeSWITCH-users mailing list
>> FreeSWITCH-users at lists.freeswitch.org
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
>> http://www.freeswitch.org
> 
> _________________________________________________________________________
> Professional FreeSWITCH Consulting Services: 
> consulting at freeswitch.org
> http://www.freeswitchsolutions.com
> 
> Official FreeSWITCH Sites
> http://www.freeswitch.org
> http://confluence.freeswitch.org
> http://www.cluecon.com
> 
> FreeSWITCH-users mailing list
> FreeSWITCH-users at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
> http://www.freeswitch.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freeswitch.org/pipermail/freeswitch-users/attachments/20160904/d22d6658/attachment.html