Skip to content
Snippets Groups Projects
Commit 78e742eb authored by Elmar Ludwig's avatar Elmar Ludwig Committed by David Siegfried
Browse files

default to `false` for `consider_contingent`, fixes #2428

Closes #2428

Merge request studip/studip!1624
parent 15831cc8
Branches
No related tags found
No related merge requests found
...@@ -295,7 +295,7 @@ class Course_MembersController extends AuthenticatedController ...@@ -295,7 +295,7 @@ class Course_MembersController extends AuthenticatedController
$countAdded = 0; $countAdded = 0;
$msg = []; $msg = [];
foreach ($mp->getAddedUsers() as $a) { foreach ($mp->getAddedUsers() as $a) {
if ($this->addMember($a, true, Request::bool('consider_contingent'), $msg)) { if ($this->addMember($a, true, Request::bool('consider_contingent', false), $msg)) {
$countAdded++; $countAdded++;
} }
} }
...@@ -1015,7 +1015,7 @@ class Course_MembersController extends AuthenticatedController ...@@ -1015,7 +1015,7 @@ class Course_MembersController extends AuthenticatedController
$msgs = $this->insertAdmissionMember( $msgs = $this->insertAdmissionMember(
$users, $users,
$target_status, $target_status,
Request::bool('consider_contingent'), Request::bool('consider_contingent', false),
$status === 'accepted' $status === 'accepted'
); );
if ($msgs) { if ($msgs) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment