From 746f0821211857de3af063bf369da30979ac3b45 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Thu, 24 Oct 2019 14:14:48 +0200
Subject: [PATCH] Fixed: Excluded dummy lectors from all selections/filters

Some selections/filters were not selecting/filtering correctly out the
dummy lectors OR filtering out Betreuung.
It needs: personalnummer >= 0
  . > to ensure no dummies
  . = to retrieve also Betreuungen
     (where personalnummer is provided with NULL)
---
 application/views/lehre/lehrauftrag/approveLehrauftrag.php | 4 ++--
 application/views/lehre/lehrauftrag/orderLehrauftrag.php   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/application/views/lehre/lehrauftrag/approveLehrauftrag.php b/application/views/lehre/lehrauftrag/approveLehrauftrag.php
index ed2bf96a6..6f7094252 100644
--- a/application/views/lehre/lehrauftrag/approveLehrauftrag.php
+++ b/application/views/lehre/lehrauftrag/approveLehrauftrag.php
@@ -193,7 +193,7 @@ $this->load->view(
     function filter_showChanged(data){
 
         // Filters geaenderte from status bestellt on
-        return  data.personalnummer > 0 &&          // NOT dummy lector AND
+        return  data.personalnummer >= 0 &&          // NOT dummy lector AND
                 data.bestellt != null &&            // bestellt AND
                 data.betrag != data.vertrag_betrag; // geaendert
     }
@@ -512,7 +512,6 @@ $(function() {
     $("#show-new").click(function(){
         $('#filterTabulator').tabulator('setFilter',
             [
-                {field: 'personalnummer', type: '!=', value: null},
                 {field: 'personalnummer', type: '>=', value: 0},
                 {field: 'bestellt', type: '=', value: null},
                 {field: 'erteilt', type: '=', value: null},
@@ -525,6 +524,7 @@ $(function() {
     $("#show-ordered").click(function(){
         $('#filterTabulator').tabulator('setFilter',
             [
+                {field: 'personalnummer', type: '>=', value: 0},
                 {field: 'bestellt', type: '!=', value: null},
                 {field: 'erteilt', type: '=', value: null},
                 {field: 'akzeptiert', type: '=', value: null}
diff --git a/application/views/lehre/lehrauftrag/orderLehrauftrag.php b/application/views/lehre/lehrauftrag/orderLehrauftrag.php
index 87315da43..0406f0816 100644
--- a/application/views/lehre/lehrauftrag/orderLehrauftrag.php
+++ b/application/views/lehre/lehrauftrag/orderLehrauftrag.php
@@ -198,7 +198,7 @@ $this->load->view(
     function filter_showChanged(data){
 
         // Filters geaenderte from status bestellt on
-        return  data.personalnummer > 0 &&          // NOT dummy lector AND
+        return  data.personalnummer >= 0 &&         // NOT dummy lector AND
                 data.bestellt != null &&            // bestellt AND
                 data.betrag != data.vertrag_betrag; // geaendert
     }
@@ -543,7 +543,6 @@ $(function() {
     $("#show-new").click(function(){
         $('#filterTabulator').tabulator('setFilter',
             [
-                {field: 'personalnummer', type: '!=', value: null},
                 {field: 'personalnummer', type: '>=', value: 0},
                 {field: 'bestellt', type: '=', value: null},
                 {field: 'erteilt', type: '=', value: null},
@@ -556,6 +555,7 @@ $(function() {
     $("#show-ordered").click(function(){
         $('#filterTabulator').tabulator('setFilter',
             [
+                {field: 'personalnummer', type: '>=', value: 0},
                 {field: 'bestellt', type: '!=', value: null},
                 {field: 'erteilt', type: '=', value: null},
                 {field: 'akzeptiert', type: '=', value: null}