| Summary | Recurring events when I delete a recurrence ALWAYS create an exception. |
| Queue | Kronolith |
| Queue Version | 2.0.2 |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | chuck (at) horde (dot) org |
| Requester | scalero (at) datadec (dot) es |
| Created | 3/7/05 (7703 days ago) |
| Due | |
| Updated | 3/31/05 (7679 days ago) |
| Assigned | 3/28/05 (7682 days ago) |
| Resolved | 3/31/05 (7679 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
2.- I was mistaken in this point. I thought the function
'hasActiveRecurrence' must to return only true if the event has more
than ONE recurrence. Sorry again.
Your patch seems optimal and I have already applied it.
Greetings and thanks.
Sebastian Calero.
State ⇒ Resolved
of pasting them.
2. I don't think that patch makes sense. Why wouldn't the
hasActiveRecurrences() call catch the case you're talking about?
I've committed a fix for what I've been able to reproduce.
diff deleventaction.php
line 38
< $event->addException(Util::getFormData('year'),
< Util::getFormData('month'),
< Util::getFormData('mday'));
---
$event->start->mday, $event->start->year) ==
mktime(0, 0, 0, Util::getFormData('month'),
Util::getFormData('mday'), Util::getFormData('year'))) {
$kronolith->deleteEvent($event->getID());
} else
$event->save();
< if ($event->hasRecurType(KRONOLITH_RECUR_NONE) ||
Util::getFormData('all') || !$event->hasActiveRecurrence()) {
---
($event->hasRecurType(KRONOLITH_RECUR_NONE) ||
Util::getFormData('all') || !$event->hasActiveRecurrence())) {
Assigned to Chuck Hagenbuch
State ⇒ Feedback
simply left over and shouldn't be there at all - please test what
happens when you just remove it?
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Recurring events when I delete a recurrence ALWAYS create an exception.
Queue ⇒ Kronolith
month, if I delete the recurrence for day 15 and click "Future" all is
OK.
But I select the same event and modify the end date again at 30 March
the recurrence of the day 15 not appear because has a exception.
I think we must add a exception only for the fisrt recurrence of the
recurring event.
POSIBLE FIX:
diff -r1.1 deleventaction.php
38,40c38,43
< $event->addException(Util::getFormData('year'),
< Util::getFormData('month'),
< Util::getFormData('mday'));
---
$event->start->mday, $event->start->year) ==
mktime(0, 0, 0, Util::getFormData('month'),
Util::getFormData('mday'), Util::getFormData('year'))) {
$event->addException(Util::getFormData('year'),
Util::getFormData('month'),
Util::getFormData('mday'));
}