From c18e1dfdb92661fcb8d2eaff87517ec5232f0f46 Mon Sep 17 00:00:00 2001
From: ulrich
Date: Fri, 23 Apr 2021 17:53:30 +0000
Subject: [PATCH] Abspielprozess wird nicht mehr auf null gesetzt

---
 src/de/uhilger/avdirektor/handler/OMXPlayer.java |   28 ++++++++++++++++++++++------
 1 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/src/de/uhilger/avdirektor/handler/OMXPlayer.java b/src/de/uhilger/avdirektor/handler/OMXPlayer.java
index 0fa96c2..926e9c8 100644
--- a/src/de/uhilger/avdirektor/handler/OMXPlayer.java
+++ b/src/de/uhilger/avdirektor/handler/OMXPlayer.java
@@ -136,11 +136,11 @@
     try {
       Process o = App.getPlayerProcess();
       if(o == null) {
-        antwort = "Es ist kein Player zum Beenden vorhanden, aber der Servlet-Kontext wurde bereinigt.";
-        App.setPlayerProcess(null);
+        antwort = "Es ist kein Player zum Beenden vorhanden.";
+        //App.setPlayerProcess(null);
       } else {
         kommando(CMD_STOP); // setzt den Prozess der App auf null
-        antwort = "Player gestoppt, Kontext bereinigt.";
+        antwort = "Player gestoppt.";
       }
     } 
     catch(Exception ex) {
@@ -163,7 +163,7 @@
       //Object o = t.getAttribute(App.PI_PLAYER);
       Process o = App.getPlayerProcess();
       if(o == null) {
-        App.setPlayerProcess(null);
+        //App.setPlayerProcess(null);
         //servletContext.removeAttribute(PI_PLAYER);
         //t.setAttribute(App.PI_PLAYER, null);
         antwort = "Es wird nichts abgespielt dem ein Kommando gesendet werden kann.";
@@ -175,7 +175,16 @@
         out.flush();
         if(k.equals(CMD_STOP)) {
           out.close();
-          App.setPlayerProcess(null);
+          
+          /*
+            fuer den Fall, dass ein Stopp-Signal den Player nicht erreicht 
+            oder dort nicht funktioniert, gibt es keine Moeglichkeit festzustellen,
+            dass der Player noch spielt. Damit in einem solchen Fall der Zeiger 
+            auf den Abspielprozess nicht verloren geht, wird  der Zeiger nicht 
+            auf null gesetzt.
+          */
+          
+          //App.setPlayerProcess(null);
           //player_process.destroy();
           //player_process = null;
           //t.setAttribute(App.PI_PLAYER, null);
@@ -202,7 +211,14 @@
       logger.log(Level.INFO, 
               "Abspielen beendet, Meldung an {0} mit Statuscode {1} gesendet.", 
               new Object[]{meldeUrlStr, status});
-      App.setPlayerProcess(null);
+          /*
+            fuer den Fall, dass ein Stopp-Signal den Player nicht erreicht 
+            oder dort nicht funktioniert, gibt es keine Moeglichkeit festzustellen,
+            dass der Player noch spielt. Damit in einem solchen Fall der Zeiger 
+            auf den Abspielprozess nicht verloren geht, wird  der Zeiger nicht 
+            auf null gesetzt.
+          */
+      //App.setPlayerProcess(null);
     } catch(IOException ex) {
       logger.log(Level.INFO, ex.getMessage(), ex);
     }

--
Gitblit v1.9.3