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

Reported error contains other expression than the one provided #90

Closed
francisdb opened this issue May 25, 2016 · 8 comments
Closed

Reported error contains other expression than the one provided #90

francisdb opened this issue May 25, 2016 · 8 comments

Comments

@francisdb
Copy link
Contributor

parsing 0/1 * * * * * for quartz returns

Invalid CRON expression: Invalid cron expression: * * * * * *. Both, a day-of-week AND a day-of-month parameter, are not supported.

I would expect the error to contain the actual expression

@jmrozanec
Copy link
Owner

@francisdb This is a bug. The message should return the expression. Thank you for reporting it!

@jmrozanec
Copy link
Owner

@francisdb Fixed!

@francisdb
Copy link
Contributor Author

👍

@jmrozanec
Copy link
Owner

@francisdb The fix may not provide the same expression, but always one that is equivalent to the specified. Most times will be the same string.

@jmrozanec
Copy link
Owner

Fix published in release 4.0.1

@francisdb
Copy link
Contributor Author

@jmrozanec then this still is confusing...

@francisdb
Copy link
Contributor Author

with 5.0.5 we now get this:

Failed to parse '0/1 * * * * *'. Invalid cron expression: 0 * * * * *. Both, a day-of-week AND a day-of-month parameter, are not supported.

Which is ok for my part

@jmrozanec
Copy link
Owner

@francisdb great! Glad that you find the new message more useful. Thank you for the feedback!

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

No branches or pull requests

2 participants