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/authenaction.cpp

    r20798 r22984  
    4747  arg_ainfo.longname = "action";
    4848  arg_ainfo.multiplechar = true;
     49  arg_ainfo.multiplevalue = false;
    4950  arg_ainfo.defaultstatus = cgiarginfo::weak;
    5051  arg_ainfo.argdefault = "a";
    … …  
    5657  arg_ainfo.longname = "user account status";
    5758  arg_ainfo.multiplechar = true;
     59  arg_ainfo.multiplevalue = false;
    5860  arg_ainfo.defaultstatus = cgiarginfo::weak;
    5961  arg_ainfo.argdefault = "invalid";
    … …  
    6567  arg_ainfo.longname = "user groups"; // comma seperated list
    6668  arg_ainfo.multiplechar = true;
     69  arg_ainfo.multiplevalue = false;
    6770  arg_ainfo.defaultstatus = cgiarginfo::weak;
    6871  arg_ainfo.argdefault = g_EmptyText;
    … …  
    7477  arg_ainfo.longname = "user name";
    7578  arg_ainfo.multiplechar = true;
     79  arg_ainfo.multiplevalue = false;
    7680  arg_ainfo.defaultstatus = cgiarginfo::weak;
    7781  arg_ainfo.argdefault = g_EmptyText;
    … …  
    8387  arg_ainfo.longname = "password";
    8488  arg_ainfo.multiplechar = true;
     89  arg_ainfo.multiplevalue = false;
    8590  arg_ainfo.defaultstatus = cgiarginfo::weak;
    8691  arg_ainfo.argdefault = g_EmptyText;
    … …  
    9398  arg_ainfo.longname = "user time key";
    9499  arg_ainfo.multiplechar = true;
     100  arg_ainfo.multiplevalue = false;
    95101  arg_ainfo.defaultstatus = cgiarginfo::weak;
    96102  arg_ainfo.argdefault = g_EmptyText;
    … …  
    102108  arg_ainfo.longname = "whether a user has been authenticated";
    103109  arg_ainfo.multiplechar = true;
     110  arg_ainfo.multiplevalue = false;
    104111  arg_ainfo.defaultstatus = cgiarginfo::weak;
    105112  arg_ainfo.argdefault = g_EmptyText;
    … …  
    111118  arg_ainfo.longname = "the compressed args of the refer page";
    112119  arg_ainfo.multiplechar = true;
     120  arg_ainfo.multiplevalue = false;
    113121  arg_ainfo.defaultstatus = cgiarginfo::weak;
    114122  arg_ainfo.argdefault = g_EmptyText;
    … …  
    120128  arg_ainfo.longname = "whether user authentication is needed";
    121129  arg_ainfo.multiplechar = true;
     130  arg_ainfo.multiplevalue = false;
    122131  arg_ainfo.defaultstatus = cgiarginfo::weak;
    123132  arg_ainfo.argdefault = g_EmptyText;
Note: See TracChangeset for help on using the changeset viewer.