Nearly all <FORM> blocks now contain a hidden input

[citadel.git] / webcit / listsub.c
diff --git a/webcit/listsub.c b/webcit/listsub.c

index 66d03aac8faa70da74007dbfc0fb5eb6300369a2..2000c3c181a7bc18890e8a66bdef9eaab422ba5b 100644 (file)
--- a/webcit/listsub.c
+++ b/webcit/listsub.c
@@ -1,16 +1,18 @@
  /*
   * $Id$
- *
- * Web forms for handling mailing list subscribe/unsubscribe requests.
- *
+ */
+/**
+ * \defgroup ListSubForms Web forms for handling mailing list subscribe/unsubscribe requests.
+ * \ingroup WebcitDisplayItems
   */
  
+/*@{*/
  #include "webcit.h"
  
  
  
-/*
- * List subscription handling
+/**
+ * \brief List subscription handling
   */
  void do_listsub(void)
  {
@@ -26,11 +28,19 @@ void do_listsub(void)
         int self;
         char sroom[SIZ];
  
+       strcpy(WC->wc_fullname, "");
         strcpy(WC->wc_username, "");
         strcpy(WC->wc_password, "");
         strcpy(WC->wc_roomname, "");
  
-       wprintf("<HTML><HEAD><TITLE>");
+       output_headers(1, 0, 0, 1, 1, 0);
+       begin_burst();
+
+       wprintf("<HTML><HEAD>\n"
+               "<meta name=\"MSSmartTagsPreventParsing\" content=\"TRUE\" />\n"
+               "<link href=\"static/webcit.css\" rel=\"stylesheet\" type=\"text/css\">\n"
+               "<TITLE>\n"
+       );
         wprintf(_("List subscription"));
         wprintf("</TITLE></HEAD><BODY>\n");
  
@@ -41,12 +51,12 @@ void do_listsub(void)
         strcpy(subtype, bstr("subtype"));
  
         wprintf("<CENTER>"
-               "<TABLE WIDTH=100%% BORDER=0 BGCOLOR=\"#444455\"><TR><TD>"
+               "<TABLE class=\"listsub_banner\"><TR><TD>"
                 "<SPAN CLASS=\"titlebar\">");
         wprintf(_("List subscribe/unsubscribe"));
         wprintf("</SPAN></TD></TR></TABLE><br />\n");
  
-       /*
+       /**
          * Subscribe command
          */
         if (!strcasecmp(cmd, "subscribe")) {
@@ -88,7 +98,7 @@ void do_listsub(void)
                 }
         }
  
-       /*
+       /**
          * Unsubscribe command
          */
         else if (!strcasecmp(cmd, "unsubscribe")) {
@@ -126,7 +136,7 @@ void do_listsub(void)
                 }
         }
  
-       /* 
+       /**
          * Confirm command
          */
         else if (!strcasecmp(cmd, "confirm")) {
@@ -154,13 +164,13 @@ void do_listsub(void)
                 wprintf("%s</CENTER><br />\n", &buf[4]);
         }
  
-       /*
+       /**
          * Any other (invalid) command causes the form to be displayed
          */
         else {
-FORM:          wprintf("<FORM METHOD=\"POST\" action=\"listsub\">\n"
-                       "<TABLE BORDER=0>\n"
-               );
+FORM:          wprintf("<FORM METHOD=\"POST\" action=\"listsub\">\n");
+               wprintf("<input type=\"hidden\" name=\"nonce\" value=\"%ld\">\n", WC->nonce);
+               wprintf("<TABLE BORDER=0>\n");
  
                 wprintf("<TR><TD>Name of list</TD><TD>"
                         "<SELECT NAME=\"room\" SIZE=1>\n");
@@ -192,10 +202,10 @@ FORM:             wprintf("<FORM METHOD=\"POST\" action=\"listsub\">\n"
  
                 wprintf("</TABLE>"
                         "(If subscribing) preferred format: "
-                       "<INPUT TYPE=\"radio\" NAME=\"subtype\""
-                       "VALUE=\"list\">One message at a time&nbsp; "
-                       "<INPUT TYPE=\"radio\" NAME=\"subtype\""
-                       "VALUE=\"digest\" CHECKED>Digest format&nbsp; "
+                       "<INPUT TYPE=\"radio\" NAME=\"subtype\" "
+                       "VALUE=\"list\" CHECKED>One message at a time&nbsp; "
+                       "<INPUT TYPE=\"radio\" NAME=\"subtype\" "
+                       "VALUE=\"digest\">Digest format&nbsp; "
                         "<br />\n"
                         "<INPUT TYPE=\"submit\" NAME=\"cmd\""
                         " VALUE=\"subscribe\">\n"
@@ -214,11 +224,11 @@ FORM:             wprintf("<FORM METHOD=\"POST\" action=\"listsub\">\n"
  
         }
  
-       /*
-        * Since this isn't part of a normal Citadel session, we bail right
-        * out without maintaining any state.
-        */
-       /* wDumpContent(2); */
         wprintf("</BODY></HTML>\n");
+       wDumpContent(0);
         end_webcit_session();
  }
+
+
+
+/*@}*/