Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed

  • main/trunk/greenstone2/runtime-src/src/recpt/usersaction.cpp

    r15589 r22984  
    4141  arg_ainfo.longname = "action";
    4242  arg_ainfo.multiplechar = true;
     43  arg_ainfo.multiplevalue = false;
    4344  arg_ainfo.defaultstatus = cgiarginfo::weak;
    4445  arg_ainfo.argdefault = "um"; // user management
    … …  
    5051  arg_ainfo.longname = "user management action";
    5152  arg_ainfo.multiplechar = true;
     53  arg_ainfo.multiplevalue = false;
    5254  arg_ainfo.defaultstatus = cgiarginfo::weak;
    5355  arg_ainfo.argdefault = "listusers";
    … …  
    5961  arg_ainfo.longname = "user management user name";
    6062  arg_ainfo.multiplechar = true;
     63  arg_ainfo.multiplevalue = false;
    6164  arg_ainfo.defaultstatus = cgiarginfo::weak;
    6265  arg_ainfo.argdefault = g_EmptyText;
    … …  
    6871  arg_ainfo.longname = "user management password";
    6972  arg_ainfo.multiplechar = true;
     73  arg_ainfo.multiplevalue = false;
    7074  arg_ainfo.defaultstatus = cgiarginfo::weak;
    7175  arg_ainfo.argdefault = g_EmptyText;
    … …  
    7781  arg_ainfo.longname = "user management new password 1";
    7882  arg_ainfo.multiplechar = true;
     83  arg_ainfo.multiplevalue = false;
    7984  arg_ainfo.defaultstatus = cgiarginfo::weak;
    8085  arg_ainfo.argdefault = g_EmptyText;
    … …  
    8691  arg_ainfo.longname = "user management new password 2";
    8792  arg_ainfo.multiplechar = true;
     93  arg_ainfo.multiplevalue = false;
    8894  arg_ainfo.defaultstatus = cgiarginfo::weak;
    8995  arg_ainfo.argdefault = g_EmptyText;
    … …  
    95101  arg_ainfo.longname = "user management account status";
    96102  arg_ainfo.multiplechar = true;
     103  arg_ainfo.multiplevalue = false;
    97104  arg_ainfo.defaultstatus = cgiarginfo::weak;
    98105  arg_ainfo.argdefault = g_EmptyText;
    … …  
    104111  arg_ainfo.longname = "user management groups"; // comma seperated list
    105112  arg_ainfo.multiplechar = true;
     113  arg_ainfo.multiplevalue = false;
    106114  arg_ainfo.defaultstatus = cgiarginfo::weak;
    107115  arg_ainfo.argdefault = g_EmptyText;
    … …  
    113121  arg_ainfo.longname = "user management comment";
    114122  arg_ainfo.multiplechar = true;
     123  arg_ainfo.multiplevalue = false;
    115124  arg_ainfo.defaultstatus = cgiarginfo::weak;
    116125  arg_ainfo.argdefault = g_EmptyText;
    … …  
    122131  arg_ainfo.longname = "change password submit button";
    123132  arg_ainfo.multiplechar = true;
     133  arg_ainfo.multiplevalue = false;
    124134  arg_ainfo.defaultstatus = cgiarginfo::weak;
    125135  arg_ainfo.argdefault = g_EmptyText;
    … …  
    131141  arg_ainfo.longname = "edit user submit button";
    132142  arg_ainfo.multiplechar = true;
     143  arg_ainfo.multiplevalue = false;
    133144  arg_ainfo.defaultstatus = cgiarginfo::weak;
    134145  arg_ainfo.argdefault = g_EmptyText;
    … …  
    140151  arg_ainfo.longname = "confirm an action";
    141152  arg_ainfo.multiplechar = true;
     153  arg_ainfo.multiplevalue = false;
    142154  arg_ainfo.defaultstatus = cgiarginfo::weak;
    143155  arg_ainfo.argdefault = g_EmptyText;
Note: See TracChangeset for help on using the changeset viewer.