Skip to content

Commit 5c46f16

Browse files
committed
Fix an issue with the addcourse script.
The `addcourse` script does not use a database object to create a user record. Instead it directly calls `WeBWorK::DB::Record::User->new(%record)`. So if the `%record` hash does not have the `accommodation_time_factor` key set, then the script errors out when it attempts to create a user. So make sure that is set.
1 parent 4e047f8 commit 5c46f16

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

bin/addcourse

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ use Getopt::Long;
4949

5050
BEGIN {
5151
use Mojo::File qw(curfile);
52-
use Env qw(WEBWORK_ROOT);
52+
use Env qw(WEBWORK_ROOT);
5353

5454
$WEBWORK_ROOT = curfile->dirname->dirname;
5555
}
@@ -59,9 +59,9 @@ use lib "$ENV{WEBWORK_ROOT}/lib";
5959

6060
use WeBWorK::CourseEnvironment;
6161
use WeBWorK::File::Classlist;
62-
use WeBWorK::Utils qw(runtime_use cryptPassword);
62+
use WeBWorK::Utils qw(runtime_use cryptPassword);
6363
use WeBWorK::Utils::CourseManagement qw(addCourse);
64-
use WeBWorK::File::Classlist qw(parse_classlist);
64+
use WeBWorK::File::Classlist qw(parse_classlist);
6565
use WeBWorK::DB::Record::User;
6666
use WeBWorK::DB::Record::Password;
6767
use WeBWorK::DB::Record::PermissionLevel;
@@ -124,6 +124,8 @@ if ($users) {
124124
$record{permission} = $ce->{userRoles}{professor};
125125
}
126126

127+
$record{accommodation_time_factor} = 1;
128+
127129
delete $professors{$user_id};
128130

129131
push @users,

0 commit comments

Comments
 (0)