Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid Exception on cron: 0 0 12 1W * ? * #75

Closed
DanoOM opened this issue Mar 29, 2016 · 2 comments
Closed

Invalid Exception on cron: 0 0 12 1W * ? * #75

DanoOM opened this issue Mar 29, 2016 · 2 comments
Labels

Comments

@DanoOM
Copy link
Contributor

DanoOM commented Mar 29, 2016

The above expression (Quartz) should return the first workday of the month. (i.e. the one closest to the '1st').

Instead an exception is thrown when calling nextExecution

java.lang.IllegalArgumentException: Values must not be empty
at org.apache.commons.lang3.Validate.notEmpty(Validate.java:300)
at com.cronutils.model.time.TimeNode.(TimeNode.java:31)
at com.cronutils.model.time.ExecutionTime.generateDays(ExecutionTime.java:320)
at com.cronutils.model.time.ExecutionTime.nextClosestMatch(ExecutionTime.java:169)
at com.cronutils.model.time.ExecutionTime.nextExecution(ExecutionTime.java:123)
at com.cronutils.model.time.ExecutionTimeQuartzIntegrationTest.testCronWithFirstWorkDayOfWeek(ExecutionTimeQuartzIntegrationTest.java:326)

Test pull-request coming..

DanoOM pushed a commit to DanoOM/cron-utils that referenced this issue Mar 30, 2016
@jmrozanec
Copy link
Owner

@DanoOM Thank you for fixing this! Will soon release a version with it.

@jmrozanec
Copy link
Owner

@DanoOM New version released! Please check details here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants