Changeset 22984 for main/trunk/greenstone2/runtime-src/src/recpt/browseaction.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/browseaction.cpp (modified) (6 diffs)

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

@@ -40 +40 @@
   arg_ainfo.longname = "action";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "br";
@@ -49 +50 @@
   arg_ainfo.longname = "browse criterion 1";
   arg_ainfo.multiplechar = false;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "0";
@@ -59 +61 @@
   arg_ainfo.longname = "browse criterion 2";
   arg_ainfo.multiplechar = false;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "0";
@@ -70 +73 @@
   arg_ainfo.longname = "browse docs per page";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::good;
   arg_ainfo.argdefault = g_EmptyText;
@@ -80 +84 @@
   arg_ainfo.longname = "browse filter text";
   arg_ainfo.multiplechar = true;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::none;
   arg_ainfo.argdefault = g_EmptyText;
@@ -91 +96 @@
   arg_ainfo.longname = "browse all or any";
   arg_ainfo.multiplechar = false;
+  arg_ainfo.multiplevalue = false;
   arg_ainfo.defaultstatus = cgiarginfo::weak;
   arg_ainfo.argdefault = "0";
   arg_ainfo.savedarginfo = cgiarginfo::must;
-  argsinfo.addarginfo (NULL, arg_ainfo);
-  
+  argsinfo.addarginfo (NULL, arg_ainfo); 
   
 }