Skip to content
Snippets Groups Projects
Commit 5e9e85ac authored by Roger W Feese's avatar Roger W Feese
Browse files

Added checks to make sure course is currently ACE and not removed.

parent 90a8324c
No related branches found
No related tags found
1 merge request!16Projects Task 31
......@@ -161,6 +161,18 @@ class Iace_ReportsController extends Creq_Controller_Action
$data = new Unl_Model_Collection('Unl_Model_Array');
foreach ($courses as $requestId => $course) {
$currentCourse = $currentCourses[$course->getCourseId()];
//skip course if it has been removed
if($currentCourse->getRemoved()){
continue;
}
//skip course if it is not ACE
if(!$currentCourse->isAce()){
continue;
}
$request = $requests[$requestId];
$finalizeTime = $finalizeTimes[$request->getId()];
$title = $course->getTitle();
......@@ -169,7 +181,6 @@ class Iace_ReportsController extends Creq_Controller_Action
$title = strtr($title, array('JDEP' => 'RAIK'));
}
foreach ($course->getAceOutcomes() as $aceOutcome) {
$currentCourse = $currentCourses[$course->getCourseId()];
$currentSLOs = array();
foreach ($currentCourse->getAceOutcomes() as $currentAceOutcome) {
$currentSLOs[] = $currentAceOutcome['slo'];
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment