Changeset 30373 for main/trunk/greenstone2/runtime-src/src/recpt/cgiutils.cpp

Timestamp:

2016-02-10T14:53:51+13:00 (8 years ago)

Author:

kjdon

Message:

first stab at fixing diego's bug where you can't add accented values from a select in the depositor. the problem seems to be that argsstr form a get is not unicode, but from a post it is??

File:

• main/trunk/greenstone2/runtime-src/src/recpt/cgiutils.cpp (modified) (5 diffs)

main/trunk/greenstone2/runtime-src/src/recpt/cgiutils.cpp

@@ -412 +412 @@
   text_t::const_iterator here = argstr.begin();
   text_t::const_iterator end = argstr.end();
+  // get seems to be not unicode, while post is, so don't want to just assume encoding is 1 (not unicode)
+  unsigned short args_encoding = argstr.getencoding();
+  cerr << "args enc = "<< args_encoding<<endl;
   text_t key, value;
  
@@ -425 +428 @@
     safe_cgi_arg(key, value); // mitigate obvious cross-site scripting hacks in URL cgi-params
 
-    value.setencoding(1); // other encoding
+    value.setencoding(args_encoding); //1 // other encoding
     // store this key=value pair
     if (!key.empty()) {
@@ -441 +444 @@
     // If the arg is not multi valued, then you need to decode the commas.
     if (args.getarg(key)==NULL) {
-      args.setarg (key, encode_commas(value), cgiarg_t::cgi_arg);
+      // encode_commas returns a text_t without encoding bit set
+      text_t newvalue = encode_commas(value);
+      newvalue.setencoding(args_encoding);
+      args.setarg (key, newvalue, cgiarg_t::cgi_arg);
     }
     else {
@@ -447 +453 @@
 
       newvalue += "," + encode_commas(value);
-      newvalue.setencoding(1); // other encoding
+      newvalue.setencoding(args_encoding); // other encoding
       args.setarg (key, newvalue, cgiarg_t::cgi_arg);
     }
@@ -457 +463 @@
       if (args.lookupcgiarg(key).source == cgiarg_t::cgi_arg) newvalue += ",";
       newvalue += encode_commas(value);
-      newvalue.setencoding(1); // other encoding
+      newvalue.setencoding(args_encoding); // other encoding
       args.setarg (key, newvalue, cgiarg_t::cgi_arg);