[client] clear input fields after adding share

Closes #3359

1 files changed, 23 insertions, 19 deletions

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetshareConfigurator.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetshareConfigurator.java
index 7f6a2c96..365ab03f 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetshareConfigurator.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/control/NetshareConfigurator.java
@@ -71,26 +71,19 @@ public class NetshareConfigurator extends NetshareConfiguratorLayout {
 				NetShare item = tblNetshare.getSelectedItem();
 				// ugly block-wise sets, but only one test needed compared to
 				// doing lots of item != null ? ... : ...
-				if (item != null) {
-					// share from the list is selected: fill bottom form and change "Add" to "Apply"
-					btnDel.setEnabled(true);
-					tfSharePath.setText(item.path);
-					tfShareName.setText(item.displayname);
-					tfUsername.setText(item.username);
-					tfPassword.setText(item.password);
-					cboNetshareAuth.setSelectedItem(item.auth);
-					cboNetshareMountPoint.setSelectedItem(Character.valueOf(item.mountpoint.charAt(0)));
-					btnAdd.setText("Ändern");
-				} else {
-					btnDel.setEnabled(false);
-					tfSharePath.setText(null);
-					tfShareName.setText(null);
-					tfUsername.setText(null);
-					tfPassword.setText(null);
-					cboNetshareAuth.setSelectedItem(null);
-					cboNetshareMountPoint.setSelectedItem(null);
-					btnAdd.setText("Hinzufügen");
+				if (item == null) {
+					clearInputFields();
+					return;
 				}
+				// share from the list is selected: fill bottom form and change "Add" to "Apply"
+				btnDel.setEnabled(true);
+				tfSharePath.setText(item.path);
+				tfShareName.setText(item.displayname);
+				tfUsername.setText(item.username);
+				tfPassword.setText(item.password);
+				cboNetshareAuth.setSelectedItem(item.auth);
+				cboNetshareMountPoint.setSelectedItem(Character.valueOf(item.mountpoint.charAt(0)));
+				btnAdd.setText("Ändern");
 			}
 		});
 
@@ -207,6 +200,7 @@ public class NetshareConfigurator extends NetshareConfiguratorLayout {
 				lblError.setText(null);
 				tblNetshareData.add(input);
 				tblNetshare.getModel().setData(tblNetshareData);
+				clearInputFields();
 			}
 		});
 
@@ -250,6 +244,16 @@ public class NetshareConfigurator extends NetshareConfiguratorLayout {
 		lblPassword.setEnabled(false);
 	}
 
+	private void clearInputFields() {
+		btnDel.setEnabled(false);
+		tfSharePath.setText(null);
+		tfShareName.setText(null);
+		tfUsername.setText(null);
+		tfPassword.setText(null);
+		cboNetshareAuth.setSelectedItem(null);
+		cboNetshareMountPoint.setSelectedItem(null);
+		btnAdd.setText("Hinzufügen");
+	}
 	public List<NetShare> getState() {
 		return tblNetshareData;
 	}