Skip to content
Snippets Groups Projects
Commit b2e8e62e authored by Tim Steiner's avatar Tim Steiner
Browse files

Adding a new course was broken by some changes to CourseCrosslisting model. Resolved those bugs.

parent b4bba720
No related branches found
No related tags found
No related merge requests found
......@@ -19,11 +19,6 @@ class CourseCrosslisting extends Nmc_Db_Table_Row
$this->_integratedStudies = $courseCode->integratedStudies;
$this->_courseCodeRow = $courseCode;
}
$courseCodeRelation = new Nmc_Db_Table_Relation_HasOne(CourseCodes::getInstance(),
$this, 'course_code', 'courseCode', true);
$courseCodeRelation->setDoNotClone();
$this->_registerRelation($courseCodeRelation);
}
public function _get($name)
......
......@@ -153,10 +153,13 @@ class CourseGeneration extends Asset
public function isEssentialStudies($college = null)
{
if (is_null($this->_homeCrosslisting->courseCode)) {
$courseCode = CourseCodes::getInstance()->findBySubjectNumberAndLetter($this->subject,
$this->courseNumber,
$this->courseLetter);
if (is_null($courseCode)) {
return false;
}
return CourseEsDesignations::getInstance()->isCourseCodeEssentialStudies($this->_homeCrosslisting->courseCode);
return CourseEsDesignations::getInstance()->isCourseCodeEssentialStudies($courseCode);
}
/**
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment