[dev] Re: [cvs] commit: horde/lib/iCalendar vevent.php

Mike Cochrane mike at graftonhall.co.nz
Mon Jan 20 00:49:02 PST 2003


> ----- Message from jan at horde.org ---------
> 
> Zitat von Chuck Hagenbuch <chuck at horde.org>:
> 
> > Quoting Mike Cochrane <mike at graftonhall.co.nz>:
> >
> > >   Log:
> > >   Add toKronolithHash() to convert the vEvent to a Kronolith event
> > hash.
> >
> > This *really* belongs in Kronolith, not in Horde.
> 
> No, we have these methods in the Data classes already. But it should be
> named toHash() then.
> 
> Jan.

I think Kronolith_Event should have a fromvEvent() eventually. This would save a
iCalender_vEvent::toHash() then a Kronoltih_Event::fromHash(), and do away with
the intermediate representation.

- Mike :-)



More information about the dev mailing list