From: Art Cancro <ajc@citadel.org>
Date: Tue, 23 Sep 2008 17:36:11 +0000 (+0000)
Subject: recurrence editor
X-Git-Tag: v7.86~1914
X-Git-Url: https://code.citadel.org/?a=commitdiff_plain;ds=sidebyside;h=ab0f4b451e2ffb4f8af40ccf51b69a77ac29832a;p=citadel.git

recurrence editor
---

diff --git a/webcit/event.c b/webcit/event.c
index 4618e8bd6..dcce662e2 100644
--- a/webcit/event.c
+++ b/webcit/event.c
@@ -448,20 +448,13 @@ void display_edit_individual_event(icalcomponent *supplied_vevent, long msgnum,
 	wprintf("value=\"%d\"> ", recur.interval);
 
 	wprintf("<select name=\"freq\" id=\"freq_selector\" size=\"1\" "
-		"onclick=\"RecurrenceShowHide();\">\n");
+		"onChange=\"RecurrenceShowHide();\">\n");
 	for (i=0; i<(sizeof frequency_units / sizeof(char *)); ++i) {
-
-		/* We only want to offer daily, weekly, monthly, or yearly, which are
-		 * options 3 through 6, but if one of the others hapens to be selected,
-		 * we will grudgingly add it to the list of options.
-		 */
-		if ( (i == recur.freq) || ((i>=3) && (i<=6)) ) {
-			wprintf("<option %s value=\"%d\">%s</option>\n",
-				((i == recur.freq) ? "selected" : ""),
-				i,
-				frequency_units[i]
-			);
-		}
+		wprintf("<option %s value=\"%d\">%s</option>\n",
+			((i == recur.freq) ? "selected" : ""),
+			i,
+			frequency_units[i]
+		);
 	}
 
 	wprintf("</td></tr>\n");
diff --git a/webcit/static/wclib.js b/webcit/static/wclib.js
index 00d6c577f..7b61e03c0 100644
--- a/webcit/static/wclib.js
+++ b/webcit/static/wclib.js
@@ -994,6 +994,8 @@ function RecurrenceShowHide() {
 		$('rrule').style.display = 'none';
 	}
 
+	alert('selected index is ' + $('freq_selector').selectedIndex );
+
 	if ($('freq_selector').selectedIndex == 4) {
 		$('weekday_selector').style.display = 'block';
 	}