Backport sieve editor fixes from git master
authorArt Cancro <ajc@uncensored.citadel.org>
Thu, 8 Mar 2012 18:30:20 +0000 (13:30 -0500)
committerArt Cancro <ajc@uncensored.citadel.org>
Thu, 8 Mar 2012 18:30:20 +0000 (13:30 -0500)
webcit/sieve.c
webcit/static/t/sieve/add.html
webcit/static/t/sieve/list.html

index 0f34442d34efafa4c740462b740f3b3f633a2f61..6b7279365e5285ec1f675cfc092567ae2b7ca4a1 100644 (file)
@@ -1,19 +1,13 @@
 /*
- * Copyright (c) 1996-2011 by the citadel.org team
+ * Copyright (c) 1996-2012 by the citadel.org team
  *
  * This program is open source software.  You can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 3 of the
- * License, or (at your option) any later version.
+ * modify it under the terms of the GNU General Public License version 3.
  *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
  */
 
 #include "webcit.h"
@@ -21,7 +15,8 @@
 #define MAX_SCRIPTS    100
 #define MAX_RULES      50
 #define RULES_SCRIPT   "__WebCit_Generated_Script__"
-/*#define FOO 1*/
+
+
 /*
  * Helper function for output_sieve_rule() to output strings with quotes escaped
  */
@@ -40,13 +35,6 @@ void osr_sanitize(char *str) {
        }
 }
 
-void display_add_remove_scripts(char *message);
-void display_rules_editor_inner_div(void);
-
-
-
-
-
 
 /*
  * Output parseable Sieve script code based on rules input
@@ -201,7 +189,6 @@ void output_sieve_rule(char *hfield, char *compare, char *htext, char *sizecomp,
                serv_printf("{");
        }
 
-
        /* Do action */
 
        if (!strcasecmp(action, "keep")) {
@@ -228,26 +215,22 @@ void output_sieve_rule(char *hfield, char *compare, char *htext, char *sizecomp,
                serv_printf("vacation :addresses [%s]\n\"%s\";", my_addresses, automsg);
        }
 
-
        /* Do 'final' action */
 
        if (!strcasecmp(final, "stop")) {
                serv_printf("stop;");
        }
 
-
        /* Close the braces if we're in a conditional loop */
 
        if (strcasecmp(hfield, "all")) {
                serv_printf("}");
        }
 
-
        /* End of rule. */
 }
 
 
-
 /*
  * Translate the fields from the rule editor into something we can save...
  */
@@ -364,7 +347,6 @@ void parse_fields_from_rule_editor(void) {
 }
 
 
-
 /*
  * save sieve config
  */
@@ -372,7 +354,6 @@ void save_sieve(void) {
        int bigaction;
        char script_names[MAX_SCRIPTS][64];
        int num_scripts = 0;
-       int active_script = (-1);       /* this throws a 'set but not used' warning , check this ! */
        int i;
        char this_name[64];
        char buf[256];
@@ -390,9 +371,6 @@ void save_sieve(void) {
        if (buf[0] == '1') while (serv_getln(buf, sizeof(buf)), strcmp(buf, "000")) {
                if (num_scripts < MAX_SCRIPTS) {
                        extract_token(script_names[num_scripts], buf, 0, '|', 64);
-                       if (extract_int(buf, 1) > 0) {
-                               active_script = num_scripts;
-                       }
                        ++num_scripts;
                }
        }
@@ -439,6 +417,7 @@ void save_sieve(void) {
        return;
 }
 
+
 /*
  * create a new script
  * take the web environment script name and create it on the citadel server
@@ -452,9 +431,6 @@ void create_script(void) {
                while (serv_getln(buf, sizeof(buf)), strcmp(buf, "000")) {
                        /* flush */
                }
-#if FOO
-               display_add_remove_scripts(_("A script by that name already exists."));
-#endif
                return;
        }
        
@@ -463,28 +439,18 @@ void create_script(void) {
        if (buf[0] == '4') {
                serv_puts("keep;");
                serv_puts("000");
-#if FOO
-               display_add_remove_scripts(_("A new script has been created.  Return to the script editing screen to edit and activate it."));
-#else
        output_headers(1, 1, 2, 0, 0, 0);
        do_template("sieve_add");
        wDumpContent(1);
-#endif
                return;
        }
 
-#if FOO
-       display_add_remove_scripts(&buf[4]);
-#else
        output_headers(1, 1, 2, 0, 0, 0);
        do_template("sieve_add");
        wDumpContent(1);
-#endif
 }
 
 
-
-
 /*
  * delete a script
  */
@@ -493,15 +459,10 @@ void delete_script(void) {
 
        serv_printf("MSIV deletescript|%s", bstr("script_name"));
        serv_getln(buf, sizeof buf);
-#if FOO
-       display_add_remove_scripts(&buf[4]);
-#else
        output_headers(1, 1, 2, 0, 0, 0);
        do_template("sieve_add");
        wDumpContent(1);
-#endif
 }
