--- loncom/interface/loncreateuser.pm 2007/12/21 21:02:10 1.222 +++ loncom/interface/loncreateuser.pm 2007/12/23 15:50:23 1.223 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Create a user # -# $Id: loncreateuser.pm,v 1.222 2007/12/21 21:02:10 raeburn Exp $ +# $Id: loncreateuser.pm,v 1.223 2007/12/23 15:50:23 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -2109,7 +2109,11 @@ sub update_roles { if ($url =~ m-^/($match_domain)/($match_courseid)/?(\w*)$-) { my $result = &Apache::loncommon::commit_studentrole(\$logmsg,$udom,$uname,$url,$role,$now,0,$1,$2,$3); if (($result =~ /^error/) || ($result eq 'not_in_class') || ($result eq 'unknown_course') || ($result eq 'refused')) { - $output = "Error: $result\n"; + if ($result eq 'refused' && $logmsg) { + $output = $logmsg; + } else { + $output = "Error: $result\n"; + } } else { $output = &mt('Assigning').' '.$role.' in '.$url. &mt('starting').' '.localtime($now). @@ -2836,10 +2840,10 @@ sub handler { 'User_Management_List')); if ($permission->{'cusr'}) { &Apache::lonuserutils::update_user_list($r,$context,$setting,$choice); - $r->print('

'.&mt('Display User Lists').''); $r->print(&Apache::loncommon::end_page()); } else { $r->print(&mt('You are not authorized to make bulk changes to user roles')); + $r->print('

'.&mt('Display User Lists').''); $r->print(&Apache::loncommon::end_page()); } } else {