L option for Quartz Parser is not working

[gokhanoner]

Hi

When I try t use the Quartz Parser for below expression, I got error

expression:

0 0/10 22 L * ?

Error message

Exception in thread "main" java.lang.IllegalArgumentException: Number 0 out of range [1,31]
at com.cronutils.model.field.constraint.FieldConstraints.validateInRange(FieldConstraints.java:111)
at com.cronutils.model.field.expression.FieldExpression.validate(FieldExpression.java:48)
at com.cronutils.model.field.expression.On.<init>(On.java:52)
at com.cronutils.parser.field.FieldParser.parseOn(FieldParser.java:136)
at com.cronutils.parser.field.FieldParser.parse(FieldParser.java:51)
at com.cronutils.parser.field.CronParserField.parse(CronParserField.java:68)
at com.cronutils.parser.CronParser.parse(CronParser.java:88)

I believe the issue is FieldParser/parseOn method. It set 0 for this fileds IntegerFieldValue on On class and it raises exception because DAY_OF_MONTH should be between 1-31

    if (exp.contains("L")) {
        specialChar = new SpecialCharFieldValue(SpecialChar.L);
        exp = exp.replace("L", "");
        if ("".equals(exp)) {
            expression = "0";//to avoid a NumberFormatException
        } else {
            expression = exp;
        }
    }

[jmrozanec]

@gokhanoner good catch! Thank you for reporting this and for the insight where may the issue be!

[jmrozanec]

Test case for this issue:

/**
 * Issue #23: we should support L in DoM
 */
@Test
public void testLSupportedInDoM() throws Exception {
    parser.parse("0 0/10 22 L * ?");
}

[jmrozanec]

@gokhanoner issue fixed! Thank you for reporting!