[client] Formatting/Wording/Layout of lecture details and netrules improved

1 files changed, 17 insertions, 13 deletions

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetrulesConfigurator.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetrulesConfigurator.java
index d471349f..d98c4cd6 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetrulesConfigurator.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetrulesConfigurator.java
@@ -44,7 +44,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 	 * Example: "8.8.8.8 80 in" would be split in -hostname "8.8.8.8" -port "80"
 	 * -direction "in"
 	 */
-	private static final String FIELD_DELIMITER = "\\s";
+	private static final String FIELD_DELIMITER = "\\s+";
 
 	public NetrulesConfigurator() {
 		super();
@@ -113,8 +113,8 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 						+ ":" + currentRule.port);
 				continue;
 			}
-			currentLine += currentRule.host + " ";
-			currentLine += currentRule.port + " ";
+			currentLine += currentRule.host + " \t ";
+			currentLine += currentRule.port + " \t ";
 			currentLine += currentRule.direction.name();
 			decodedRules += currentLine
 					+ (it.hasNext() ? System.lineSeparator() : "");
@@ -160,7 +160,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 						+ fields.length);
 				Gui.showMessageBox(
 						"Ungültige Syntax: Bitte definieren Sie Ihre Regel im Format: <host> <port> [in|out|both]",
-						MessageType.ERROR, LOGGER, null);
+						MessageType.ERROR, null, null);
 				break;
 			}
 
@@ -175,7 +175,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 						+ ruleDirection);
 				Gui.showMessageBox(
 						"Ungültige Richtung: Bitte nutzen Sie 'in' bzw. 'out'.",
-						MessageType.ERROR, LOGGER, null);
+						MessageType.ERROR, null, null);
 				break;
 			}
 			// check port: accept if > -2 and < 65535
@@ -191,8 +191,8 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 				markText(ruleLine, Color.RED);
 				LOGGER.debug("Invalid port! Got: " + port);
 				Gui.showMessageBox(
-						"Ungültiges Port! Bitte nutzen Sie einen Port aus dem Bereich [0-65536].",
-						MessageType.ERROR, LOGGER, null);
+						"Ungültiger Port! Bitte nutzen Sie einen Port aus dem Bereich [0-65536].",
+						MessageType.ERROR, null, null);
 				break;
 			}
 			// check hostname: bit more to do here
@@ -222,7 +222,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 							"Konnte '"
 									+ fields[0]
 									+ "' nicht auflösen. Wollen Sie diesen Hostnamen trotzdem verwenden?",
-							MessageType.WARNING_RETRY, LOGGER, null)) {
+							MessageType.WARNING_RETRY, null, null)) {
 						break;
 					}
 				} else {
@@ -262,7 +262,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 	private boolean checkHostnameSimple(final String hostname) {
 		if (hostname.length() > 254) {
 			Gui.showMessageBox("Hostname ist zu lang!", MessageType.ERROR,
-					LOGGER, null);
+					null, null);
 			return false;
 		}
 		// split by '.' to get domain levels
@@ -272,7 +272,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 				// fail since domain level should be max 63 chars
 				Gui.showMessageBox("Domain-Ebene '" + domainLabel
 						+ "' länger als 63 Zeichen!", MessageType.ERROR,
-						LOGGER, null);
+						null, null);
 				return false;
 			}
 			// length is ok, check for invalid characters
@@ -283,7 +283,7 @@ public class NetrulesConfigurator extends NetrulesConfiguratorLayout {
 				if (!(Character.isDigit(c) || Character.isLetter(c)
 						|| c.equals('-') || c.equals('/'))) {
 					Gui.showMessageBox("Ungültiges Zeichen '" + c
-							+ "' in hostname!", MessageType.ERROR, LOGGER, null);
+							+ "' in hostname!", MessageType.ERROR, null, null);
 					return false;
 				}
 			}
@@ -382,7 +382,11 @@ class NetrulesConfiguratorLayout extends JPanel {
 
 	private static final long serialVersionUID = 5266120380443817325L;
 	private final static String txtNetworkOptionsTitle = "Netzwerk Einstellungen";
-	private final static String txtNetworkOptionsDesc = "Wenn Sie den Internetzugriff deaktiviert haben, können Sie hier Ausnahmen definieren (Whitelist). Bitte definieren Sie Ihre Regeln im Format <host> <port> <in|out>.";
+	private final static String txtNetworkOptionsDesc = "Wenn Sie den Internetzugriff deaktiviert haben,"
+			+ " können Sie hier Ausnahmen definieren (Whitelist)."
+			+ " Bitte definieren Sie Ihre Regeln im Format\n<host> <port> <in|out>.\n"
+			+ "Sie können Port 0 angeben, was sämtlichen TCP und UDP Ports eines Hosts entspricht.\n\n"
+			+ "Wenn Sie Internetzugriff aktivieren, hat diese Liste den gegenteiligen Effekt";
 	private final static String txtNetworkRulesTitle = "Netzwerkregeln";
 
 	private final JPanel pnlNetworkOptions;
@@ -407,7 +411,7 @@ class NetrulesConfiguratorLayout extends JPanel {
 		pnlNetworkOptions.setBorder(BorderFactory
 				.createTitledBorder(txtNetworkRulesTitle));
 		gridNetworkOptions
-				.add(new WordWrapLabel(txtNetworkOptionsDesc, false, true))
+				.add(new WordWrapLabel(txtNetworkOptionsDesc))
 				.fill(true, false).expand(true, false);
 		gridNetworkOptions.nextRow();
 		gridNetworkOptions.add(scpNetworkRules).fill(true, true)