--- loncom/automation/batchcreatecourse.pm 2005/06/01 19:20:32 1.5 +++ loncom/automation/batchcreatecourse.pm 2005/07/11 20:02:02 1.6 @@ -1,5 +1,5 @@ # -# $Id: batchcreatecourse.pm,v 1.5 2005/06/01 19:20:32 raeburn Exp $ +# $Id: batchcreatecourse.pm,v 1.6 2005/07/11 20:02:02 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -436,12 +436,13 @@ sub build_course { 'linefeed' => $linefeed, 'role' => 'cc', }; - $outcome = &LONCAPA::Enrollment::create_newuser($ownerargs,$logmsg,$newusermsg,$enrollcount,$addresult,$longroles,$courseinfo); + $outcome = &LONCAPA::Enrollment::create_newuser($ownerargs,$logmsg,$newusermsg,$enrollcount,$addresult,$longroles,\%courseinfo); } else { $outcome = 'ok'; } - my $courseargs = { + if ($outcome eq 'ok') { + my $courseargs = { ccuname => $$details{$num}{'owner'}, ccdomain => $$details{$num}{'domain'}, cdescr => $$details{$num}{'title'}, @@ -479,7 +480,6 @@ sub build_course { firstres => $firstres }; - if ($outcome eq 'ok') { my %host_servers = &Apache::loncommon::get_library_servers($cdom); if (! exists($host_servers{$$details{$num}{'coursehome'}})) { $$logmsg .= &mt('Invalid home server for course').': '.$$details{$num}{'coursehome'}; @@ -529,7 +529,7 @@ sub build_course { 'linefeed' => $linefeed, 'role' => $$details{$num}{'users'}{$userkey}{'roles'}[0], }; - $outcome = &LONCAPA::Enrollment::create_newuser($userargs,$logmsg,$newusermsg,$enrollcount,$addresult,$longroles,$courseinfo); + $outcome = &LONCAPA::Enrollment::create_newuser($userargs,$logmsg,$newusermsg,$enrollcount,$addresult,$longroles,\%courseinfo); # now add other roles and other sections. if ($outcome eq 'ok') { if (($firstrole ne 'st') && (@{$$details{$num}{'users'}{$userkey}{$firstrole}{'usec'}} > 1)) {