[#QUARTZ-353] Correct JavaDoc in CronExpression

Quartz Scheduler

Correct JavaDoc in CronExpression

Created: 05/Mar/06 02:19 AM Updated: 24/Dec/07 01:30 AM

Component/s:

Documentation

Affects Version/s:

1.5.2

Fix Version/s:

1.6

Issue Links:

Related

This issue is related to:
	QUARTZ-447 CronExpression allows any char after ...

Description

« Hide

In CronExpression, all mention of the 'C' character in expressions should be removed. This hasn't ever really been present, and was completely removed from the JavaDocs (or so I thought) in CronExpression. It seems I missed one, though.

The only location I see is in the table explaining the fields:
Day-of-month 1-31 , - * ? / L W C

C should be removed.

Incidentally, I wonder if an expression containing C would parse. If so, that should probably be fixed as well.

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

Jasper Rosenberg - [16/May/06 01:26 PM ]

Unfortunately, removing the explicit support for "C" doesn't keep the current CronExpression parsing code from accepting a token like "5C" because of the this issue: QUARTZ-447

[ Permlink | « Hide ]

Jasper Rosenberg - [16/May/06 01:29 PM ]

Once I removed the parsing support, it also made sense to remove the unused members calendardayOfWeek and calendardayOfMonth (as well as updating getExpressionSummary() to not ouput those values).