From b0d9010aec6f85128753c41f4c3e5c7b0698ca36 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Tue, 12 Nov 2019 16:17:45 +0100
Subject: [PATCH] =?UTF-8?q?Fixed:=20now=20status=20'ge=C3=A4ndert'=20rende?=
 =?UTF-8?q?rs=20correctly=20when=20Stunden=20are=20NULL?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When FAS Stunden were left empty, the vertrag_stunden were compared with
null stunden. So: comparing 0 with null was setting status to 'geaendert',
although practically there was no change to the contract.
This is fixed now.
---
 .../lehre/lehrauftrag/approveLehrauftrag.js   | 70 ++++++++++++++++
 .../js/lehre/lehrauftrag/orderLehrauftrag.js  | 79 ++++++++++++++++++-
 2 files changed, 146 insertions(+), 3 deletions(-)

diff --git a/public/js/lehre/lehrauftrag/approveLehrauftrag.js b/public/js/lehre/lehrauftrag/approveLehrauftrag.js
index f07251882..2cc42abd0 100644
--- a/public/js/lehre/lehrauftrag/approveLehrauftrag.js
+++ b/public/js/lehre/lehrauftrag/approveLehrauftrag.js
@@ -118,6 +118,21 @@ function func_rowFormatter(row){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     /*
     Formats the color of the rows depending on their status
     - blue: dummy lectors
@@ -167,6 +182,21 @@ function func_selectableCheck(row){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     // only allow to select bestellte Lehraufträge
     return  !is_dummy &&                        // NOT dummy lector
         row.getData().bestellt != null &&   // AND NOT neue
@@ -210,6 +240,21 @@ function func_renderStarted(table){
             betrag = 0;
         }
 
+        if (isNaN(stunden))
+        {
+            stunden = 0;
+        }
+
+        if (isNaN(vertrag_stunden))
+        {
+            vertrag_stunden = 0;
+        }
+
+        if (isNaN(vertrag_betrag))
+        {
+            vertrag_betrag = 0;
+        }
+
         if ((bestellt != null && betrag != vertrag_betrag) ||
             (bestellt != null && stunden != vertrag_stunden))
         {
@@ -332,6 +377,21 @@ status_formatter = function(cell, formatterParams, onRendered){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     // commented icons would be so nice to have with fontawsome 5.11...
     if (is_dummy)
     {
@@ -387,6 +447,16 @@ status_tooltip = function(cell){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     if (isNaN(vertrag_stunden)){
         vertrag_stunden = 0;
     }
diff --git a/public/js/lehre/lehrauftrag/orderLehrauftrag.js b/public/js/lehre/lehrauftrag/orderLehrauftrag.js
index 8a7af5236..dfcc46636 100644
--- a/public/js/lehre/lehrauftrag/orderLehrauftrag.js
+++ b/public/js/lehre/lehrauftrag/orderLehrauftrag.js
@@ -118,14 +118,26 @@ function func_rowFormatter(row){
     var betrag = parseFloat(row.getData().betrag);
     var vertrag_betrag = parseFloat(row.getData().vertrag_betrag);
 
-    // console.log(betrag);
-    // console.log(vertrag_betrag);
-
     if (isNaN(betrag))
     {
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     /*
     Formats the color of the rows depending on their status
     - blue: dummy lectors
@@ -175,6 +187,22 @@ function func_selectableCheck(row){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
+
     // Only allow to select neue and geaenderte
     return  !is_dummy &&                                                    // NOT dummy lector
         row.getData().bestellt == null ||                               // AND neue
@@ -217,6 +245,21 @@ function func_renderStarted(table){
             betrag = 0;
         }
 
+        if (isNaN(stunden))
+        {
+            stunden = 0;
+        }
+
+        if (isNaN(vertrag_stunden))
+        {
+            vertrag_stunden = 0;
+        }
+
+        if (isNaN(vertrag_betrag))
+        {
+            vertrag_betrag = 0;
+        }
+
         if ((bestellt != null && betrag != vertrag_betrag) ||
             (bestellt != null && stunden != vertrag_stunden))
         {
@@ -341,6 +384,21 @@ status_formatter = function(cell, formatterParams, onRendered){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     // commented icons would be so nice to have with fontawsome 5.11...
     if (is_dummy)
     {
@@ -401,6 +459,21 @@ status_tooltip = function(cell){
         betrag = 0;
     }
 
+    if (isNaN(stunden))
+    {
+        stunden = 0;
+    }
+
+    if (isNaN(vertrag_stunden))
+    {
+        vertrag_stunden = 0;
+    }
+
+    if (isNaN(vertrag_betrag))
+    {
+        vertrag_betrag = 0;
+    }
+
     // Calculate vertrag stundensatz
     if (vertrag_stunden != 0)
     {