-               
 
 
 /*
@@ -514,687 +475,6 @@ void display_no_sieve(void) {
        wDumpContent(1);
 }
 
-#if FOO
-/*
- * view/edit sieve config
- */
-void display_sieve(void)
-{
-       char script_names[MAX_SCRIPTS][64];
-       int num_scripts = 0;
-       int active_script = (-1);
-       char buf[SIZ];          /* Don't make this buffer smaller or it will restrict line length */
-       int i;
-       int rules_script_is_active = 0;
-
-       if (!WC->serv_info->serv_supports_sieve) {
-               display_no_sieve();
-               return;
-       }
-
-       memset(script_names, 0, sizeof script_names);
-
-       serv_puts("MSIV listscripts");
-       serv_getln(buf, sizeof(buf));
-       if (buf[0] == '1') while (serv_getln(buf, sizeof(buf)), strcmp(buf, "000")) {
-               if (num_scripts < MAX_SCRIPTS) {
-                       extract_token(script_names[num_scripts], buf, 0, '|', 64);
-                       if (extract_int(buf, 1) > 0) {
-                               active_script = num_scripts;
-                               if (!strcasecmp(script_names[num_scripts], RULES_SCRIPT)) {
-                                       rules_script_is_active = 1;
-                               }
-                       }
-                       ++num_scripts;
-               }
-       }
-
-       output_headers(1, 1, 2, 0, 0, 0);
-
-       wc_printf("<script type=\"text/javascript\">\n"
-                 "\n"
-                 "var previously_active_script;\n"
-                 "\n"
-                 "function ToggleSievePanels() {\n"
-                 " d = ($('sieveform').bigaction.options[$('sieveform').bigaction.selectedIndex].value);\n"
-                 " for (i=0; i<3; ++i) {\n"
-                 "  if (i == d) {\n"
-                 "   $('sievediv' + i).style.display = 'block';\n"
-                 "  }\n"
-                 "  else {\n"
-                 "   $('sievediv' + i).style.display = 'none';\n"
-                 "  }\n"
-                 " }\n"
-                 "}\n"
-                 "\n"
-                 "function ToggleScriptPanels() {\n"
-                 " d = ($('sieveform').active_script.options[$('sieveform').active_script.selectedIndex].value);\n"
-                 " if ($('script_' + previously_active_script)) {\n"
-                 "  $('script_' + previously_active_script).style.display = 'none';\n"
-                 " }\n"
-                 " $('script_' + d).style.display = 'block';\n"
-                 " previously_active_script = d;\n"
-                 "}\n"
-                 "\n"
-                 "</script>\n"
-);
-
-       wc_printf("<div id=\"banner\">\n");
-       wc_printf("<img src=\"static/webcit_icons/essen/32x32/config.png\">");
-       wc_printf("<h1>");
-       wc_printf(_("View/edit server-side mail filters"));
-       wc_printf("</h1>\n");
-       wc_printf("</div>\n");
-
-       wc_printf("<div id=\"content\" class=\"service\">\n");
-
-       wc_printf("<table class=\"sieve_background\">\n"
-               "<tr>\n<td valign=top>\n");
-
-
-       wc_printf("<form id=\"sieveform\" method=\"post\" action=\"save_sieve\">\n");
-       wc_printf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
-
-       wc_printf(_("When new mail arrives: "));
-        wc_printf("<select name=\"bigaction\" size=1 onChange=\"ToggleSievePanels();\">\n");
-
-       wc_printf("<option %s value=\"0\">", ((active_script < 0) ? "selected" : ""));
-       wc_printf(_("Leave it in my inbox without filtering"));
-       wc_printf("</option>\n");
-
-       wc_printf("<option %s value=\"1\">", ((rules_script_is_active) ? "selected" : ""));
-       wc_printf(_("Filter it according to rules selected below"));
-       wc_printf("</option>\n");
-
-       wc_printf("<option %s value=\"2\">",
-                       (((active_script >= 0) && (!rules_script_is_active)) ? "selected" : ""));
-       wc_printf(_("Filter it through a manually edited script (advanced users only)"));
-       wc_printf("</option>\n");
-
-       wc_printf("</select>\n\n");
-
-
-
-       /* The "no filtering" div */
-
-       wc_printf("<div id=\"sievediv0\" style=\"display:none\">\n");
-       wc_printf("<div align=\"center\"><br><br>");
-       wc_printf(_("Your incoming mail will not be filtered through any scripts."));
-       wc_printf("<br><br></div>\n");
-       wc_printf("</div>\n");
-
-       /* The "webcit managed scripts" div */
-
-       wc_printf("<div id=\"sievediv1\" style=\"display:none\">\n");
-       display_rules_editor_inner_div();
-       wc_printf("</div>\n");
-
-       /* The "I'm smart and can write my own Sieve scripts" div */
-
-       wc_printf("<div id=\"sievediv2\" style=\"display:none\">\n");
-
-       if (num_scripts > 0) {
-               wc_printf(_("The currently active script is: "));
-               wc_printf("<select name=\"active_script\" size=1 onChange=\"ToggleScriptPanels();\">\n");
-               for (i=0; i<num_scripts; ++i) {
-                       if (strcasecmp(script_names[i], RULES_SCRIPT)) {
-                               wc_printf("<option %s value=\"%s\">%s</option>\n",
-                                       ((active_script == i) ? "selected" : ""),
-                                       script_names[i],
-                                       script_names[i]
-                               );
-                       }
-               }
-               wc_printf("</select>\n\n");
-       }
-
-       wc_printf("&nbsp;&nbsp;&nbsp;");
-       wc_printf("<a href=\"display_add_remove_scripts\">%s</a>\n", _("Add or delete scripts"));
-
-       wc_printf("<br>\n");
-
-       if (num_scripts > 0) {
-               for (i=0; i<num_scripts; ++i) {
-                       if (strcasecmp(script_names[i], RULES_SCRIPT)) {
-                               wc_printf("<div id=\"script_%s\" style=\"display:none\">\n", script_names[i]);
-                               wc_printf("<textarea name=\"text_%s\" wrap=soft rows=20 cols=80 width=80>\n",
-                                       script_names[i]);
-                               serv_printf("MSIV getscript|%s", script_names[i]);
-                               serv_getln(buf, sizeof buf);
-                               if (buf[0] == '1') while(serv_getln(buf, sizeof (buf)), strcmp(buf, "000")) {
-                                       wc_printf("%s\n", buf);
-                               }
-                               wc_printf("</textarea>\n");
-                               wc_printf("</div>\n");
-                       }
-               }
-       }
-
-       wc_printf("<script type=\"text/javascript\">    \n"
-               "ToggleScriptPanels();                  \n"
-               "</script>                              \n"
-       );
-
-       wc_printf("</div>\n");
-
-
-       /* The rest of this is common for all panels... */
-
-       wc_printf("<div align=\"center\"><br>");
-       wc_printf("<input type=\"submit\" name=\"save_button\" value=\"%s\">", _("Save changes"));
-       wc_printf("&nbsp;");
-       wc_printf("<input type=\"submit\" name=\"cancel_button\" value=\"%s\">\n", _("Cancel"));
-       wc_printf("</div></form>\n");
-
-       wc_printf("</td></tr></table>\n");
-
-       wc_printf("<script type=\"text/javascript\">    \n"
-               "ToggleSievePanels();                   \n"
-               "</script>                              \n"
-       );
-
-       wDumpContent(1);
-
-}
-
-
-
-
-/*
- * show a list of available scripts to add/remove them
- */
-void display_add_remove_scripts(char *message)
-{
-       char buf[256];
-       char script_name[256];
-
-       output_headers(1, 1, 2, 0, 0, 0);
-       wc_printf("<div id=\"banner\">\n");
-       wc_printf("<img src=\"static/webcit_icons/essen/32x32/config.png\">");
-       wc_printf(_("Add or delete scripts"));
-       wc_printf("</h1>\n");
-       wc_printf("</div>\n");
-       
-       wc_printf("<div id=\"content\" class=\"service\">\n");
-
-       if (message != NULL) {
-               wc_printf("%s", message);
-       }
-
-       wc_printf("<table border=0 cellspacing=10><tr valign=top><td>\n");
-
-       do_template("box_begin_1");
-       StrBufAppendBufPlain(WC->WBuf, _("Add a new script"), -1, 0);
-       do_template("box_begin_2");
-
-       wc_printf(_("To create a new script, enter the desired "
-               "script name in the box below and click 'Create'."));
-       wc_printf("<br><br>");
-
-        wc_printf("<center><form method=\"POST\" action=\"create_script\">\n");
-       wc_printf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
-        wc_printf(_("Script name: "));
-        wc_printf("<input type=\"text\" name=\"script_name\"><br>\n"
-               "<input type=\"submit\" name=\"create_button\" value=\"%s\">"
-               "</form></center>\n", _("Create"));
-
-       do_template("box_end");
-
-       do_template("box_begin_1");
-       StrBufAppendBufPlain(WC->WBuf, _("Edit scripts"), -1, 0);
-       do_template("box_begin_2");
-       wc_printf("<br><div align=center><a href=\"display_sieve\">%s</a><br><br>\n",
-               _("Return to the script editing screen")
-       );
-       do_template("box_end");
-
-       wc_printf("</td><td>");
-
-       do_template("box_begin_1");
-       StrBufAppendBufPlain(WC->WBuf, _("Delete scripts"), -1, 0);
-       do_template("box_begin_2");
-
-       wc_printf(_("To delete an existing script, select the script "
-               "name from the list and click 'Delete'."));
-       wc_printf("<br><br>");
-       
-        wc_printf("<center>"
-               "<form method=\"POST\" action=\"delete_script\">\n");
-       wc_printf("<input type=\"hidden\" name=\"nonce\" value=\"%d\">\n", WC->nonce);
-        wc_printf("<select name=\"script_name\" size=10 style=\"width:100%%\">\n");
-
-        serv_puts("MSIV listscripts");
-        serv_getln(buf, sizeof buf);
-        if (buf[0] == '1') {
-                while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                        extract_token(script_name, buf, 0, '|', sizeof script_name);
-                       if ( (extract_int(buf, 1) == 0) && (strcasecmp(script_name, RULES_SCRIPT)) ) {
-                               wc_printf("<option>");
-                               escputs(script_name);
-                               wc_printf("</option>\n");
-                       }
-                }
-        }
-        wc_printf("</select>\n\n<br>\n");
-
-        wc_printf("<input type=\"submit\" name=\"delete_button\" value=\"%s\" "
-               "onClick=\"return confirm('%s');\">", _("Delete script"), _("Delete this script?"));
-        wc_printf("</form></center>\n");
-       do_template("box_end");
-
-       wc_printf("</td></tr></table>\n");
-
-       wDumpContent(1);
-}
-
-
-
-
-
-
-void display_rules_editor_inner_div(void) {
-       int i, j;
-       char buf[4096];
-       char rules[MAX_RULES][2048];
-
-       struct {
-               char name[128];
-       } *rooms = NULL;
-       int num_roomnames = 0;
-       int num_roomnames_alloc = 0;
-
-       int active;
-       char hfield[256];
-       char compare[32];
-       char htext[256];
-       char sizecomp[32];
-       int sizeval;
-       char action[32];
-       char fileinto[128];
-       char redirect[256];
-       char automsg[1024];
-       char final[32];
-
-       /* load the rules */
-       memset(rules, 0, sizeof rules);
-       serv_printf("MSIV getscript|%s", RULES_SCRIPT);
-       serv_getln(buf, sizeof buf);
-       if (buf[0] == '1') while(serv_getln(buf, sizeof (buf)), strcmp(buf, "000")) {
-               if (!strncasecmp(buf, "# WEBCIT_RULE|", 14)) {
-                       j = extract_int(buf, 1);
-                       remove_token(buf, 0, '|');
-                       remove_token(buf, 0, '|');
-                       CtdlDecodeBase64(rules[j], buf, strlen(buf));
-               }
-       }
-
-       /* load the roomnames */
-       serv_puts("LKRA");
-       serv_getln(buf, sizeof buf);
-       if (buf[0] == '1') {
-               while (serv_getln(buf, sizeof buf), strcmp(buf, "000")) {
-                       ++num_roomnames;
-                       if (num_roomnames > num_roomnames_alloc) {
-                               num_roomnames_alloc += 250;
-                               rooms = realloc(rooms, (num_roomnames_alloc * 128));
-                       }
-                       extract_token(rooms[num_roomnames-1].name, buf, 0, '|', 128);
-               }
-       }
-
-
-/*
- * This script should get called by every onChange event...
- *
- */
-       wc_printf("<script type=\"text/javascript\">\n"
-                 "\n"
-                 "var highest_active_rule = (-1);\n"
-                 "\n"
-                 "function UpdateRules() {\n");
-/*
- * Show only the active rows...
- */
-       wc_printf("  highest_active_rule = (-1);\n");
-       wc_printf("  for (i=0; i<%d; ++i) {\n", MAX_RULES);
-       wc_printf("   if ($('active'+i).checked) {\n"
-                 "     $('rule' + i).style.display = 'block';\n"
-                 "     highest_active_rule = i;\n"
-                 "   }\n"
-                 "   else {\n"
-                 "     $('rule' + i).style.display = 'none';\n"
-                 "   }\n"
-                 "  }\n");
-/*
- * Show only the fields relevant to the rules...
- */
-       wc_printf("  for (i=0; i<=highest_active_rule; ++i) {\n"
-                 "    d = ($('movedown'+i));\n"
-                 "    if (i < highest_active_rule) {\n"
-                 "      d.style.display = 'block';\n"
-                 "    }\n"
-                 "    else {\n"
-                 "      d.style.display = 'none';\n"
-                 "    }\n"
-                 "    d = ($('hfield'+i).options[$('hfield'+i).selectedIndex].value);\n"
-                 "    if (d == 'all') {\n"
-                 "      $('div_size'+i).style.display = 'none'; \n"
-                 "      $('div_compare'+i).style.display = 'none';\n"
-                 "      $('div_nocompare'+i).style.display = 'block';\n"
-                 "    }\n"
-                 "    else if (d == 'size') {\n"
-                 "      $('div_size'+i).style.display = 'block'; \n"
-                 "      $('div_compare'+i).style.display = 'none';\n"
-                 "      $('div_nocompare'+i).style.display = 'none';\n"
-                 "    }\n"
-                 "    else {\n"
-                 "      $('div_size'+i).style.display = 'none'; \n"
-                 "      $('div_compare'+i).style.display = 'block';\n"
-                 "      $('div_nocompare'+i).style.display = 'none';\n"
-                 "    }\n"
-                 "    d = ($('action'+i).options[$('action'+i).selectedIndex].value);\n"
-                 "    if (d == 'fileinto') {\n"
-                 "      $('div_fileinto'+i).style.display = 'block';\n"
-                 "      $('div_redirect'+i).style.display = 'none';\n"
-                 "      $('div_automsg'+i).style.display = 'none';\n"
-                 "    } else if (d == 'redirect') {\n"
-                 "      $('div_fileinto'+i).style.display = 'none';\n"
-                 "      $('div_redirect'+i).style.display = 'block';\n"
-                 "      $('div_automsg'+i).style.display = 'none';\n"
-                 "    } else if ((d == 'reject') || (d == 'vacation'))  {\n"
-                 "      $('div_fileinto'+i).style.display = 'none';\n"
-                 "      $('div_redirect'+i).style.display = 'none';\n"
-                 "      $('div_automsg'+i).style.display = 'block';\n"
-                 "    } else {\n"
-                 "      $('div_fileinto'+i).style.display = 'none';\n"
-                 "      $('div_redirect'+i).style.display = 'none';\n"
-                 "      $('div_automsg'+i).style.display = 'none';\n"
-                 "    }\n"
-                 "    if (highest_active_rule < %d) {\n", MAX_RULES-1 );
-       wc_printf("      $('div_addrule').style.display = 'block';\n"
-                 "    } else {\n"
-                 "      $('div_addrule').style.display = 'none';\n"
-                 "    }\n"
-                 "  }\n"
-                 "}\n"
-/*
- * Add a rule (really, just un-hide it)
- */
-                 "function AddRule() {\n"
-                 "  highest_active_rule = highest_active_rule + 1;\n"
-                 "  $('active'+highest_active_rule).checked = true;\n"
-                 "  UpdateRules();\n"
-                 "}\n"
-/*
- * Swap two rules
- */
-                 "function SwapRules(ra, rb) {\n"
-                 "\n"
-                 "  var things = new Array();\n"
-                 "  things[0] = 'hfield';\n"
-                 "  things[1] = 'compare';\n"
-                 "  things[2] = 'htext';\n"
-                 "  things[3] = 'action';\n"
-                 "  things[4] = 'fileinto';\n"
-                 "  things[5] = 'redirect';\n"
-                 "  things[6] = 'final';\n"
-                 "  things[7] = 'sizecomp';\n"
-                 "  things[8] = 'sizeval';\n"
-                 "  things[9] = 'automsg';\n"
-                 "\n"
-                 "  for (i=0; i<=9; ++i) {\n"
-                 "    tempval=$(things[i]+ra).value;\n"
-                 "    $(things[i]+ra).value = $(things[i]+rb).value;\n"
-                 "    $(things[i]+rb).value = tempval;\n"
-                 "  }\n"
-                 "}\n"
-/*
- * Delete a rule (percolate the deleted rule out to the end, then deactivate it)
- */
-                 "function DeleteRule(rd) {\n"
-                 "  for (j=rd; j<=highest_active_rule; ++j) {\n"
-                 "    SwapRules(j, (j+1));\n"
-                 "  }\n"
-                 "  $('active'+highest_active_rule).checked = false;\n"
-                 "}\n"
-                 "</script>\n"
-               );
-
-
-       wc_printf("<br>");
-
-       wc_printf("<table cellpadding=2 width=100%%>");
-
-       for (i=0; i<MAX_RULES; ++i) {
-
-               /* Grab our existing values to populate */
-               active = extract_int(rules[i], 0);
-               extract_token(hfield, rules[i], 1, '|', sizeof hfield);
-               extract_token(compare, rules[i], 2, '|', sizeof compare);
-               extract_token(htext, rules[i], 3, '|', sizeof htext);
-               extract_token(sizecomp, rules[i], 4, '|', sizeof sizecomp);
-               sizeval = extract_int(rules[i], 5);
-               extract_token(action, rules[i], 6, '|', sizeof action);
-               extract_token(fileinto, rules[i], 7, '|', sizeof fileinto);
-               extract_token(redirect, rules[i], 8, '|', sizeof redirect);
-               extract_token(automsg, rules[i], 9, '|', sizeof automsg);
-               extract_token(final, rules[i], 10, '|', sizeof final);
-               
-               /* now generate the table row */
-
-               wc_printf("<tr id=\"rule%d\" class=\"%s\">",
-                       i,
-                       ((i%2) ? "odd" : "even")
-               );
-
-               wc_printf("<td width=5%% align=\"center\">\n");
-
-               wc_printf("<div style=\"display:none\">\n");
-               wc_printf("<input type=\"checkbox\" name=\"active%d\" id=\"active%d\" %s>\n",
-                       i, i,
-                       (active ? "checked" : "")
-               );
-               wc_printf("</div>\n");
-
-               if (i>0) wc_printf("<a href=\"javascript:SwapRules(%d,%d);UpdateRules();\">"
-                       "<img border=\"0\" src=\"static/webcit_icons/up_pointer.gif\" "
-                       "title=\"%s\"/></a>\n",
-                       i-1, i, _("Move rule up") );
-
-               wc_printf("<a href=\"javascript:SwapRules(%d,%d);UpdateRules();\">"
-                       "<img id=\"movedown%d\" border=\"0\" src=\"static/webcit_icons/down_pointer.gif\" "
-                       "title=\"%s\"/></a>\n",
-                       i, i+1, i, _("Move rule down") );
-
-               wc_printf("<a href=\"javascript:DeleteRule(%d);UpdateRules();\">"
-                       "<img id=\"delete%d\" border=\"0\" src=\"static/webcit_icons/delete.gif\" "
-                       "title=\"%s\"/></a>\n",
-                       i, i, _("Delete rule") );
-
-               wc_printf("</td>\n\n\n");
-
-               wc_printf("<td width=5%% align=\"center\">\n");
-               wc_printf("<font size=+2>%d</font>\n", i+1);
-               wc_printf("</td>\n");
-
-               wc_printf("<td width=20%%>%s ", _("If") );
-
-               char *hfield_values[15][2] = {
-                       {       "from",         _("From")               },
-                       {       "tocc",         _("To or Cc")           },
-                       {       "subject",      _("Subject")            },
-                       {       "replyto",      _("Reply-to")           },
-                       {       "sender",       _("Sender")             },
-                       {       "resentfrom",   _("Resent-From")        },
-                       {       "resentto",     _("Resent-To")          },
-                       {       "envfrom",      _("Envelope From")      },
-                       {       "envto",        _("Envelope To")        },
-                       {       "xmailer",      _("X-Mailer")           },
-                       {       "xspamflag",    _("X-Spam-Flag")        },
-                       {       "xspamstatus",  _("X-Spam-Status")      },
-                       {       "listid",       _("List-ID")            },
-                       {       "size",         _("Message size")       },
-                       {       "all",          _("All")                }
-               };
-
-               wc_printf("<select id=\"hfield%d\" name=\"hfield%d\" size=1 onChange=\"UpdateRules();\">",
-                       i, i);
-               for (j=0; j<15; ++j) {
-                       wc_printf("<option %s value=\"%s\">%s</option>\n",
-                               ( (!strcasecmp(hfield, hfield_values[j][0])) ? "selected" : ""),
-                               hfield_values[j][0],
-                               hfield_values[j][1]
-                       );
-               }
-
-               wc_printf("</select>\n\n");
-               wc_printf("</td>");
-
-               wc_printf("<td width=20%%>");
-
-               char *compare_values[6][2] = {
-                       {       "contains",     _("contains")           },
-                       {       "notcontains",  _("does not contain")   },
-                       {       "is",           _("is")                 },
-                       {       "isnot",        _("is not")             },
-                       {       "matches",      _("matches")            },
-                       {       "notmatches",   _("does not match")     }
-               };
-
-               wc_printf("<div id=\"div_compare%d\">\n", i);
-               wc_printf("<select id=\"compare%d\" name=\"compare%d\" size=1 onChange=\"UpdateRules();\">\n",
-                       i, i);
-               for (j=0; j<6; ++j) {
-                       wc_printf("<option %s value=\"%s\">%s</option>\n",
-                               ( (!strcasecmp(compare, compare_values[j][0])) ? "selected" : ""),
-                               compare_values[j][0],
-                               compare_values[j][1]
-                       );
-               }
-               wc_printf("</select>\n\n");
-
-               wc_printf("<input type=\"text\" id=\"htext%d\" name=\"htext%d\" value=\"", i, i);
-               escputs(htext);
-               wc_printf("\">\n</div>\n");
-
-               wc_printf("<div id=\"div_nocompare%d\">", i);
-               wc_printf("%s", _("(All messages)"));
-               wc_printf("</div>\n");
-
-               char *sizecomp_values[2][2] = {
-                       {       "larger",       _("is larger than")     },
-                       {       "smaller",      _("is smaller than")    }
-               };
-
-               wc_printf("<div id=\"div_size%d\">\n", i);
-               wc_printf("<select id=\"sizecomp%d\" name=\"sizecomp%d\" size=1 onChange=\"UpdateRules();\">\n",
-                       i, i);
-               for (j=0; j<2; ++j) {
-                       wc_printf("<option %s value=\"%s\">%s</option>\n",
-                               ( (!strcasecmp(sizecomp, sizecomp_values[j][0])) ? "selected" : ""),
-                               sizecomp_values[j][0],
-                               sizecomp_values[j][1]
-                       );
-               }
-               wc_printf("</select>\n\n");
-
-               wc_printf("<input type=\"text\" id=\"sizeval%d\" name=\"sizeval%d\" value=\"%d\">",
-                       i, i, sizeval);
-               wc_printf("bytes");
-               wc_printf("</div>");
-
-               wc_printf("</td>");
-
-               char *action_values[6][2] = {
-                       {       "keep",         _("Keep")               },
-                       {       "discard",      _("Discard silently")   },
-                       {       "reject",       _("Reject")             },
-                       {       "fileinto",     _("Move message to")    },
-                       {       "redirect",     _("Forward to")         },
-                       {       "vacation",     _("Vacation")           }
-               };
-
-               wc_printf("<td width=20%%>\n");
-               wc_printf("<select id=\"action%d\" name=\"action%d\" size=1 onChange=\"UpdateRules();\">\n",
-                       i, i);
-               for (j=0; j<6; ++j) {
-                       wc_printf("<option %s value=\"%s\">%s</option>\n",
-                               ( (!strcasecmp(action, action_values[j][0])) ? "selected" : ""),
-                               action_values[j][0],
-                               action_values[j][1]
-                       );
-               }
-               wc_printf("</select>\n\n");
-
-               wc_printf("<div id=\"div_fileinto%d\">\n", i);
-               wc_printf("<select name=\"fileinto%d\" id=\"fileinto%d\">\n", i, i);
-               for (j=0; j<num_roomnames; ++j) {
-                       wc_printf("<option ");
-                       if (!strcasecmp(rooms[j].name, fileinto)) {
-                               wc_printf("selected ");
-                       }
-                       wc_printf("value=\"");
-                       escputs(rooms[j].name);
-                       wc_printf("\">");
-                       escputs(rooms[j].name);
-                       wc_printf("</option>\n");
-               }
-               wc_printf("</select>\n\n");
-               wc_printf("</div>");
-
-               wc_printf("<div id=\"div_redirect%d\">\n", i);
-               wc_printf("<input type=\"text\" id=\"redirect%d\" name=\"redirect%d\" value=\"", i, i);
-               escputs(redirect);
-               wc_printf("\">\n</div>\n");
-
-               wc_printf("<div id=\"div_automsg%d\">\n", i);
-               wc_printf(_("Message:"));
-               wc_printf("<br>\n");
-               wc_printf("<textarea name=\"automsg%d\" id=\"automsg%d\" wrap=soft rows=5>\n", i, i);
-               escputs(automsg);
-               wc_printf("</textarea>");
-               wc_printf("</div>\n");
-
-               wc_printf("</td>\n");
-
-               char *final_values[2][2] = {
-                       {       "continue",     _("continue processing")        },
-                       {       "stop",         _("stop")                       }
-               };
-
-               wc_printf("<td width=10%% align=\"center\">%s</td>\n", _("and then") );
-
-               wc_printf("<td width=20%%>\n");
-               wc_printf("<select name=\"final%d\" id=\"final%d\" size=1 onChange=\"UpdateRules();\">\n",
-                       i, i);
-               for (j=0; j<2; ++j) {
-                       wc_printf("<option %s value=\"%s\">%s</option>\n",
-                               ( (!strcasecmp(final, final_values[j][0])) ? "selected" : ""),
-                               final_values[j][0],
-                               final_values[j][1]
-                       );
-               }
-               wc_printf("</select>\n\n");
-               wc_printf("</td>\n");
-
-               wc_printf("</tr>\n");
-
-       }
-
-       wc_printf("</table>\n");
-       wc_printf("<div id=\"div_addrule\"><a href=\"javascript:AddRule();\">%s</a><br></div>\n",
-               _("Add rule")
-       );
-
-       wc_printf("<script type=\"text/javascript\">\n");
-       wc_printf("UpdateRules();\n");
-       wc_printf("</script>\n");
-
-       free(rooms);
-}
-void _display_add_remove_scripts(void) {display_add_remove_scripts(NULL);}
-#endif
-
 
 typedef struct __SieveListing {
        int IsActive;
@@ -1243,8 +523,9 @@ HashList *GetSieveScriptListing(StrBuf *Target, WCTemplputParams *TP)
        int Done = 0;
        SieveListing *Ruleset;
 
-       if (WCC->KnownSieveScripts != NULL)
+       if (WCC->KnownSieveScripts != NULL) {
                return WCC->KnownSieveScripts;
+       }
 
        serv_puts("MSIV listscripts");
        Line = NewStrBuf();
@@ -1283,8 +564,10 @@ HashList *GetSieveScriptListing(StrBuf *Target, WCTemplputParams *TP)
                                ++num_scripts;
                        }
        }
