--- loncom/interface/createaccount.pm	2020/09/08 18:35:31	1.72.4.4
+++ loncom/interface/createaccount.pm	2017/08/03 16:22:08	1.74
@@ -4,7 +4,7 @@
 # kerberos, or SSO) or an e-mail address. Requests to use an e-mail address as
 # username may be processed automatically, or may be queued for approval.
 #
-# $Id: createaccount.pm,v 1.72.4.4 2020/09/08 18:35:31 raeburn Exp $
+# $Id: createaccount.pm,v 1.74 2017/08/03 16:22:08 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -37,7 +37,6 @@ use Apache::lonacc;
 use Apache::lonnet;
 use Apache::loncommon;
 use Apache::lonhtmlcommon;
-use Apache::lonuserutils;
 use Apache::lonlocal;
 use Apache::lonauth;
 use Apache::resetpw;
@@ -329,7 +328,7 @@ sub handler {
         &print_header($r,$start_page,$courseid,$pagetitle);
         my $now=time;
         if ((grep(/^login$/,@{$cancreate})) && (!grep(/^email$/,@{$cancreate}))) {
-            if (open(my $jsh,"<","$include/londes.js")) {
+            if (open(my $jsh,"<$include/londes.js")) {
                 while(my $line = <$jsh>) {
                     $r->print($line);
                 }
@@ -521,12 +520,10 @@ ENDSCRIPT
 }
 
 sub javascript_checkpass {
-    my ($now,$context,$domain) = @_;
+    my ($now,$context) = @_;
     my $nopass = &mt('You must enter a password.');
     my $mismatchpass = &mt('The passwords you entered did not match.')."\n".
                        &mt('Please try again.'); 
-    my ($numrules,$intargjs) =
-        &Apache::lonuserutils::passwd_validation_js('upass',$domain);
     &js_escape(\$nopass);
     &js_escape(\$mismatchpass);
     my $js = <<"ENDSCRIPT";
@@ -543,10 +540,6 @@ sub javascript_checkpass {
                 return false;
             }
             if (upass == upasscheck) {
-                var numrules = $numrules;
-                if (numrules > 0) {
-$intargjs
-                }
                 client.elements.upasscheck$now.value='';
                 if (validate_email(client)) {
                     send(one,two,'$context');
@@ -570,7 +563,8 @@ sub javascript_validmail {
     my %js_lt = &Apache::lonlocal::texthash (
                email => 'The e-mail address you entered',
                notv  => 'is not a valid e-mail address',
-               avae  => 'A valid e-mail address is not formed when the value you entered is combined with the required domain',
+               avae  => 'A valid e-mail address is not formed when the value you entered',
+               isco  => 'is combined with the required domain',
     );
     my $output =  "\n".'<script type="text/javascript">'."\n".
                   '// <![CDATA['."\n".
@@ -582,7 +576,7 @@ function validate_email(client) {
     var condition = '$condition';
     if (validmail(field,condition) == false) {
         if ((condition != undefined) && (condition != '')) {
-            alert("$js_lt{'avae'}: "+condition);
+            alert("$js_lt{'avae'}: "+field.value+" $js_lt{'isco'}: "+condition);
         } else {
             alert("$js_lt{'email'}: "+field.value+" $js_lt{'notv'}.");
         }
@@ -640,7 +634,7 @@ sub print_username_form {
                     push(@posstypes,'default');
                 }
                 $output .= '<p>'.&mt('Choose your affiliation at [_1]',$domdesc).'</p>'."\n".
-                           '<form name="reportstatus" id="LC_reportstatus" action="/adm/createaccount" method="post" '.
+                           '<form name="reportstatus" id="LC_reportstatus" action="" method="post" '.
                            'onsubmit="return checkVerification();"><p>';
                 foreach my $type (@posstypes) {
                     my $name;
@@ -654,11 +648,7 @@ sub print_username_form {
                             }
                         }
                     }
-                    my $checked;
-                    if ($env{'form.type'} eq $type) {
-                        $checked = ' checked="checked"';
-                    }
-                    $output .= '<label><input type="radio" name="type" value="'.$type.'"'.$checked.' />'.
+                    $output .= '<label><input type="radio" name="type" value="'.$type.'" />'.
                                $name.'</label>'.('&nbsp;'x2);
                 }
                 if ($env{'form.courseid'} =~ /^$match_domain\_$match_courseid$/) {
@@ -697,18 +687,6 @@ sub print_username_form {
                     }
                     if ($usertype eq '') {
                         $usertype = 'default';
-                    } elsif (ref($usertypes) eq 'HASH') {
-                        my $usertitle;
-                        if ($usertype eq 'default') {
-                            $usertitle = $othertitle;
-                        } elsif (exists($usertypes->{$usertype})) {
-                            $usertitle = $usertypes->{$usertype};
-                        }
-                        if ($usertitle ne '') {
-                            $output .= &mt('Self-reported affiliation: [_1]',
-                                           '<span style="font-style: italic;">'.$usertitle.'</span>').
-                                       '<br />';
-                        }
                     }
                     $output .= &print_dataentry_form($r,$domain,$lonhost,$include,$now,$captchaform,
                                                      $courseid,$emailusername,$captcha,$usertype,
@@ -841,24 +819,8 @@ sub process_email_request {
             } else {
                 my ($captcha_chk,$captcha_error) = &Apache::loncommon::captcha_response('usercreation',$server);
                 if ($captcha_chk != 1) {
-                    $output = '<span class="LC_warning">'.
-                              &mt('Validation of the code you entered failed.').'</span>'.
-                              '<br />'.$captcha_error."\n".'<br /><p>'.
-                               &mt('[_1]Return[_2] to the previous page to try again.',
-                                   '<a href="javascript:document.retryemail.submit();">','</a>')."\n".
-                              '<form name="retryemail" action="/adm/createaccount" method="post" />'.
-                              '<input type="hidden" name="domain" value="'.$domain.'" />'."\n";
-                    if ($env{'form.courseid'} =~ /^$match_domain\_$match_courseid$/) {
-                        $output .= '<input type="hidden" name="courseid" value="'.$env{'form.courseid'}.'" />'."\n";
-                    }
-                    if ($env{'form.type'}) {
-                        my $usertype = &get_usertype($domain);
-                        if ($usertype ne '') {
-                            $output .= '<input type="hidden" name="type" value="'.$usertype.'" />'."\n".
-                                       '<input type="hidden" name="reportedtype" value="'.&mt('Submit').'" />'."\n";
-                        }
-                    }
-                    $output .= '</form></p>';
+                    $output = &invalid_state('captcha',$domdesc,$contact_name,
+                                             $contact_email,$captcha_error);
                     return $output;
                 }
                 my (%rulematch,%inst_results,%curr_rules,%got_rules,%alerts);
@@ -916,72 +878,16 @@ sub send_token {
     $env{'form.logtoken'} =~ s/(`)//g;
     if ($env{'form.logtoken'}) {
         my $logtoken = $env{'form.logtoken'};
-        my $earlyout;
         my $tmpinfo=Apache::lonnet::reply('tmpget:'.$logtoken,$server);
         if (($tmpinfo=~/^error/) || ($tmpinfo eq 'con_lost')) {
             $msg = &mt('Information needed to process your request is missing, inaccessible or expired.')
-                  .'<br /><p>'.&mt('[_1]Return[_2] to the previous page to try again.',
-                                   '<a href="javascript:document.retryemail.submit();">','</a>');
-            $earlyout = 1;
+                  .'<br />'.&mt('Return to the previous page to try again.');
         } else {
             my $reply = &Apache::lonnet::reply('tmpdel:'.$logtoken,$server);
             unless ($reply eq 'ok') {
                 $msg .= &mt('Request could not be processed.');
             }
         }
-# Check if the password entered by the user satisfies domain's requirements
-        my %passwdconf = &Apache::lonnet::get_passwdconf($domain);
-        my ($min,$max,@chars);
-        $min = $Apache::lonnet::passwdmin;
-        if (ref($passwdconf{'chars'}) eq 'ARRAY') {
-            if ($passwdconf{'min'} =~ /^\d+$/) {
-                if ($passwdconf{'min'} > $min) {
-                    $min = $passwdconf{'min'};
-                }
-            }
-            if ($passwdconf{'max'} =~ /^\d+$/) {
-                $max = $passwdconf{'max'};
-            }
-            @chars = @{$passwdconf{'chars'}};
-        }
-        my $encpass = $env{'form.upass'};
-        if ($encpass eq '') {
-            $msg = &mt('Password retrieved was blank.').
-                   '<br /><p>'.&mt('[_1]Return[_2] to the previous page to try again.',
-                                   '<a href="javascript:document.retryemail.submit();">','</a>');
-            $earlyout = 1;
-        } else {
-# Split the logtoken to retrieve the DES key and decrypt the encypted password
-            my ($key,$caller)=split(/&/,$tmpinfo);
-            if ($caller eq 'createaccount') {
-                my $plainpass = &Apache::loncommon::des_decrypt($key,$encpass);
-                if (($min > 0) || ($max ne '') || (@chars > 0)) {
-                    my $warning = &Apache::loncommon::check_passwd_rules($domain,$plainpass);
-                    if ($warning) {
-                        $msg = $warning.
-                               '<p>'.&mt('[_1]Return[_2] to the previous page to try again.',
-                                         '<a href="javascript:document.retryemail.submit();">','</a>');
-                        $earlyout = 1;
-                    }
-                }
-            }
-        }
-        if ($earlyout) {
-            $msg .= '<form name="retryemail" action="/adm/createaccount" method="post" />'.
-                    '<input type="hidden" name="domain" value="'.$domain.'" />'."\n";
-            if ($env{'form.courseid'} =~ /^$match_domain\_$match_courseid$/) {
-                $msg .= '<input type="hidden" name="courseid" value="'.$env{'form.courseid'}.'" />'."\n";
-            }
-            if ($env{'form.type'}) {
-                my $usertype = &get_usertype($domain);
-                if ($usertype ne '') {
-                    $msg .= '<input type="hidden" name="type" value="'.$usertype.'" />'.
-                            '<input type="hidden" name="reportedtype" value="'.&mt('Submit').'" />'."\n";
-                }
-            }
-            $msg .= '</form></p>';
-            return $msg;
-        }
         my %info = ('ip'         => $ENV{'REMOTE_ADDR'},
                     'time'       => $now,
                     'domain'     => $domain,
@@ -1167,14 +1073,14 @@ sub print_dataentry_form {
     my ($r,$domain,$lonhost,$include,$now,$captchaform,$courseid,$emailusername,$captcha,
         $usertype,$recaptchaversion,$usernameset,$condition,$excluded) = @_;
     my ($error,$output);
-    if (open(my $jsh,"<","$include/londes.js")) {
+    if (open(my $jsh,"<$include/londes.js")) {
         while(my $line = <$jsh>) {
             $r->print($line);
         }
         close($jsh);
         $output = &javascript_setforms($now,$emailusername,$captcha,$usertype,$recaptchaversion,
                                        $usernameset,$condition,$excluded).
-                  "\n".&javascript_checkpass($now,'email',$domain);
+                  "\n".&javascript_checkpass($now,'email');
         my ($lkey,$ukey) = &Apache::loncommon::des_keys();
         my ($lextkey,$uextkey) = &getkeys($lkey,$ukey);
         my $logtoken=Apache::lonnet::reply('tmpput:'.$ukey.$lkey.'&createaccount:createaccount',
@@ -1782,6 +1688,8 @@ sub invalid_state {
         $msg .= &mt('Username rules at this institution do not allow the e-mail address you provided to be used as a username.');
     } elsif ($error eq 'userformat') {
         $msg .= &mt('The e-mail address you provided may not be used as a username at this LON-CAPA institution.');
+    } elsif ($error eq 'captcha') {
+        $msg .= &mt('Validation of the code you entered failed.');
     } elsif ($error eq 'noemails') {
         $msg .= &mt('Creation of a new user account using an e-mail address as username is not permitted at this LON-CAPA institution.');
     } elsif ($error eq 'emailfail') {
@@ -1962,14 +1870,14 @@ sub sso_logout_frag {
     if (defined($r->dir_config('lonSSOUserLogoutMessageFile_'.$domain))) {
         my $msgfile = $r->dir_config('lonSSOUserLogoutMessageFile_'.$domain);
         if (-e $msgfile) {
-            open(my $fh,"<",$msgfile);
+            open(my $fh,"<$msgfile");
             $endsessionmsg = join('',<$fh>);
             close($fh);
         }
     } elsif (defined($r->dir_config('lonSSOUserLogoutMessageFile'))) {
         my $msgfile = $r->dir_config('lonSSOUserLogoutMessageFile');
         if (-e $msgfile) {     
-            open(my $fh,"<",$msgfile);
+            open(my $fh,"<$msgfile");
             $endsessionmsg = join('',<$fh>);
             close($fh);
         }
@@ -2088,7 +1996,6 @@ function checkVerification() {
             if (curr == types[j]) {
                 if (!cancreate) {
                     alert('Creation of an account via verification by e-mail unavailable for user type: "'+names[j]+'"');
-                    setElements();
                 }
                 break;
             }