Re: [Pyatcron-devel-list] Scheduler Class Definition

[Julien Olivier]

I'm resending this email as I guess it was lost due to savannah being down.

---

On Fri, 2003-12-05 at 13:40, Childers, Matthew wrote:
> Hey all, I have attached a simple definition of the scheduler class.
> I've never done any serious documentation, so just let me know what
> needs to be added/removed.
> 
> Also, I changed the year attribute from an integer array to integer,
> since I figure that the year will only be used for At and can't be used
> in a cron command. 
> 
> Let me know what you think,
> Matt
> 

Thanks for this doc.

I have 2 comments:

 - Don't you think the "active" attribute belongs to the Scheduler task
too ? I mean, the activated/deactivated status defines whether the task
should executed or not. Doesn't that sound like a property for a
scheduler ? I wouldn't mind having it in the GenericTack class though...

 - You forgot the weekDays attribute. If someone wants to set a task to
run every mondays, you have to handle the weekDays as well as the days
(implicitly monthDays).

I attached a slightly modified diagram including the change for the year
attribute (int VS int array) and a renamed "monthDays" attribute
(instead of just days). It's a RFC too by the way.

Again, thanks for this and sorry for being late to answer.

-- 
Julien Olivier <address@hidden>

class-diagram.dia
Description: application/dia-diagram