-       if ((num_scripts > 0) && (rules_script_active == 0))
+
+       if ((num_scripts > 0) && (rules_script_active == 0)) {
                PutBstr(HKEY("__SIEVE:EXTERNAL_SCRIPT"), NewStrBufPlain(HKEY("1")));
+       }
 
        if (num_scripts > have_rules_script)
        {
@@ -1304,18 +587,14 @@ HashList *GetSieveScriptListing(StrBuf *Target, WCTemplputParams *TP)
                {
                        Ruleset = (SieveListing *) vRuleset;
 
-                       /*
-                        * its the webcit rule? we don't need to load that here.
-                        */
-                       if (Ruleset->IsRulesScript)
-                               continue;
+                       // FIXME add logic to skip if it's the webcit generated script
 
-                       if (!serv_printf("MSIV getscript|%s", ChrPtr(Ruleset->Name)))
-                               break;
+                       serv_printf("MSIV getscript|%s", ChrPtr(Ruleset->Name));
                        StrBuf_ServGetln(Line);
                        if (GetServerStatus(Line, NULL) == 1) 
                        {
                                Ruleset->Content = NewStrBuf();
+                               Done = 0;
                                while(!Done && (rc = StrBuf_ServGetln(Line), rc >= 0) )
                                        if ( (StrLength(Line)==3) && 
                                             !strcmp(ChrPtr(Line), "000")) 
@@ -1674,13 +953,7 @@ InitModule_SIEVE
 
        /* fetch our room into WCC->ThisRoom, to evaluate while iterating over rooms with COND:THIS:THAT:ROOM */
        RegisterNamespace("SIEVE:SCRIPT:LOOKUP_FILEINTO", 0, 1, tmplput_SieveRule_lookup_FileIntoRoom, NULL, CTX_SIEVESCRIPT);
-
-#if FOO
-       WebcitAddUrlHandler(HKEY("display_sieve"), "", 0, display_sieve, 0);
-       WebcitAddUrlHandler(HKEY("display_add_remove_scripts"), "", 0, _display_add_remove_scripts, 0);
-#endif
        WebcitAddUrlHandler(HKEY("save_sieve"), "", 0, save_sieve, 0);
-
        WebcitAddUrlHandler(HKEY("create_script"), "", 0, create_script, 0);
        WebcitAddUrlHandler(HKEY("delete_script"), "", 0, delete_script, 0);
 }
index 51c1f39714ac5b3d9bd58d6c42bd9ec546ee8290..b0c4bad48f05f0d4e438deb68b68d73237888a36 100644 (file)
@@ -1,3 +1,5 @@
+<?=("head")><?=("important_msg")><?%("COND:LOGGEDIN", 1, 1, 1, "", ="paging")>
+
 <div id="banner">
        <img src="static/webcit_icons/essen/32x32/config.png">
        <h1><?_("Add or delete scripts")></h1>
@@ -18,7 +20,7 @@
                <?=("box_end")>
                <?=("box_begin_1")><?_("Edit scripts")><?=("box_begin_2")>
                        <br><div align="center">
-                       <a href="display_sieve"><?_("Return to the script editing screen")></a><br><br>
+                       <a href="do_template?template=sieve_list"><?_("Return to the script editing screen")></a><br><br>
                <?=("box_end")>
        </td><td>
                <?=("box_begin_1")><?_("Delete scripts")><?=("box_begin_2")>
                        <select name="script_name" size=10 style="width:100%%">
                                <?ITERATE("SIEVE:SCRIPTS", ="sieve_script_select")>
                        </select>
+                       <input type="submit" name="delete_button" value='<?_("Delete")>'></form>
+                       </form>
                <?=("box_end")>
+       <td>
+       </tr>
+       </table>
+
+there is all sorts of stuff missing here, wtf ?
+
+</div> <!-- end of 'content' div -->
+<?=("trailing")>
+
index f393d3d7ce16f0239046b0da5a7bc14ea6e40f86..86c0480b3bc9a0d2f537ca97f6d1f4626c9c25a2 100644 (file)
@@ -27,7 +27,6 @@
        }
 </script>
 
-
 <div id="banner">
        <img src="static/webcit_icons/essen/32x32/config.png">
        <h1><?_("View/edit server-side mail filters")></h1>
@@ -39,7 +38,6 @@
     <tr><td valign="top"><form id="sieveform" method="post" action="save_sieve">
        <input type="hidden" name="nonce" value="<?NONCE>">
 
-<?--("The no filtering div")>
        <?_("When new mail arrives: ")>
        <select name="bigaction" size=1 onChange="ToggleSievePanels();">
          <option <?%("COND:BSTR", 1, "__SIEVE:RULESSCRIPT", 1, "", "selected")> value="0"><?_("Leave it in my inbox without filtering")></option>
@@ -47,6 +45,7 @@
          <option <?%("COND:BSTR", 1, "__SIEVE:EXTERNAL_SCRIPT", 1, "selected", "")> value="2"><?_("Filter it through a manually edited script (advanced users only)")></option>
        </select>
 
+<?--("The no filtering div")>
        <div id="sievediv0" style="display:none">
                <div align="center">
                        <br><br>
                <script type="text/javascript">UpdateRules();</script>
        </div>
 
-</div> <!-- one closing div too many? --the_mgt -->
-
-<?--("The I am smart and can write my own Sieve scripts div ")>
+<?--("The I am smart and can write my own Sieve scripts div")>
        <div id="sievediv2" style="display:none">
-               <??("COND:BSTR", 1, "__SIEVE:RULESSCRIPT", 1)>
+               <??("COND:BSTR", 1, "__SIEVE:EXTERNALSCRIPT", 1)>
                        <?_("The currently active script is: ")>
                        <select name="active_script" size=1 onChange="ToggleScriptPanels();">
                                <?ITERATE("SIEVE:SCRIPTS", ="sieve_list_select_one")>
                        </select>
                        &nbsp;&nbsp;&nbsp;
-                       <a href="display_add_remove_scripts"><?_("Add or delete scripts")></a>
+                       <a href="do_template?template=sieve_add"><?_("Add or delete scripts")></a>
                        <br>
                        <?ITERATE("SIEVE:SCRIPTS", ="sieve_display_one_script")>
                <??("X", 1)>
                <script type="text/javascript">ToggleScriptPanels();</script>
-     </div>
+       </div>
 
 <?--("The rest of this is common for all panels... ")>
        <div align="center"><br>