[seam-issues] [JBoss JIRA] Commented: (SEAMCRON-11) Add java.util.concurrent.TimeUnit attribute on @Every

Add java.util.concurrent.TimeUnit attribute on @Every ----------------------------------------------------- Key: SEAMCRON-11 URL: https://issues.jboss.org/browse/SEAMCRON-11 Project: Seam Cron Issue Type: Enhancement Reporter: Peter Royle Assignee: Peter Royle Fix For: 3.0.0.Alpha1 Idea courtesy of George Gastaldi. This would allow us to replace Second, Minute and Hour with the standard CronEvent for better consistency, without breaking the readability. public void doStuff(@Observes @Every(SECONDS) CronEvent Unfortunately there's a couple of teeny tiny hiccups: - TimeUnit only goes up to SECONDS in JDK 1.5, and I want to support 1.5 so that Seam Cron can be used n the various CDI JavaSE flavors. - TimeUnit values are pluralised which don't read very well, eg: @Observes @Every(HOURS) So obviously, we just create our own replacement for TimeUnit. I guess it will have to be ... CronTimeUnit? I should also note that this flies in the face of the possibility of implementing something like: @Observes @Every @Nth(5) Minute m We could maybe do something like this instead?: @Observes @Every(nth=5, value=MINUTE) CronEvent e