Merge branch 'master' of git.openslx.org:openslx-ng/tutor-module

11 files changed, 21 insertions, 31 deletions

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/MainWindow.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/MainWindow.java
index 8fd419bd..cc40bef6 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/MainWindow.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/MainWindow.java
@@ -560,7 +560,7 @@ public abstract class MainWindow {
 												+ "Start- und Endzeiten von Veranstaltungen kommen. Kontaktieren Sie in diesem\n"
 												+ "Fall den zuständigen Administrator, damit die Uhrzeit auf dem Satellitenserver\n"
 												+ "korrigiert werden kann.\n\n" + "Ihr Computer: "
-												+ FormatHelper.longDate(now) + "\nSatelliten-Server: "
+												+ FormatHelper.longDate(now) + "\nSatellitenserver: "
 												+ FormatHelper.longDate(status.serverTime),
 										MessageType.WARNING, LOGGER, null);
 							}
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/activity/PassiveUploadPanel.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/activity/PassiveUploadPanel.java
index c8a62736..64e6b994 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/activity/PassiveUploadPanel.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/activity/PassiveUploadPanel.java
@@ -16,7 +16,7 @@ public class PassiveUploadPanel extends TransferPanel {
 	private final PassiveUploadPanel panel = this;
 
 	public PassiveUploadPanel(TransferEventEmitter state, String name) {
-		super(state, name, "<Transfer zwischen Master-Server und Satelliten-Server>");
+		super(state, name, "<Transfer zwischen Masterserver und Satellitenserver>");
 		btnClose.addActionListener(new ButtonAction());
 		this.state = state;
 		state.addListener(this);
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 7008898c..e1ba414a 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
@@ -64,6 +64,7 @@ import org.openslx.dozmod.util.FormatHelper;
 import org.openslx.dozmod.util.MapHelper;
 import org.openslx.dozmod.util.OpenLinks;
 import org.openslx.sat.thrift.version.Feature;
+import org.openslx.thrifthelper.TConst;
 import org.openslx.thrifthelper.ThriftManager;
 import org.openslx.util.QuickTimer;
 import org.openslx.util.QuickTimer.Task;
@@ -395,7 +396,7 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
 						}
 
 						String tmp = ThriftUtil.byteBufferToString(machineDescription);
-						if ("qemukvm".equals(image.virtId)) { // TODO: Constants in master-sync-shared after merge
+						if (TConst.VIRT_QEMU.equals(image.virtId)) {
 							VirtConfigEditorWindow.open(me, actionHandler, selected.versionId,
 									machineDescription, tmp);
 						} else {
@@ -520,7 +521,7 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout implements UiFe
 						// Inform user
 						Gui.showMessageBox(
 								ImageDetailsWindow.this,
-								"Die Übertragung läuft direkt zwischen Satelliten-Server und"
+								"Die Übertragung läuft direkt zwischen Satellitenserver und"
 										+ " dem Zentral-Server in Freiburg.\n"
 										+ "Wenn Sie die bwLehrpool-Suite schließen, wird der Transfer trotzdem"
 										+ "weiterlaufen.", MessageType.INFO, null, null);
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImagePublishedWindow.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImagePublishedWindow.java
index 6bf7200b..0e108d0a 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImagePublishedWindow.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImagePublishedWindow.java
@@ -119,7 +119,7 @@ public class ImagePublishedWindow extends ImagePublishedWindowLayout implements
 				}
 				MainWindow.addPassiveTransfer(transferToken, item.imageName,
 						false);
-				Gui.showMessageBox(ImagePublishedWindow.this, "Die Übertragung läuft direkt zwischen Satelliten-Server und"
+				Gui.showMessageBox(ImagePublishedWindow.this, "Die Übertragung läuft direkt zwischen Satellitenserver und"
 						+ " dem Zentral-Server in Freiburg.\n"
 						+ "Wenn Sie die bwLehrpool-Suite schließen, wird der Transfer trotzdem"
 						+ "weiterlaufen.", MessageType.INFO, null, null);
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/LoginWindow.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/LoginWindow.java
index 5dc5b608..07b36328 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/LoginWindow.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/LoginWindow.java
@@ -370,7 +370,7 @@ public class LoginWindow extends LoginWindowLayout {
 					if (t == null || !t.getClass().equals(RuntimeException.class)) {
 						Gui.showMessageBox(me, "Internal error!", MessageType.ERROR, null, t);
 					} else {
-						Gui.showMessageBox(me, "Der Master-Server hat den Loginversuch mit der"
+						Gui.showMessageBox(me, "Der Masterserver hat den Loginversuch mit der"
 								+ " folgenden Nachricht abgewiesen:\n\n"
 								+ t.getMessage(), MessageType.ERROR, null, null);
 					}
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/ConfigWindowLayout.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/ConfigWindowLayout.java
index ab5a9b41..f2f8bfbe 100755
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/ConfigWindowLayout.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/ConfigWindowLayout.java
@@ -141,7 +141,7 @@ public class ConfigWindowLayout extends JDialog {
 						+ " wenn die Einstellung auf 1 belassen wird. Falls die Übertragungsgeschwindigkeit"
 						+ " ihre Netzwerkanbindung nicht auslastet, probieren Sie den nächsthöheren"
 						+ " Wert. Zu hohe Werte können einen negativen Effekt auf die"
-						+ " Übertragungsgeschwindigkeit haben, und belasten den Satelliten-Server stärker.",
+						+ " Übertragungsgeschwindigkeit haben, und belasten den Satellitenserver stärker.",
 						false, true))
 				.fill(true, false)
 				.expand(true, false);
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/SatelliteListWindowLayout.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/SatelliteListWindowLayout.java
index bf6d1bc5..2e28a4b9 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/SatelliteListWindowLayout.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/layout/SatelliteListWindowLayout.java
@@ -29,7 +29,7 @@ public class SatelliteListWindowLayout extends JDialog {
 	protected final JRadioButton rdoCusomAddress;
 	protected final Map<JRadioButton, Satellite> radioToSat = new HashMap<>();
 
-	private static String title = "Satelliten-Server wählen";
+	private static String title = "Satellitenserver wählen";
 
 	protected SatelliteListWindowLayout(Window modalParent, List<Satellite> satList) {
 		super(modalParent, title, modalParent != null ? ModalityType.APPLICATION_MODAL
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadPage.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadPage.java
index 192ab82e..2a798f27 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadPage.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadPage.java
@@ -30,6 +30,7 @@ import org.openslx.dozmod.thrift.UploadInitiator;
 import org.openslx.dozmod.thrift.WrappedException;
 import org.openslx.dozmod.thrift.cache.MetaDataCache;
 import org.openslx.sat.thrift.version.Feature;
+import org.openslx.thrifthelper.TConst;
 import org.openslx.util.vm.DiskImage;
 import org.openslx.util.vm.DiskImage.UnknownImageFormatException;
 import org.openslx.util.vm.VboxMetaData;
@@ -106,9 +107,9 @@ public class ImageUploadPage extends ImageUploadPageLayout {
 		fc.setFileFilter(allSupportedFilter);
 		// differentiate between existing and new VMs
 		if (existingImage != null) {
-			if (existingImage.virtId.equals("vmware")) {
+			if (existingImage.virtId.equals(TConst.VIRT_VMWARE)) {
 				fc.setFileFilter(vmxFilter);
-			} else if (existingImage.virtId.equals("virtualbox")) {
+			} else if (existingImage.virtId.equals(TConst.VIRT_VIRTUALBOX)) {
 				fc.setFileFilter(vboxFilter);
 			}
 		}
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java b/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java
index 7873b8ca..0b0d56a7 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java
@@ -94,7 +94,7 @@ public class ThriftActions {
 		long remoteVersion = -1;
 
 		if (interactive && !forceCustomSatellite && (data.satellites == null || data.satellites.isEmpty())) {
-			Gui.asyncMessageBox("Login erfolgreich, aber es wurde kein Satelliten-Server gefunden.\n"
+			Gui.asyncMessageBox("Login erfolgreich, aber es wurde kein Satellitenserver gefunden.\n"
 					+ " Bitte geben Sie die Adresse Ihres Servers manuell an.", MessageType.ERROR, LOGGER,
 					null);
 		}
@@ -147,7 +147,7 @@ public class ThriftActions {
 				}
 				if (sat.addressList == null || sat.addressList.isEmpty()) {
 					Gui.asyncMessageBox(
-							"Login erfolgreich, aber für den ausgewählten Satelliten-Server ist\n"
+							"Login erfolgreich, aber für den ausgewählten Satellitenserver ist\n"
 									+ "keine Adresse hinterlegt. Kann nicht verbinden.", MessageType.ERROR,
 							LOGGER, null);
 					continue;
@@ -176,7 +176,7 @@ public class ThriftActions {
 			if (client == null || remoteVersion == -1) {
 				if (interactive) {
 					Gui.asyncMessageBox(
-							"Authentifizierung erfolgreich, die Verbindung zum Satelliten-Server ist jedoch nicht möglich.\n\n"
+							"Authentifizierung erfolgreich, die Verbindung zum Satellitenserver ist jedoch nicht möglich.\n\n"
 									+ "Möglicherweise ist der Server nicht verfügbar, oder die Netzwerkverbindung gestört.",
 							MessageType.ERROR, null, null);
 					if (data.satellites.size() == 1) {
@@ -190,7 +190,7 @@ public class ThriftActions {
 			if (remoteVersion < Version.MIN_VERSION || remoteVersion > Version.VERSION) {
 				if (interactive) {
 					Gui.asyncMessageBox("Das von Ihnen verwendete Dozentenmodul ist nicht mit dem"
-							+ " gewählten Satelliten-Server kompatibel.\n" + "Ihre Version: "
+							+ " gewählten Satellitenserver kompatibel.\n" + "Ihre Version: "
 							+ Version.VERSION + "\n" + "Satelliten-Version: " + remoteVersion,
 							MessageType.ERROR, LOGGER, null);
 					continue;
@@ -204,7 +204,7 @@ public class ThriftActions {
 			} catch (TAuthorizationException e) {
 				if (interactive) {
 					ThriftError.showMessage(window, LOGGER, e,
-							"Authentifizierung erfolgreich, der Satelliten-Server verweigert jedoch die Verbindung.\n"
+							"Authentifizierung erfolgreich, der Satellitenserver verweigert jedoch die Verbindung.\n"
 									+ "Versuchen Sie, sich erneut anzumelden.\n");
 				}
 				return false;
@@ -212,13 +212,13 @@ public class ThriftActions {
 				if (interactive) {
 					ThriftError.showMessage(window, LOGGER, e,
 							"Authentifizierung erfolgreich, bei der Kommunikation mit"
-									+ " dem Satelliten-Server trat jedoch ein interner Fehler auf.");
+									+ " dem Satellitenserver trat jedoch ein interner Fehler auf.");
 					continue;
 				}
 				return false;
 			} catch (Exception e) {
 				if (interactive) {
-					Gui.asyncMessageBox("Unbekannter Fehler beim Verbinden mit dem Satelliten-Server.",
+					Gui.asyncMessageBox("Unbekannter Fehler beim Verbinden mit dem Satellitenserver.",
 							MessageType.ERROR, LOGGER, e);
 					continue;
 				}
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/util/MapHelper.java b/dozentenmodul/src/main/java/org/openslx/dozmod/util/MapHelper.java
index 487a9ac6..0e92bc6d 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/util/MapHelper.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/util/MapHelper.java
@@ -1,9 +1,7 @@
 package org.openslx.dozmod.util;
 
-import java.util.HashSet;
 import java.util.Map;
 import java.util.Map.Entry;
-import java.util.Set;
 
 import org.apache.log4j.Logger;
 
@@ -34,15 +32,4 @@ public class MapHelper {
 		return false;
 	}
 
-	private static <T> Set<String> getAddedUsers(final Map<String, T> oldMap, final Map<String, T> newMap) {
-		Set<String> addedUsers = new HashSet<String>(newMap.keySet());
-		addedUsers.removeAll(oldMap.keySet());
-		return addedUsers;
-	}
-
-	private static <T> Set<String> getRemovedUsers(final Map<String, T> oldMap, final Map<String, T> newMap) {
-		Set<String> removedUsers = new HashSet<String>(oldMap.keySet());
-		removedUsers.removeAll(newMap.keySet());
-		return removedUsers;
-	}
 }
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/util/VmWrapper.java b/dozentenmodul/src/main/java/org/openslx/dozmod/util/VmWrapper.java
index d37fc5b5..e58653e7 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/util/VmWrapper.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/util/VmWrapper.java
@@ -23,6 +23,7 @@ import org.openslx.bwlp.thrift.iface.OperatingSystem;
 import org.openslx.dozmod.gui.Gui;
 import org.openslx.dozmod.gui.helper.MessageType;
 import org.openslx.dozmod.thrift.cache.MetaDataCache;
+import org.openslx.thrifthelper.TConst;
 import org.openslx.util.vm.DiskImage;
 import org.openslx.util.vm.QemuMetaData;
 import org.openslx.util.vm.VboxMetaData;
@@ -40,7 +41,7 @@ public class VmWrapper {
 			throw new FileNotFoundException("Disk file " + diskFile.getAbsolutePath() + " does not exist");
 
 		if (machineDescription == null || machineDescription.length < 10) {
-			if (!"vmware".equals(virtualizerId)) {
+			if (!TConst.VIRT_VMWARE.equals(virtualizerId)) {
 				throw new MetaDataMissingException();
 			}
 			machineDescription = getFallbackVmx(diskImageInfo);