1. UNIFYNow Forum
  2. Bugs
0
Fixed

Update WeeklyExclusion to account for exclusion periods

Matthew Clark 13 years ago • updated by anonymous 8 years ago • 8

The current setup for the WeeklyExclusion timing allows a user to specify days of the week when the timing should not fire. However, for Event Broker v3.0.0, consider the following use case:

Not between 4pm and 7pm on Thursdays

The current setup as per Framework v3.0.1 to present instead does the following:

Not on Thursdays or between 4pm and 7pm any day

The first case may be a more desirable case, although both should be accounted for.

Matthew Clark 13 years ago

DailyExclusion has now been updated to take a "DaysExclusionApplies" parameter which specifies what days of the week the exclusion applies. If no days are specified, the exclusion is assumed to apply all days of the week.

The following now addresses the below use case:

<timing name="DailyExclusion" UseLocal="true">
<ExclusionPeriods>
<ExclusionPeriod DaysExclusionApplies="Monday,Tuesday,Wednesday" start="08:30:00" end="17:00:00" />
</ExclusionPeriods>
<timing name="Daily" TimeOfDay="13:00:00" UseLocal="true" />
</timing>

Run every day at 1pm, but don't run between 8:30am and 5pm on Monday, Tuesday or Wednesday

This also allows for the specification of multiple exclusion periods in a week

Matthew Clark 13 years ago

Marked as resolved until tested by the engine

Adam van Vliet 13 years ago

Matthew, has this been tested?

Thanks.

Matthew Clark 13 years ago

Yes, the combination allows for the use case to be met successfully. Closed.

Matthew Clark 13 years ago

Reopening

Matthew Clark 13 years ago

Discovered issue where daily exclusion timing was actually considering every day as inside the exclusion period. Added unit test, tested in Event Broker, and resolved. To be confirmed by another person tomorrow.

A way to test this is to add a schedule running every 12 hours from 5pm, not running Sunday to Friday between midnight and 11pm, and with a date exclusion of this Saturday.

Matthew Clark 13 years ago

Assigned to Tony to confirm resolution in the service.

Tony Sheehy 13 years ago

Tested and confirmed complete.

Customer support service by UserEcho