Changeset 2417

Timestamp:

2001-05-15T12:20:00+12:00 (23 years ago)

Author:

sjboddie

Message:

fixed a bug in the way multi-value cgi arguments were being handled

File:

• trunk/gsdl/src/recpt/cgiutils.cpp (modified) (1 diff)

trunk/gsdl/src/recpt/cgiutils.cpp

@@ -112 +112 @@
       // multiple checkboxes using the same name) we'll 
       // create a comma separated list of all the values
-      // (if multiplevalue)
-      if (!args[key].empty()) {
-    const cgiarginfo *info = argsinfo.getarginfo (key);
-    if (info != NULL && info->multiplevalue)
-      args[key] += "," + value;
-    else args.setarg (key, value, cgiarg_t::cgi_arg);
-      } else 
+      // (we should probably use some kind of array here to 
+      // avoid problems when values already contain commas)
+      const cgiarginfo *info = argsinfo.getarginfo (key);
+      if (info != NULL && info->multiplevalue) {
+    text_t newvalue = args[key];
+    if (args.lookupcgiarg(key).source == cgiarg_t::cgi_arg) newvalue += ",";
+    newvalue += value;
+    args.setarg (key, newvalue, cgiarg_t::cgi_arg);
+
+      } else {
     args.setarg (key, value, cgiarg_t::cgi_arg);
+      }
     }
   }