Tickets :: Comment on [#2150] Add support for recurring tasks

* Your Email Address
* Spam protection	Enter the letters below: .___.__ . .. .. . [__ [__)\| \|\ /\| \| \| \| \|/\\| \/ \|__\|
Comment	>> Horde_Form_Type_nag_recurrence should use $vars instead of > >> Util::getFormData() and Nag_Recurrence instead of > >> Horde_Date_Recurrance. And we probably have to use different constant > >> names to avoid collisions when showing tasks in Kronolith. I just see > >> you already did this, but you don't use them everywhere yet. > > > > Fixed. > > > >> > >> The varrenderers won't work the way you implemented them though. > >> Either you need different form fields for the different recurrence > >> settings, or you have to implement the complete widget in a single > >> cell, like any other form field. > > > > This is not yet fixed. I'll need to take a closer look at how var > renderers work. > > > >> > >> And you shouldn't create new tasks when completing one recurrence > >> instance. Instead, you should store the completion of a single > >> recurrence, similar to how exceptions are stored at the moment. And > >> of course we should (at some point, not necessarily from the start) > >> add the ability to actually create exceptions. > > > > Adapted the patch to the new completion support in the recurrence library.
Attachment
Watch this ticket