Fix for the bug #333689 [calendar]

pchenthill <[email protected]>
Newsgroups gmane.comp.gnome.evolution.patches
Message-ID <[email protected]>
Hi,
   Have attached the fix for the bug. Set the menu label only if the
alarm interval is a valid one.

thanks, Chenthill.

_______________________________________________
Evolution-patches mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/evolution-patches
333689.diff (text/x-patch, 1.9 KB)
Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/calendar/ChangeLog,v
retrieving revision 1.3011
diff -u -p -r1.3011 ChangeLog
--- ChangeLog	6 Mar 2006 15:58:53 -0000	1.3011
+++ ChangeLog	7 Mar 2006 07:42:07 -0000
@@ -1,3 +1,10 @@
+2006-03-07  Chenthill Palanisamy  <[email protected]>
+
+	Fixes #333689
+	* gui/dialogs/event-page.c
+	(init_widgets): Check if the units is -1 before
+	adding the menu item.
+
 2006-03-06  Veerapuram Varadhan  <[email protected]>
 
 	**Fixes #321832
Index: gui/dialogs/event-page.c
===================================================================
RCS file: /cvs/gnome/evolution/calendar/gui/dialogs/event-page.c,v
retrieving revision 1.110
diff -u -p -r1.110 event-page.c
--- gui/dialogs/event-page.c	4 Mar 2006 10:41:20 -0000	1.110
+++ gui/dialogs/event-page.c	7 Mar 2006 07:42:09 -0000
@@ -2920,7 +2920,7 @@ init_widgets (EventPage *epage)
 	
 	switch (priv->alarm_units) {
 	case CAL_DAYS:
-		if (priv->alarm_interval != 1) {
+		if (priv->alarm_interval != -1) {
 			menu_label = g_strdup_printf (ngettext("%d day before appointment", "%d days before appointment", priv->alarm_interval), priv->alarm_interval);
 		} else {
 			priv->alarm_interval = -1;
@@ -2928,7 +2928,7 @@ init_widgets (EventPage *epage)
 		break;
 		
 	case CAL_HOURS:
-		if (priv->alarm_interval != 1) {
+		if (priv->alarm_interval != -1) {
 			menu_label = g_strdup_printf (ngettext("%d hour before appointment", "%d hours before appointment", priv->alarm_interval), priv->alarm_interval);
 		} else {
 			priv->alarm_interval = -1;
@@ -2936,7 +2936,7 @@ init_widgets (EventPage *epage)
 		break;
 		
 	case CAL_MINUTES:
-		if (priv->alarm_interval != 15) {
+		if (priv->alarm_interval != -1) {
 			menu_label = g_strdup_printf (ngettext("%d minute before appointement", "%d minutes before appointment", priv->alarm_interval), priv->alarm_interval);
 		} else {
 			priv->alarm_interval = -1;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.