From 49f7ac1c72a8261f29d93de2127898856fd26ee2 Mon Sep 17 00:00:00 2001 From: Cris Date: Wed, 16 Oct 2019 15:13:46 +0200 Subject: [PATCH] Minor changes: Changed tooltip text / Enhanced code . standardized tooltip text . geaenderte Lehrauftraege: bold in bestellen (selectable), marked orange and not selectable in erteilen and akzeptieren . removed parseFloat (not needed, now better perfomance) . better var names for readability reasons --- .../lehre/lehrauftrag/acceptLehrauftrag.php | 34 +++++++++----- .../lehre/lehrauftrag/approveLehrauftrag.php | 47 +++++++++++-------- .../lehre/lehrauftrag/orderLehrauftrag.php | 35 ++++++++------ 3 files changed, 71 insertions(+), 45 deletions(-) diff --git a/application/views/lehre/lehrauftrag/acceptLehrauftrag.php b/application/views/lehre/lehrauftrag/acceptLehrauftrag.php index 454ce2857..e40e21b43 100644 --- a/application/views/lehre/lehrauftrag/acceptLehrauftrag.php +++ b/application/views/lehre/lehrauftrag/acceptLehrauftrag.php @@ -136,6 +136,9 @@ $this->load->view( // Formats the rows function func_rowFormatter(row){ var bestellt = row.getData().bestellt; + var erteilt = row.getData().bestellt; + var akzeptiert = row.getData().akzeptiert; + var betrag = parseFloat(row.getData().betrag); var vertrag_betrag = parseFloat(row.getData().vertrag_betrag); @@ -149,13 +152,13 @@ $this->load->view( row.getCells().forEach(function(cell){ if (bestellt != null && (betrag != vertrag_betrag)) { - row._row.getElement().style['font-weight'] = 'bold'; // geaendert + cell.getElement().classList.add('bg-warning'); // geaenderte } - else if(row.getData().bestellt != null && row.getData().erteilt != null && row.getData().akzeptiert == null) + else if(bestellt != null && erteilt != null && akzeptiert == null) { return; // bestellte + erteilte } - else if(row.getData().bestellt != null && row.getData().erteilt != null && row.getData().akzeptiert != null) + else if(bestellt != null && erteilt != null && akzeptiert != null) { cell.getElement().classList.add('bg-success') // akzeptierte } @@ -168,8 +171,8 @@ $this->load->view( // Formats row selectable/unselectable function func_selectableCheck(row){ - var betrag = parseFloat(row.getData().betrag); - var vertrag_betrag = parseFloat(row.getData().vertrag_betrag); + var betrag = row.getData().betrag; + var vertrag_betrag = row.getData().vertrag_betrag; // only allow to select bestellte && erteilte && nicht geaenderte Lehraufträge return row.getData().bestellt != null && // bestellt @@ -241,6 +244,7 @@ $this->load->view( // Hide betrag, if lector has inkludierte Lehre function func_renderComplete(table){ + // If the lectors actual Verwendung has inkludierte Lehre, hide the column betrag if (has_inkludierteLehre) { @@ -258,8 +262,8 @@ $this->load->view( var erteilt = cell.getRow().getData().erteilt; var akzeptiert = cell.getRow().getData().akzeptiert; - var betrag = parseFloat(cell.getRow().getData().betrag); - var vertrag_betrag = parseFloat(cell.getRow().getData().vertrag_betrag); + var betrag = cell.getRow().getData().betrag; + var vertrag_betrag = cell.getRow().getData().vertrag_betrag; // commented icons would be so nice to have with fontawsome 5.11... if (bestellt != null && isNaN(vertrag_betrag)) @@ -299,14 +303,22 @@ $this->load->view( // Generates status tooltip status_tooltip = function(cell){ var bestellt = cell.getRow().getData().bestellt; + var erteilt = cell.getRow().getData().erteilt; + var akzeptiert = cell.getRow().getData().akzeptiert; + var betrag = parseFloat(cell.getRow().getData().betrag); var vertrag_betrag = parseFloat(cell.getRow().getData().vertrag_betrag); - if (bestellt != null && betrag != vertrag_betrag) { - var text = 'Lehrauftrag wird bearbeitet.'; - text += "\n"; - text += 'Sobald dieser (erneut) erteilt wurde, kann der Lehrauftrag angenommen werden.'; + var text = 'Lehrauftrag in Bearbeitung.'; + text += "\n"; + if (bestellt != null && erteilt == null && akzeptiert == null && betrag != vertrag_betrag) { + text += 'Wartet auf Erteilung, danach können können Sie den Lehrauftrag annehmen.'; + return text; + } + else if (bestellt != null && erteilt != null && akzeptiert == null && betrag != vertrag_betrag) + { + text += 'Wartet auf erneute Erteilung, danach können können Sie den Lehrauftrag annehmen.'; return text; } } diff --git a/application/views/lehre/lehrauftrag/approveLehrauftrag.php b/application/views/lehre/lehrauftrag/approveLehrauftrag.php index 0e3f7ea0b..17113b4bc 100644 --- a/application/views/lehre/lehrauftrag/approveLehrauftrag.php +++ b/application/views/lehre/lehrauftrag/approveLehrauftrag.php @@ -175,12 +175,16 @@ $this->load->view( var is_dummy = (row.getData().personalnummer <= 0 && row.getData().personalnummer != null); var bestellt = row.getData().bestellt; - var betrag = parseFloat(row.getData().betrag); - var vertrag_betrag = parseFloat(row.getData().vertrag_betrag); + var erteilt = row.getData().erteilt; + var akzeptiert = row.getData().akzeptiert; + + var betrag = row.getData().betrag; + var vertrag_betrag = row.getData().vertrag_betrag; + /* Formats the color of the rows depending on their status - blue: dummy lectors - - bold: geaendert + - orange: geaendert - default (white) : bestellte - green: akzeptiert - grey: all other (marks unselectable) @@ -192,13 +196,13 @@ $this->load->view( } else if (bestellt != null && (betrag != vertrag_betrag) && !row._row.element.classList.contains('tabulator-calcs')) // exclude calculation rows { - row._row.getElement().style['font-weight'] = 'bold'; + cell.getElement().classList.add('bg-warning'); // geaenderte } - else if(row.getData().bestellt != null && row.getData().erteilt == null) + else if(bestellt != null && erteilt == null) { return; // bestellt } - else if(row.getData().bestellt != null && row.getData().erteilt != null && row.getData().akzeptiert != null) + else if(bestellt != null && erteilt != null && akzeptiert != null) { cell.getElement().classList.add('bg-success') // akzeptiert } @@ -296,8 +300,8 @@ $this->load->view( var erteilt = cell.getRow().getData().erteilt; var akzeptiert = cell.getRow().getData().akzeptiert; - var betrag = parseFloat(cell.getRow().getData().betrag); - var vertrag_betrag = parseFloat(cell.getRow().getData().vertrag_betrag); + var betrag = cell.getRow().getData().betrag; + var vertrag_betrag = cell.getRow().getData().vertrag_betrag; // commented icons would be so nice to have with fontawsome 5.11... if (is_dummy) @@ -306,26 +310,26 @@ $this->load->view( } else if (bestellt != null && (betrag != vertrag_betrag)) { - return ""; // geaendert + return ""; // geaendert // return ""; // geaendert } else if (bestellt == null && erteilt == null && akzeptiert == null) { - return ""; // neu + return ""; // neu } else if (bestellt != null && erteilt == null && akzeptiert == null) { - return ""; // bestellt + return ""; // bestellt // return ""; // bestellt } else if (bestellt != null && erteilt != null && akzeptiert == null) { - return ""; // erteilt + return ""; // erteilt // return ""; // erteilt } else if (bestellt != null && erteilt != null && akzeptiert != null) { - return ""; // akzeptiert + return ""; // akzeptiert // return ""; // akzeptiert } else @@ -340,12 +344,11 @@ $this->load->view( var bestellt = cell.getRow().getData().bestellt; var erteilt = cell.getRow().getData().erteilt; - var akzeptiert = cell.getRow().getData().akzeptiert; - var betrag = parseFloat(cell.getRow().getData().betrag); - var vertrag_betrag = parseFloat(cell.getRow().getData().vertrag_betrag); + var betrag = cell.getRow().getData().betrag; + var vertrag_betrag = cell.getRow().getData().vertrag_betrag; - var text = 'Lehrauftragsstunden und/oder -betrag wurde/n geändert.'; + var text = 'Lehrauftragstunden/-stundensatz geändert.'; text += "\n"; if (is_dummy) @@ -354,15 +357,19 @@ $this->load->view( } else if (bestellt != null && erteilt == null && betrag != vertrag_betrag) { - return text += 'Erteilen möglich, wenn die Änderungen erneut bestellt worden sind.'; + return text += 'Wartet auf Bestellung, danach Erteilen möglich.'; } else if (bestellt != null && erteilt != null && betrag != vertrag_betrag) { - return text += 'Neuerliches Erteilen möglich, wenn die Änderungen erneut bestellt worden sind.'; + return text += 'Wartet auf neuerliche Bestellung, danach erneut Erteilen möglich.'; + } + else if (bestellt == null) + { + return 'Neuer Lehrauftrag. Wartet auf Bestellung.'; } else if (bestellt != null) { - return text = 'Lehrauftrag wurde bestellt von: ' + cell.getRow().getData().vertrag_insertvon; + return 'Lehrauftrag bestellt von: ' + cell.getRow().getData().vertrag_insertvon; } } $(function() { diff --git a/application/views/lehre/lehrauftrag/orderLehrauftrag.php b/application/views/lehre/lehrauftrag/orderLehrauftrag.php index 6cf57e312..b7796c9ea 100644 --- a/application/views/lehre/lehrauftrag/orderLehrauftrag.php +++ b/application/views/lehre/lehrauftrag/orderLehrauftrag.php @@ -175,8 +175,11 @@ $this->load->view( var is_dummy = (row.getData().personalnummer <= 0 && row.getData().personalnummer != null); var bestellt = row.getData().bestellt; - var betrag = parseFloat(row.getData().betrag); - var vertrag_betrag = parseFloat(row.getData().vertrag_betrag); + var erteilt = row.getData().erteilt; + var akzeptiert = row.getData().akzeptiert; + + var betrag = row.getData().betrag; + var vertrag_betrag = row.getData().vertrag_betrag; /* Formats the color of the rows depending on their status @@ -193,19 +196,19 @@ $this->load->view( } else if (bestellt != null && (betrag != vertrag_betrag) && !row._row.element.classList.contains('tabulator-calcs')) // exclude calculation rows { - // cell.getElement().classList.add('bg-warning') // geaendert + row.getElement().style['font-weight'] = 'bold'; // geaendert } - else if(row.getData().bestellt == null) + else if(bestellt == null) { - return; // neu und erteilt + return; // neu und erteilt } - else if(row.getData().bestellt != null && row.getData().erteilt != null && row.getData().akzeptiert != null) + else if(bestellt != null && erteilt != null && akzeptiert != null) { - cell.getElement().classList.add('bg-success') // akzeptiert + cell.getElement().classList.add('bg-success') // akzeptiert } else { - row.getElement().style["background-color"] = COLOR_LIGHTGREY; // default + row.getElement().style["background-color"] = COLOR_LIGHTGREY; // default } }); } @@ -246,8 +249,8 @@ $this->load->view( var erteilt = row.getData().erteilt; var akzeptiert = row.getData().akzeptiert; - var betrag = parseFloat(row.getData().betrag); - var vertrag_betrag = parseFloat(row.getData().vertrag_betrag); + var betrag = row.getData().betrag; + var vertrag_betrag = row.getData().vertrag_betrag; if (bestellt != null && (betrag != vertrag_betrag)) { @@ -307,13 +310,17 @@ $this->load->view( var erteilt = cell.getRow().getData().erteilt; var akzeptiert = cell.getRow().getData().akzeptiert; - var betrag = parseFloat(cell.getRow().getData().betrag); - var vertrag_betrag = parseFloat(cell.getRow().getData().vertrag_betrag); + var betrag = cell.getRow().getData().betrag; + var vertrag_betrag = cell.getRow().getData().vertrag_betrag; // commented icons would be so nice to have with fontawsome 5.11... if (is_dummy) { - return ""; // kein Vertrag + return ""; // dummy lector + } + else if (bestellt != null && isNaN(vertrag_betrag)) + { + return ""; // kein Vertrag } else if (bestellt != null && (betrag != vertrag_betrag)) { @@ -370,7 +377,7 @@ $this->load->view( } else if (isNaN(vertrag_betrag)) { - return 'Neuer Lehrauftrag. Noch kein Vertrag vorhanden.' + return 'Neuer Lehrauftrag. Wartet auf Bestellung.' } else if (betrag != vertrag_betrag) { var text = 'NACH Änderung: Stundensatz: ' + stundensatz + ' Stunden: ' + stunden;