summaryrefslogtreecommitdiffstats
path: root/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
diff options
context:
space:
mode:
authorSimon Rettberg2015-08-11 17:50:41 +0200
committerSimon Rettberg2015-08-11 17:50:41 +0200
commit67d6892eaa5f004674c8641438d09ffc6baade18 (patch)
tree1d147fbf7dcd8c49fcf214fa5d3234374f241215 /dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
parent[server] Don't try to deserialize null (diff)
downloadtutor-module-67d6892eaa5f004674c8641438d09ffc6baade18.tar.gz
tutor-module-67d6892eaa5f004674c8641438d09ffc6baade18.tar.xz
tutor-module-67d6892eaa5f004674c8641438d09ffc6baade18.zip
[client] Show image name in lecture wizard; check form validity on "Next"
Diffstat (limited to 'dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java')
-rw-r--r--dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java18
1 files changed, 13 insertions, 5 deletions
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
index c5e124a2..3c9ccf56 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
@@ -17,6 +17,8 @@ import javax.swing.event.DocumentListener;
import org.apache.log4j.Logger;
import org.openslx.bwlp.thrift.iface.ImageDetailsRead;
+import org.openslx.bwlp.thrift.iface.ImageSummaryRead;
+import org.openslx.bwlp.thrift.iface.ImageVersionDetails;
import org.openslx.bwlp.thrift.iface.OperatingSystem;
import org.openslx.bwlp.thrift.iface.ShareMode;
import org.openslx.bwlp.thrift.iface.Virtualizer;
@@ -84,7 +86,12 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
if (e.getSource().equals(popupItemNew)) {
// TODO new lecture wizard with this image preset
LOGGER.debug("New lecture clicked");
- new LectureWizard(me, versionTable.getSelectedItem()).setVisible(true);
+ ImageVersionDetails selected = versionTable.getSelectedItem();
+ if (selected == null)
+ return;
+ ImageSummaryRead summary = new ImageSummaryRead();
+ summary.setImageName(image.getImageName()); // Maybe create a helper class/function some day that transforms all fields
+ new LectureWizard(me, summary, selected.getVersionId()).setVisible(true);
}
if (e.getSource().equals(popupItemDownload)) {
// TODO delete that image
@@ -149,7 +156,8 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
synchronized (me) {
if (image != null)
return;
- image = ThriftManager.getSatClient().getImageDetails(Session.getSatelliteToken(), imageBaseId);
+ image = ThriftManager.getSatClient().getImageDetails(Session.getSatelliteToken(),
+ imageBaseId);
}
} catch (Exception e) {
error = e;
@@ -162,8 +170,8 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
@Override
public void run() {
if (e != null || image == null) {
- Gui.showMessageBox(null, "Konnte Daten des Images nicht abrufen", MessageType.ERROR, LOGGER,
- e);
+ Gui.showMessageBox(null, "Konnte Daten des Images nicht abrufen",
+ MessageType.ERROR, LOGGER, e);
dispose();
} else {
fill();
@@ -306,7 +314,7 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
// Image name:
if (!txtTitle.getText().equals(image.getImageName())) {
LOGGER.debug("Title changed to: " + txtTitle.getText());
- changed =true;
+ changed = true;
}
// Image description:
if (!txtDescription.getText().equals(image.getDescription())) {