From 80235d0dc4abadf82fb10ce8aaa0ef239741b441 Mon Sep 17 00:00:00 2001
From: Jonathan Bauer
Date: Wed, 16 Sep 2015 16:00:10 +0200
Subject: [client] ImageUpdateWiz: if updating version metadata failed, inform
 user

---
 .../main/java/org/openslx/dozmod/gui/wizard/ImageUpdateWizard.java  | 6 +++++-
 .../src/main/java/org/openslx/dozmod/gui/wizard/LectureWizard.java  | 2 +-
 .../src/main/java/org/openslx/dozmod/util/ClientVersion.java        | 1 -
 3 files changed, 6 insertions(+), 3 deletions(-)

(limited to 'dozentenmodul/src/main/java/org/openslx')

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/ImageUpdateWizard.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/ImageUpdateWizard.java
index 44c502fa..54b3af39 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/ImageUpdateWizard.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/ImageUpdateWizard.java
@@ -7,6 +7,8 @@ import javax.swing.JOptionPane;
 import org.apache.log4j.Logger;
 import org.openslx.bwlp.thrift.iface.ImageDetailsRead;
 import org.openslx.bwlp.thrift.iface.ImageVersionWrite;
+import org.openslx.dozmod.gui.Gui;
+import org.openslx.dozmod.gui.helper.MessageType;
 import org.openslx.dozmod.gui.helper.UiFeedback;
 import org.openslx.dozmod.gui.wizard.page.ImageUploadPage;
 import org.openslx.dozmod.state.UploadWizardState;
@@ -28,10 +30,12 @@ public class ImageUpdateWizard extends Wizard implements UiFeedback {
 
 	@Override
 	public void performFinish() {
-		// TODO use ret?
 		boolean ret = ThriftActions.updateImageVersion(JOptionPane.getFrameForComponent(this),
 				uploadWizardState.transferInformation.getToken(),
 				new ImageVersionWrite(uploadWizardState.isRestricted));
+		if (!ret) {
+			Gui.showMessageBox(this, "Konnte Metadaten der Version nicht am Satelliten speichern!", MessageType.INFO, null, null);
+		}
 	}
 	@Override
 	public boolean wantConfirmQuit() {
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/LectureWizard.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/LectureWizard.java
index e8fdeb9a..e730c5b7 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/LectureWizard.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/LectureWizard.java
@@ -93,7 +93,7 @@ public class LectureWizard extends Wizard implements UiFeedback {
 		else
 			state.uuid = uuid;
 		// now push the permissions if we have any
-		if(state.permissionMap != null && !state.permissionMap.isEmpty())
+		if (state.permissionMap != null && !state.permissionMap.isEmpty())
 			if (!ThriftActions.writeLecturePermissions(JOptionPane.getFrameForComponent(this), state.uuid, state.permissionMap))
 				return false;
 
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/util/ClientVersion.java b/dozentenmodul/src/main/java/org/openslx/dozmod/util/ClientVersion.java
index a3fcee4b..a05d23da 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/util/ClientVersion.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/util/ClientVersion.java
@@ -16,7 +16,6 @@ import org.apache.log4j.Logger;
 import org.openslx.sat.thrift.version.Version;
 import org.openslx.util.Json;
 
-// ClientVersion is a bad name, TODO find better suited one :)
 public class ClientVersion {
 
 	private static final Logger LOGGER = Logger.getLogger(ClientVersion.class);
-- 
cgit v1.2.3-55-g7522