diff --git a/application/libraries/issues/PlausicheckLib.php b/application/libraries/issues/PlausicheckLib.php index 696df1647..4df7b26ca 100644 --- a/application/libraries/issues/PlausicheckLib.php +++ b/application/libraries/issues/PlausicheckLib.php @@ -74,8 +74,8 @@ class PlausicheckLib $qry = " SELECT prestudent.person_id, prestudent.prestudent_id, status.studiensemester_kurzbz, - studiengang.orgform_kurzbz as stg_orgform, status.orgform_kurzbz as student_orgform, - stg.oe_kurzbz AS prestudent_stg_oe_kurzbz + studiengang.orgform_kurzbz AS stg_orgform, status.orgform_kurzbz AS student_orgform, + prestudent.studiengang_kz AS student_studiengang, stg.oe_kurzbz AS prestudent_stg_oe_kurzbz FROM public.tbl_studiengang studiengang JOIN public.tbl_student student USING(studiengang_kz) @@ -89,7 +89,8 @@ class PlausicheckLib AND studiengang.studiengang_kz < 10000 AND status.studiensemester_kurzbz = ? AND NOT EXISTS( - SELECT 1 FROM lehre.tbl_studienplan JOIN lehre.tbl_studienordnung USING(studienordnung_id) + SELECT 1 FROM lehre.tbl_studienplan + JOIN lehre.tbl_studienordnung USING(studienordnung_id) WHERE tbl_studienordnung.studiengang_kz = prestudent.studiengang_kz AND tbl_studienplan.orgform_kurzbz = status.orgform_kurzbz)"; @@ -171,7 +172,8 @@ class PlausicheckLib JOIN public.tbl_person USING(person_id) JOIN public.tbl_studiengang stg ON ps.studiengang_kz = stg.studiengang_kz WHERE - ps.studiengang_kz<>stordnung.studiengang_kz"; + ps.studiengang_kz<>stordnung.studiengang_kz + AND stg.melderelevant"; if (isset($studiengang_kz)) { @@ -284,7 +286,7 @@ class PlausicheckLib $qry = " SELECT DISTINCT(student.student_uid), prestudent.person_id, prestudent.prestudent_id, - status.ausbildungssemester, lv.semester, status.studiensemester_kurzbz, + status.ausbildungssemester AS status_ausbildungssemester, lv.semester AS student_ausbildungssemester, status.studiensemester_kurzbz, stg.oe_kurzbz AS prestudent_stg_oe_kurzbz FROM public.tbl_student student diff --git a/application/libraries/issues/plausichecks/AusbildungssemPrestudentUngleichAusbildungssemStatus.php b/application/libraries/issues/plausichecks/AusbildungssemPrestudentUngleichAusbildungssemStatus.php index c3fb1ae66..41df05acd 100644 --- a/application/libraries/issues/plausichecks/AusbildungssemPrestudentUngleichAusbildungssemStatus.php +++ b/application/libraries/issues/plausichecks/AusbildungssemPrestudentUngleichAusbildungssemStatus.php @@ -36,6 +36,8 @@ class AusbildungssemPrestudentUngleichAusbildungssemStatus extends PlausiChecker 'person_id' => $prestudent->person_id, 'oe_kurzbz' => $prestudent->prestudent_stg_oe_kurzbz, 'fehlertext_params' => array( + 'student_ausbildungssemester' => $prestudent->student_ausbildungssemester, + 'status_ausbildungssemester' => $prestudent->status_ausbildungssemester, 'prestudent_id' => $prestudent->prestudent_id, 'studiensemester_kurzbz' => $prestudent->studiensemester_kurzbz ), diff --git a/application/libraries/issues/plausichecks/OrgformStgUngleichOrgformPrestudent.php b/application/libraries/issues/plausichecks/OrgformStgUngleichOrgformPrestudent.php index d663f6797..b7ef64c41 100644 --- a/application/libraries/issues/plausichecks/OrgformStgUngleichOrgformPrestudent.php +++ b/application/libraries/issues/plausichecks/OrgformStgUngleichOrgformPrestudent.php @@ -33,9 +33,10 @@ class OrgformStgUngleichOrgformPrestudent extends PlausiChecker 'person_id' => $prestudent->person_id, 'oe_kurzbz' => $prestudent->prestudent_stg_oe_kurzbz, 'fehlertext_params' => array( - 'stg_orgform' => $prestudent->stg_orgform, + 'student_studiengang' => $prestudent->student_studiengang, 'student_orgform' => $prestudent->student_orgform, - 'prestudent_id' => $prestudent->prestudent_id + 'prestudent_id' => $prestudent->prestudent_id, + 'studiensemester_kurzbz' => $prestudent->studiensemester_kurzbz ), 'resolution_params' => array( 'prestudent_id' => $prestudent->prestudent_id, diff --git a/application/libraries/issues/resolvers/CORE_STUDENTSTATUS_0014.php b/application/libraries/issues/resolvers/CORE_STUDENTSTATUS_0014.php index 9b70219f4..8d96b7c65 100644 --- a/application/libraries/issues/resolvers/CORE_STUDENTSTATUS_0014.php +++ b/application/libraries/issues/resolvers/CORE_STUDENTSTATUS_0014.php @@ -12,15 +12,12 @@ class CORE_STUDENTSTATUS_0014 implements IIssueResolvedChecker if (!isset($params['prestudent_id']) || !is_numeric($params['prestudent_id'])) return error('Prestudent Id missing, issue_id: '.$params['issue_id']); - if (!isset($params['studiensemester_kurzbz']) || isEmptyString($params['studiensemester_kurzbz'])) - return error('Studiensemester missing, issue_id: '.$params['issue_id']); - $this->_ci =& get_instance(); // get code igniter instance $this->_ci->load->library('issues/PlausicheckLib'); // check if issue persists - $checkRes = $this->_ci->plausichecklib->getAbschlussstatusFehlt($params['studiensemester_kurzbz'], null, $params['prestudent_id']); + $checkRes = $this->_ci->plausichecklib->getAbschlussstatusFehlt(null, null, $params['prestudent_id']); if (isError($checkRes)) return $checkRes; diff --git a/system/fehlerupdate.php b/system/fehlerupdate.php index 0827c6225..0d8f653a6 100644 --- a/system/fehlerupdate.php +++ b/system/fehlerupdate.php @@ -137,7 +137,7 @@ $fehlerArr = array( 'fehlercode' => 'CORE_STG_0002', 'fehler_kurzbz' => 'OrgformStgUngleichOrgformPrestudent', 'fehlercode_extern' => null, - 'fehlertext' => 'Organisationsform des Studienganges (%s) stimmt nicht mit Organisationsformen des Studenten (%s) überein. (prestudent_id %s)', + 'fehlertext' => 'Es ist kein Studienplan mit Studiengang (%s) und Organisationsform (%s) des Studenten zugewiesen. (prestudent_id %s, Studiensemester %s)', 'fehlertyp_kurzbz' => 'error', 'app' => 'core' ), @@ -177,7 +177,7 @@ $fehlerArr = array( 'fehlercode' => 'CORE_STUDENTSTATUS_0003', 'fehler_kurzbz' => 'AusbildungssemPrestudentUngleichAusbildungssemStatus', 'fehlercode_extern' => null, - 'fehlertext' => 'Aktuelles Ausbildungssemester bei StudentIn (Lehrverband) stimmt nicht mit dem Ausbildungssemester des aktuellen Status überein. (prestudent_id %s, Studiensemester %s)', + 'fehlertext' => 'Aktuelles Ausbildungssemester %s bei StudentIn (Lehrverband) stimmt nicht mit dem Ausbildungssemester %s des aktuellen Status überein. (prestudent_id %s, Studiensemester %s)', 'fehlertyp_kurzbz' => 'error', 'app' => 'core' ),