1 files changed, 62 insertions, 89 deletions

diff --git a/dozentenmodul/src/main/java/gui/image/DeleteImage_GUI.java b/dozentenmodul/src/main/java/gui/image/DeleteImage_GUI.java
index 21128e48..cf448f70 100644
--- a/dozentenmodul/src/main/java/gui/image/DeleteImage_GUI.java
+++ b/dozentenmodul/src/main/java/gui/image/DeleteImage_GUI.java
@@ -6,7 +6,6 @@ import gui.intro.MainMenue_GUI;
 import java.awt.Color;
 import java.awt.FlowLayout;
 import java.awt.Font;
-import java.awt.HeadlessException;
 import java.awt.SystemColor;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
@@ -163,8 +162,7 @@ public class DeleteImage_GUI extends JInternalFrame {
 			JTextPane txtpnBitteWhlenSie = new JTextPane();
 			txtpnBitteWhlenSie.setEditable(false);
 			txtpnBitteWhlenSie.setBackground(SystemColor.menu);
-			txtpnBitteWhlenSie
-					.setText("Wählen Sie bitte aus der unten stehenden Liste das zu löschende Image aus.");
+			txtpnBitteWhlenSie.setText("Wählen Sie bitte aus der unten stehenden Liste das zu löschende Image aus.");
 			txtpnBitteWhlenSie.setBounds(10, 36, 509, 32);
 			panel.add(txtpnBitteWhlenSie);
 		}
