From: Mike G. v. a. <we...@ma...> - 2005-06-27 00:09:20
|
Log Message: ----------- Removed warning message about "users" . We closed the bug defining multiple users awhile ago so we don't need this anymore. It's causing trouble with UserList.pm which now has fields visible_users and prev_visible_users. Modified Files: -------------- webwork-modperl/lib/WeBWorK/ContentGenerator: Login.pm Revision Data ------------- Index: Login.pm =================================================================== RCS file: /webwork/cvs/system/webwork-modperl/lib/WeBWorK/ContentGenerator/Login.pm,v retrieving revision 1.26 retrieving revision 1.27 diff -Llib/WeBWorK/ContentGenerator/Login.pm -Llib/WeBWorK/ContentGenerator/Login.pm -u -r1.26 -r1.27 --- lib/WeBWorK/ContentGenerator/Login.pm +++ lib/WeBWorK/ContentGenerator/Login.pm @@ -139,7 +139,11 @@ my @fields_to_print = grep { not m/^(user|passwd|key|force_passwd_authen)$/ } $r->param; #FIXME: This next line can be removed in time. MEG 1/27/2005 - warn "Error in filtering fields : |", join("|",@fields_to_print),"|" if grep {m/user/} @fields_to_print; + # warn "Error in filtering fields : |", join("|",@fields_to_print),"|" if grep {m/user/} @fields_to_print; + # the above test was an attempt to discover why "user" was being multiply defined. + # We caught that error, but this warning causes trouble with UserList.pm which now has + # fields visible_users and prev_visible_users + # Important note. If hidden_fields is passed an empty array it prints ALL parameters as hidden fields. # That is not what we want in this case, so we don't print at all if @fields_to_print is empty. |