Skip to content

Commit

Permalink
Fix MitgliedMailSendenAction (#123)
Browse files Browse the repository at this point in the history
Gleicher Fehler wie bei Fix #118.
Beim Schließen des Mail Vorlage Dialogs mit Schließen Icon wird trotzdem weiter gemacht wie bei Weiter Button.
Try/Catch Handling vereinfacht
  • Loading branch information
JohannMaierhofer committed Jan 24, 2024
1 parent 3c6f483 commit eaf6695
Showing 1 changed file with 18 additions and 23 deletions.
41 changes: 18 additions & 23 deletions src/de/jost_net/JVerein/gui/action/MitgliedMailSendenAction.java
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
**********************************************************************/
package de.jost_net.JVerein.gui.action;

import java.rmi.RemoteException;
import java.util.ArrayList;
import java.util.TreeSet;

Expand All @@ -32,6 +31,7 @@
import de.willuhn.jameica.gui.Action;
import de.willuhn.jameica.gui.GUI;
import de.willuhn.jameica.gui.dialogs.YesNoDialog;
import de.willuhn.jameica.system.OperationCanceledException;
import de.willuhn.logging.Logger;
import de.willuhn.util.ApplicationException;

Expand Down Expand Up @@ -71,7 +71,7 @@ else if (context instanceof Mitglied[])
mitgliederohnemail.append(", ");
}
mitgliederohnemail
.append(Adressaufbereitung.getNameVorname(mitglied));
.append(Adressaufbereitung.getNameVorname(mitglied));
}
else
{
Expand All @@ -85,50 +85,45 @@ else if (context instanceof Mitglied[])
d.setTitle("Mail senden");
d.setText("Folgende Mitglieder haben keine Mail-Adresse:"
+ mitgliederohnemail.toString() + "\nWeiter?");
try
{
Boolean choice = (Boolean) d.open();
if (!choice.booleanValue())
{
return;
}
}
catch (Exception e)

Boolean choice = (Boolean) d.open();
if (!choice.booleanValue())
{
Logger.error("Fehler bei der Auswahl der Mail-Empfänger", e);
return;
}

}
MailVorlagenAuswahlDialog mvad = new MailVorlagenAuswahlDialog(
new MailVorlageControl(null),
MailVorlagenAuswahlDialog.POSITION_CENTER);
Mail mail = (Mail) Einstellungen.getDBService().createObject(Mail.class,
null);
try

MailVorlage mv = mvad.open();
if (!mvad.getAbort())
{
MailVorlage mv = mvad.open();
if (mv != null)
{
mail.setBetreff(mv.getBetreff());
mail.setTxt(mv.getTxt());
}
mail.setEmpfaenger(empf);
GUI.startView(MailDetailView.class.getName(), mail);
}
catch (Exception e)
{
Logger.error("Fehler", e);
}
mail.setEmpfaenger(empf);
GUI.startView(MailDetailView.class.getName(), mail);
}
else
{
throw new ApplicationException("Keinen Empfänger ausgewählt");
}
}
catch (RemoteException e)
catch (OperationCanceledException oce)
{
throw oce;
}
catch (Exception e)
{
throw new ApplicationException("Fehler: " + e.getLocalizedMessage());
Logger.error("Fehler", e);
GUI.getStatusBar().setErrorText(
"Fehler bei der Erzeugung der neuen Mail");
}
}
}

0 comments on commit eaf6695

Please sign in to comment.