Changeset 22984 for main/trunk/greenstone2/runtime-src/src/recpt/sqlqueryaction.cpp

Timestamp:

2010-09-28T18:35:06+13:00 (14 years ago)

Author:

ak19

Message:

1. Undoing commit of 22934 where decode_commas was called on stem and fold comma separated list: previously separated due to url-encoding of commas. Now that the problem has been fixed at the source, the decode_commas hack is no longer necessary. 2. Commas in stem and fold are no longer url-encoded because the multiple_value field of the continuously-reused struct arg_ainfo is always set back to the default false after ever being set to true. So it no longer subtly stays at true to affect Greenstone functioning in unforeseen ways (such as suddenly and unnecessarily URL-encoding commas where this is not wanted).

File:

• main/trunk/greenstone2/runtime-src/src/recpt/sqlqueryaction.cpp (modified) (6 diffs)

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

@@ -47 +47 @@
   arg_ainfo.longname = "action";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "sqlq";
@@ -56 +57 @@
   arg_ainfo.longname = "sqlquery type options";
   arg_ainfo.multiplechar = true; // can be empty or single char
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = g_EmptyText;
@@ -65 +67 @@
   arg_ainfo.longname = "sql form query num fields";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "4";
@@ -75 +78 @@
   arg_ainfo.longname = "sql form query fields";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = g_EmptyText;
@@ -85 +89 @@
   arg_ainfo.longname = "sql form query values";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = g_EmptyText;
@@ -95 +100 @@
   arg_ainfo.longname = "sql form query combines";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = g_EmptyText;