From f5d398e44cdbc5c58d2cf75f8d6a069e1c8f87de Mon Sep 17 00:00:00 2001
From: Johann Maierhofer <126261667+JohannMaierhofer@users.noreply.github.com>
Date: Fri, 26 Jan 2024 18:11:35 +0100
Subject: [PATCH] Fix MitgliedDetailAction (#130)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Wenn in den Einstellungen "Juristische Personen erlaubt" ausgewählt ist und man eine neues Mtglied erzeugt erscheint der Personenart Dialog.
Bei ESC und Abbruch Button erscheint eine Fehlermeldung.
Beim Schliesen über das Schliesen Icon kommt es zu einer Stacktraceausgabe.
Der Fix behebt beides.
---
 .../JVerein/gui/action/MitgliedDetailAction.java         | 9 +++++++++
 .../jost_net/JVerein/gui/dialogs/PersonenartDialog.java  | 2 +-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/de/jost_net/JVerein/gui/action/MitgliedDetailAction.java b/src/de/jost_net/JVerein/gui/action/MitgliedDetailAction.java
index 481432662..e9c3c2a2a 100644
--- a/src/de/jost_net/JVerein/gui/action/MitgliedDetailAction.java
+++ b/src/de/jost_net/JVerein/gui/action/MitgliedDetailAction.java
@@ -26,6 +26,7 @@
 import de.jost_net.JVerein.rmi.Mitgliedskonto;
 import de.willuhn.jameica.gui.Action;
 import de.willuhn.jameica.gui.GUI;
+import de.willuhn.jameica.system.OperationCanceledException;
 import de.willuhn.util.ApplicationException;
 
 public class MitgliedDetailAction implements Action
@@ -65,6 +66,10 @@ else if (context != null && (context instanceof Mitgliedskonto))
           PersonenartDialog pad = new PersonenartDialog(
               PersonenartDialog.POSITION_CENTER);
           String pa = pad.open();
+          if (pa == null)
+          {
+            return;
+          }
           m.setPersonenart(pa);
         }
         else
@@ -81,6 +86,10 @@ else if (context != null && (context instanceof Mitgliedskonto))
         GUI.startView(new AdresseDetailView(), m);
       }
     }
+    catch (OperationCanceledException oce)
+    {
+      throw oce;
+    }
     catch (Exception e)
     {
       throw new ApplicationException(
diff --git a/src/de/jost_net/JVerein/gui/dialogs/PersonenartDialog.java b/src/de/jost_net/JVerein/gui/dialogs/PersonenartDialog.java
index f580644e3..0442ad5f8 100644
--- a/src/de/jost_net/JVerein/gui/dialogs/PersonenartDialog.java
+++ b/src/de/jost_net/JVerein/gui/dialogs/PersonenartDialog.java
@@ -42,7 +42,7 @@ public PersonenartDialog(int position)
     super(position);
 
     setTitle("Personenart");
-    setSize(450, 200);
+    setSize(400, 140);
   }
 
   @Override