Bug 103045 - Recurring event takes wrong monthly day and can't to be changed
Summary: Recurring event takes wrong monthly day and can't to be changed
Status: RESOLVED FIXED
Alias: None
Product: korganizer
Classification: Applications
Component: general (show other bugs)
Version: 3.4
Platform: RedHat Enterprise Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Reinhold Kainhofer
URL:
Keywords:
: 103160 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-04-02 03:07 UTC by Ilpo Kantonen
Modified: 2005-04-03 19:52 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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. ***