From 61a6bc7bfc5cf5c4034799c50d98af17a97bd1fb Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Wed, 25 Sep 2019 16:25:04 +0200
Subject: [PATCH] Fixed SQL query: corrected column order in subquery for
 Projektbetreuung

---
 .../lehrauftrag/acceptLehrauftragData.php     | 20 ++++++++++---------
 .../lehrauftrag/approveLehrauftragData.php    | 20 ++++++++++---------
 .../lehre/lehrauftrag/lehrauftragData.php     | 20 ++++++++++---------
 3 files changed, 33 insertions(+), 27 deletions(-)

diff --git a/application/views/lehre/lehrauftrag/acceptLehrauftragData.php b/application/views/lehre/lehrauftrag/acceptLehrauftragData.php
index fcd22b3ea..d294cd6bd 100644
--- a/application/views/lehre/lehrauftrag/acceptLehrauftragData.php
+++ b/application/views/lehre/lehrauftrag/acceptLehrauftragData.php
@@ -97,15 +97,8 @@ FROM
 
 	    /* Projektbetreuungsaufträge and -vertragsstati */
         SELECT *,
-            /* concatinated and aggregated gruppen */
-            (SELECT
-                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
-                                   \'\n\' || gruppe_kurzbz), \', \')
-             FROM
-                 lehre.tbl_lehreinheitgruppe
-             WHERE
-                 lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
-            )                                                    AS "gruppe",
+            /* mitarbeiter uid retrieved by person_id */
+            /* NOTE: mitarbeiter MUST come after Select * to ensure correct order with select for tmp_lehrauftraege*/
             (SELECT
                  uid
              FROM
@@ -113,6 +106,15 @@ FROM
              WHERE
                  person_id = tmp_projektbetreuung.person_id
                AND aktiv = TRUE)                                 AS "mitarbeiter_uid",
+            /* concatinated and aggregated gruppen */
+            (SELECT
+                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
+                                   \'\n\' || gruppe_kurzbz), \', \')
+             FROM
+                 lehre.tbl_lehreinheitgruppe
+             WHERE
+                     lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
+            )                                                    AS "gruppe",
             /* existing contracts with status bestellt */
             (SELECT
                  datum
diff --git a/application/views/lehre/lehrauftrag/approveLehrauftragData.php b/application/views/lehre/lehrauftrag/approveLehrauftragData.php
index 05247f7ba..eedffb39c 100644
--- a/application/views/lehre/lehrauftrag/approveLehrauftragData.php
+++ b/application/views/lehre/lehrauftrag/approveLehrauftragData.php
@@ -95,15 +95,8 @@ FROM
 
 	    /* Projektbetreuungsaufträge and -vertragsstati */
         SELECT *,
-            /* concatinated and aggregated gruppen */
-            (SELECT
-                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
-                                   \'\n\' || gruppe_kurzbz), \', \')
-             FROM
-                 lehre.tbl_lehreinheitgruppe
-             WHERE
-                 lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
-            )                                                    AS "gruppe",
+            /* mitarbeiter uid retrieved by person_id */
+            /* NOTE: mitarbeiter MUST come after Select * to ensure correct order with select for tmp_lehrauftraege*/
             (SELECT
                  uid
              FROM
@@ -111,6 +104,15 @@ FROM
              WHERE
                  person_id = tmp_projektbetreuung.person_id
                AND aktiv = TRUE)                                 AS "mitarbeiter_uid",
+            /* concatinated and aggregated gruppen */
+            (SELECT
+                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
+                                   \'\n\' || gruppe_kurzbz), \', \')
+             FROM
+                 lehre.tbl_lehreinheitgruppe
+             WHERE
+                     lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
+            )                                                    AS "gruppe",
             /* existing contracts with status bestellt */
             (SELECT
                  datum
diff --git a/application/views/lehre/lehrauftrag/lehrauftragData.php b/application/views/lehre/lehrauftrag/lehrauftragData.php
index b342613de..2f736b2d5 100644
--- a/application/views/lehre/lehrauftrag/lehrauftragData.php
+++ b/application/views/lehre/lehrauftrag/lehrauftragData.php
@@ -95,15 +95,8 @@ FROM
 
 	    /* Projektbetreuungsaufträge and -vertragsstati */
         SELECT *,
-            /* concatinated and aggregated gruppen */
-            (SELECT
-                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
-                                   \'\n\' || gruppe_kurzbz), \', \')
-             FROM
-                 lehre.tbl_lehreinheitgruppe
-             WHERE
-                 lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
-            )                                                    AS "gruppe",
+            /* mitarbeiter uid retrieved by person_id */
+            /* NOTE: mitarbeiter MUST come after Select * to ensure correct order with select for tmp_lehrauftraege*/
             (SELECT
                  uid
              FROM
@@ -111,6 +104,15 @@ FROM
              WHERE
                  person_id = tmp_projektbetreuung.person_id
                AND aktiv = TRUE)                                 AS "mitarbeiter_uid",
+            /* concatinated and aggregated gruppen */
+            (SELECT
+                 string_agg(concat(stg_oe_kurzbz, \'-\', semester, verband, gruppe,
+                                   \'\n\' || gruppe_kurzbz), \', \')
+             FROM
+                 lehre.tbl_lehreinheitgruppe
+             WHERE
+                     lehreinheit_id = tmp_projektbetreuung.lehreinheit_id
+            )                                                    AS "gruppe",
             /* existing contracts with status bestellt */
             (SELECT
                  datum