Bug 103045

Summary:	Recurring event takes wrong monthly day and can't to be changed
Product:	[Applications] korganizer	Reporter:	Ilpo Kantonen <ilpo>
Component:	general	Assignee:	Reinhold Kainhofer <reinhold>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde
Priority:	NOR
Version:	3.4
Target Milestone:	---
Platform:	RedHat Enterprise Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Ilpo Kantonen 2005-04-02 03:07:14 UTC

Version:           3.4 (using KDE KDE 3.4.0)
Installed from:    RedHat RPMs
OS:                Linux

When I set an (time associated) event to first day of month recurring (for example 1 Jun 2005) the Korganizer sets it to last day of June. When I try to set it back to first day of month in event recurring dialog, the Korganizer changes it immediately back to last day monthly in the dialog when I push apply-button. So it's impossible to set it first day of month in that dialog. In general tab the day is first day of month but recurring places it to last day of month in the dialog. The monthly view of calendar shows it also to be in last day of June. Thus the day is last in the view with future month occurrencies. I guess that the bug is in the apply action in the recurring events dialog.

I think this bug is quite similair than bug #40261 and more close to #31198, but it's not the same bug.

Comment 1 Reinhold Kainhofer 2005-04-02 12:17:54 UTC

CVS commit by kainhofe: 

Fix stupid mistake (decreased the index to correct for 0-based indices, instead of increasing it).

BUG:103045


  M +1 -1      koeditorrecurrence.cpp   1.83


--- kdepim/korganizer/koeditorrecurrence.cpp  #1.82:1.83
@@ -377,5 +377,5 @@ int RecurMonthly::day()
   int day = mByDayCombo->currentItem();
   if ( day >= 31 ) day = 31-day-1;
-  else --day;
+  else ++day;
   return day;
 }

Comment 2 Reinhold Kainhofer 2005-04-02 12:18:51 UTC

CVS commit by kainhofe: 

Backport of fix for bug 103045 to the KDE_3_4_BRANCH.

CCBUGS:103045


  M +1 -1      koeditorrecurrence.cpp   1.82.2.1


--- kdepim/korganizer/koeditorrecurrence.cpp  #1.82:1.82.2.1
@@ -377,5 +377,5 @@ int RecurMonthly::day()
   int day = mByDayCombo->currentItem();
   if ( day >= 31 ) day = 31-day-1;
-  else --day;
+  else ++day;
   return day;
 }

Comment 3 Reinhold Kainhofer 2005-04-03 19:52:40 UTC

*** Bug 103160 has been marked as a duplicate of this bug. ***