diff --git a/main/badge/assign.php b/main/badge/assign.php index e9da0c85fb..8295108b71 100644 --- a/main/badge/assign.php +++ b/main/badge/assign.php @@ -74,7 +74,6 @@ if (!empty($subSkillList)) { $skillId = end($subSkillList); } - if (!$profile) { $skillRelSkill = new SkillRelSkill(); $parents = $skillRelSkill->getSkillParents($skillId); @@ -101,7 +100,6 @@ if ($profile) { 'profile' => $profileId ]); $profileLevels = []; - foreach ($levels as $level) { $profileLevels[$level->getPosition()][$level->getId()] = $level->getName(); } @@ -174,7 +172,7 @@ $form->addHidden('user', $user->getId()); $form->addHidden('id', $skillId); $form->addRule('skill', get_lang('ThisFieldIsRequired'), 'required'); $form->addSelect('acquired_level', get_lang('AcquiredLevel'), $acquiredLevel); -$form->addRule('acquired_level', get_lang('ThisFieldIsRequired'), 'required'); +//$form->addRule('acquired_level', get_lang('ThisFieldIsRequired'), 'required'); $form->addTextarea('argumentation', get_lang('Argumentation'), ['rows' => 6]); $form->addRule('argumentation', get_lang('ThisFieldIsRequired'), 'required'); $form->addRule( @@ -189,7 +187,7 @@ $form->setDefaults($formDefaultValues); if ($form->validate()) { $values = $form->exportValues(); - $skill = $skillRepo->find($values['skill']); + $skill = $skillRepo->find($values['id']); if (!$skill) { Display::addFlash( diff --git a/main/badge/issued.php b/main/badge/issued.php index 4e47f02df0..f565994ee0 100644 --- a/main/badge/issued.php +++ b/main/badge/issued.php @@ -139,7 +139,9 @@ if (!$profile) { if (!$profile && $parent['parent_id'] == 0) { $profile = $skillLevelRepo->findAll(); - $profile = $profile[0]; + if ($profile) { + $profile = $profile[0]; + } } } }