From 15121ced6d1381a03649637d33b2ba2f595be081 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Thu, 8 Sep 2022 14:08:46 +0200
Subject: [PATCH] Fixed missing disabled formular and wrong status-test when
 student emits Anrechnung with Begruendung Hochschule

---
 .../js/lehre/anrechnung/requestAnrechnung.js  | 93 ++++++++++---------
 1 file changed, 49 insertions(+), 44 deletions(-)

diff --git a/public/js/lehre/anrechnung/requestAnrechnung.js b/public/js/lehre/anrechnung/requestAnrechnung.js
index dd93394d4..5edb4e162 100644
--- a/public/js/lehre/anrechnung/requestAnrechnung.js
+++ b/public/js/lehre/anrechnung/requestAnrechnung.js
@@ -7,49 +7,6 @@ const COLOR_DANGER = '#f2dede';
 $(function(){
     const uploadMaxFilesize = $('#requestAnrechnung-uploadfile').data('maxsize')  ; // in byte
 
-    let status_kurzbz = $('#requestAnrechnung-status_kurzbz').data('status_kurzbz');
-    if (status_kurzbz != ' ' && status_kurzbz != ANRECHNUNGSTATUS_APPROVED)
-    {
-        var ectsLv = parseFloat($('#ects').text());
-        var sumEctsSchulisch = parseFloat($('#sumEctsSchulisch').text());
-        var sumEctsBeruflich = parseFloat($('#sumEctsBeruflich').text());
-        var begruendung_id = $('#requestAnrechnung-form :input[name="begruendung"]:checked').val();
-
-        // If Begründung is 'Hochschulzeugnis', return. They are accepted without limit.
-        if (begruendung_id == 5)
-        {
-            exit;
-        }
-
-        // If max ECTS is ecceeded
-        if (begruendung_id == 4)
-        {
-            if ((sumEctsSchulisch + sumEctsBeruflich + ectsLv) > 90 ||
-                (sumEctsBeruflich + ectsLv) > 60
-            )
-            {
-                // Get ECTS Überschreitungs-message, depending on schulische or berufliche Qualifikation
-                var msgBeiEctsUeberschreitung = requestAnrechnung.getMsgBeiEctsUeberschreitung(begruendung_id, ectsLv, sumEctsSchulisch, sumEctsBeruflich);
-
-                // Add to Checkbox text
-                $('#requestAnrechnung-form :input[name="begruendung"]:checked').closest('label').append(msgBeiEctsUeberschreitung);
-            }
-        }
-        else
-        {
-            if ((sumEctsSchulisch + sumEctsBeruflich + ectsLv) > 90 ||
-                (sumEctsSchulisch + ectsLv) > 60
-            )
-            {
-                // Get ECTS Überschreitungs-message, depending on schulische or berufliche Qualifikation
-                var msgBeiEctsUeberschreitung = requestAnrechnung.getMsgBeiEctsUeberschreitung(begruendung_id, ectsLv, sumEctsSchulisch, sumEctsBeruflich);
-
-                // Add to Checkbox text
-                $('#requestAnrechnung-form :input[name="begruendung"]:checked').closest('label').append(msgBeiEctsUeberschreitung);
-            }
-        }
-    }
-
     // Set status alert color
     requestAnrechnung.setStatusAlertColor();
 
@@ -62,9 +19,12 @@ $(function(){
     // Init tooltips
     requestAnrechnung.initTooltips();
 
-    // // Alert message, if maximum ECTS exceeded
+    // Alert message, if maximum ECTS exceeded
     requestAnrechnung.alertIfMaxEctsExceeded();
 
+    // Alert message inside Begruendungsbox, if maximum ECTS exceeded
+    requestAnrechnung.alertIfMaxEctsExceededInsideBegruendungsbox();
+
     // Set chars counter for textarea 'Herkunft der Kenntnisse'
     requestAnrechnung.setCharsCounter();
 
@@ -318,6 +278,51 @@ var requestAnrechnung = {
                 });
         }
     },
+    alertIfMaxEctsExceededInsideBegruendungsbox: function(){
+        let status_kurzbz = $('#requestAnrechnung-status_kurzbz').data('status_kurzbz');
+
+        if (status_kurzbz != ' ' && status_kurzbz != ANRECHNUNGSTATUS_APPROVED)
+        {
+            var ectsLv = parseFloat($('#ects').text());
+            var sumEctsSchulisch = parseFloat($('#sumEctsSchulisch').text());
+            var sumEctsBeruflich = parseFloat($('#sumEctsBeruflich').text());
+            var begruendung_id = $('#requestAnrechnung-form :input[name="begruendung"]:checked').val();
+
+            // If Begründung is 'Hochschulzeugnis', return. They are accepted without limit.
+            if (begruendung_id == 5)
+            {
+                return;
+            }
+
+            // If max ECTS is ecceeded
+            if (begruendung_id == 4)
+            {
+                if ((sumEctsSchulisch + sumEctsBeruflich + ectsLv) > 90 ||
+                    (sumEctsBeruflich + ectsLv) > 60
+                )
+                {
+                    // Get ECTS Überschreitungs-message, depending on schulische or berufliche Qualifikation
+                    var msgBeiEctsUeberschreitung = requestAnrechnung.getMsgBeiEctsUeberschreitung(begruendung_id, ectsLv, sumEctsSchulisch, sumEctsBeruflich);
+
+                    // Add to Checkbox text
+                    $('#requestAnrechnung-form :input[name="begruendung"]:checked').closest('label').append(msgBeiEctsUeberschreitung);
+                }
+            }
+            else
+            {
+                if ((sumEctsSchulisch + sumEctsBeruflich + ectsLv) > 90 ||
+                    (sumEctsSchulisch + ectsLv) > 60
+                )
+                {
+                    // Get ECTS Überschreitungs-message, depending on schulische or berufliche Qualifikation
+                    var msgBeiEctsUeberschreitung = requestAnrechnung.getMsgBeiEctsUeberschreitung(begruendung_id, ectsLv, sumEctsSchulisch, sumEctsBeruflich);
+
+                    // Add to Checkbox text
+                    $('#requestAnrechnung-form :input[name="begruendung"]:checked').closest('label').append(msgBeiEctsUeberschreitung);
+                }
+            }
+        }
+    },
     getMsgBeiEctsUeberschreitung: function(begruendung_id, ects, sumEctsSchulisch, sumEctsBeruflich){
 
         return $('<span id="sumEctsMsg"></span>')