@@ -291,97 +289,72 @@ public class DeleteImage_GUI extends JInternalFrame {
 				String imageversion = "";
 				int eingabe;
 
-				if (tablemyImages.getSelectedRowCount() != 0) {
-
-					// frage, soll das selektierte Daten geloescht werden?
-					eingabe = JOptionPane.showConfirmDialog(null, "Wollen Sie diese Daten löschen?",
-							"Löschen?", JOptionPane.YES_NO_OPTION);
-
-					if (eingabe == JOptionPane.YES_OPTION) {
-
-						imageid = modelMyImages.getValueAt(
-								tablemyImages.convertRowIndexToModel(tablemyImages.getSelectedRow()), 6)
-								.toString();
-
-						imageversion = modelMyImages.getValueAt(
-								tablemyImages.convertRowIndexToModel(tablemyImages.getSelectedRow()), 7)
-								.toString();
-
-						// Image darf nur geloescht werden, wenn keine
-						// Veranstaltung mehr auf das Image verweist.
-						// Unbdingt abfragen!
-
-						try {
-							if (!ThriftManager.getSatClient().connectedToLecture(imageid, imageversion,
-									SessionData.authToken)) {
-								try {
-
-									//  try to delete file from file system
-									if (ThriftManager.getSatClient().deleteImageServer(imageid, imageversion,
-											SessionData.authToken) == true) {
-										// successful, now delete file from DB
-										if (ThriftManager.getSatClient().deleteImageData(imageid,
-												imageversion, SessionData.authToken)) {
-
-											LOGGER.info("Image '" + imageid + "' erfolgreich gelöscht.");
-
-											GuiManager.show(new DeleteImage_GUI());
-										} else {
-											//could not delete file from DB
-											LOGGER.info("Image '" + imageid
-													+ "' konnte nicht gelöscht werden.");
-
-											JOptionPane.showMessageDialog(null,
-													"Image konnte nicht gelöscht werden.", "Fehler",
-													JOptionPane.INFORMATION_MESSAGE);
-										}
-									} else {
-										// could not delete file from file
-										// system
-										LOGGER.info("Image '" + imageid
-												+ "' konnte nicht von Server gelöscht werden.");
-
-										JOptionPane.showMessageDialog(null,
-												"Image konnte nicht vom Server gelöscht werden.", "Fehler",
-												JOptionPane.INFORMATION_MESSAGE);
-									}
-
-								} catch (HeadlessException e) {
-
-									e.printStackTrace();
-									JOptionPane.showMessageDialog(null,
-											e.getCause() + "\n" + e.getStackTrace(), "Debug-Message",
-											JOptionPane.ERROR_MESSAGE);
-								} catch (TException e) {
-
-									e.printStackTrace();
-									JOptionPane.showMessageDialog(null,
-											e.getCause() + "\n" + e.getStackTrace(), "Debug-Message",
-											JOptionPane.ERROR_MESSAGE);
-								}
-
-							} else {
-								LOGGER.info("Image '"
-										+ imageid
-										+ "' konnte nicht gelöscht werden, da mindestens eine Veranstaltung mit diesem Image verlinkt ist.");
-
-								JOptionPane
-										.showMessageDialog(
-												null,
-												"Image kann nicht gelöscht werden, da mindestens eine Veranstaltung mit diesem Image verlinkt ist.",
-												"Verlinkung vorhanden", JOptionPane.INFORMATION_MESSAGE);
-							}
-						} catch (TException e) {
-
-							e.printStackTrace();
-							JOptionPane.showMessageDialog(null, e.getCause() + "\n" + e.getStackTrace(),
-									"Debug-Message", JOptionPane.ERROR_MESSAGE);
-						}
+				if (tablemyImages.getSelectedRowCount() != 1) {
+					JOptionPane.showMessageDialog(null, "Bitte wählen Sie ein Image aus", "Info",
+							JOptionPane.INFORMATION_MESSAGE);
+					return;
+				}
+
+				// frage, soll das selektierte Daten geloescht werden?
+				eingabe = JOptionPane.showConfirmDialog(null, "Wollen Sie diese Daten löschen?", "Löschen?",
+						JOptionPane.YES_NO_OPTION);
+
+				if (eingabe != JOptionPane.YES_OPTION) {
+					return;
+				}
+
+				imageid = modelMyImages.getValueAt(
+						tablemyImages.convertRowIndexToModel(tablemyImages.getSelectedRow()), 6).toString();
+
+				imageversion = modelMyImages.getValueAt(
+						tablemyImages.convertRowIndexToModel(tablemyImages.getSelectedRow()), 7).toString();
+
+				// Image darf nur geloescht werden, wenn keine
+				// Veranstaltung mehr auf das Image verweist.
+				// Unbdingt abfragen!
+				// TODO: Check this server side on delete, then using exceptions...
+				try {
+					if (ThriftManager.getSatClient().connectedToLecture(imageid, imageversion,
+							SessionData.authToken)) {
+						LOGGER.info("Image '" + imageid
+								+ "' konnte nicht gelöscht werden, da mindestens eine Veranstaltung mit"
+								+ " diesem Image verlinkt ist.");
+
+						JOptionPane.showMessageDialog(null,
+								"Image kann nicht gelöscht werden, da mindestens eine Veranstaltung mit"
+										+ " diesem Image verlinkt ist.", "Verlinkung vorhanden",
+								JOptionPane.INFORMATION_MESSAGE);
+						return;
 					}
+				} catch (Exception e) {
+				}
+
+				try {
 
+					//  try to delete file from file system
+					if (ThriftManager.getSatClient()
+							.deleteImage(imageid, imageversion, SessionData.authToken) == true) {
+						LOGGER.info("Image '" + imageid + "' erfolgreich gelöscht.");
+
+						GuiManager.show(new DeleteImage_GUI());
+					} else {
+						// could not delete file from file
+						// system
+						LOGGER.info("Image '" + imageid + "' konnte nicht gelöscht werden.");
+
+						JOptionPane.showMessageDialog(null, "Image konnte nicht gelöscht werden.", "Fehler",
+								JOptionPane.INFORMATION_MESSAGE);
+					}
+
+				} catch (TException e) {
+
+					e.printStackTrace();
+					JOptionPane.showMessageDialog(null, e.getCause() + "\n" + e.getStackTrace(),
+							"Debug-Message", JOptionPane.ERROR_MESSAGE);
 				}
 
 			}
+
 		});
 		btnDelete.setBounds(449, 508, 118, 23);
 		contentPanel.add(btnDelete);