--- loncom/interface/selfenroll.pm 2012/05/01 16:09:26 1.25.6.1
+++ loncom/interface/selfenroll.pm 2012/05/16 21:19:39 1.27
@@ -1,7 +1,7 @@
# The LearningOnline Network
# Allow users to self-enroll in a course
#
-# $Id: selfenroll.pm,v 1.25.6.1 2012/05/01 16:09:26 raeburn Exp $
+# $Id: selfenroll.pm,v 1.27 2012/05/16 21:19:39 droeschl Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -52,11 +52,13 @@ sub handler {
}
&Apache::lonacc::get_posted_cgi($r);
&Apache::lonlocal::get_language_handle($r);
- &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['courseid','usertype']);
+ &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['courseid']);
my $js = &Apache::createaccount::catreturn_js();
- my ($coursechk,$courseid) = &validate_course_id($env{'form.courseid'});
- if ($coursechk ne 'ok') {
- &page_header($r,$courseid,$js);
+
+ my $courseid = Apache::lonnet::is_course($env{'form.courseid'});
+
+ unless ($courseid) {
+ &page_header($r,$env{'form.courseid'},$js);
$r->print('
'.&mt('Self-enrollment error').'
'.
''.
&mt('Invalid domain or course number').'');
@@ -342,8 +344,7 @@ sub enrollment_limit_check {
sub page_header {
my ($r,$courseid,$js,$desc) = @_;
my $start_page =
- &Apache::loncommon::start_page('Self-enroll in a LON-CAPA course',$js,
- {'no_inline_link' => 1,});
+ &Apache::loncommon::start_page('Self-enroll in a LON-CAPA course',$js);
$r->print($start_page);
&Apache::lonhtmlcommon::clear_breadcrumbs();
&Apache::createaccount::selfenroll_crumbs($r,$courseid,$desc);
@@ -359,17 +360,6 @@ sub page_footer {
return;
}
-sub validate_course_id {
- my ($courseid) = @_;
- my ($cdom,$cnum) = ($env{'form.courseid'} =~ /^($match_domain)_($match_courseid)$/);
- if ($cdom ne '' && $cnum ne '') {
- if (&Apache::lonnet::is_course($cdom,$cnum)) {
- return ('ok',$courseid);
- }
- }
- return;
-}
-
sub user_is_known {
my $known = 0;
if ($env{'user.name'} ne '' && $env{'user.name'} ne 'public'
@@ -442,9 +432,6 @@ sub process_self_enroll {
if ($selfenroll_section eq 'none') {
$usec = '';
}
- unless (($usec ne '') && ($env{'form.usertype'} eq 'new')) {
- $usec = 'sp12';
- }
if ($selfenroll_registered) {
my ($registered,$instsec,$message) = &check_registered($cdom,$cnum);
$usec = $instsec;