update

add textField red borders in case of validation error

Author: mikyll

bin/.gitignore

@@ -1 +1,3 @@
 /view/
+/networking/
+/controller/

bin/controller/Controller.class

@@ -330,12 +330,15 @@ public void initialize()
 		{
 			this.buttonCNR.setDisable(false);
 			this.labelErrorNicknameS.setVisible(false);
+			this.buttonCNR.setDisable(true);
+			this.textFieldNicknameS.setStyle("-fx-border-width: 0px; -fx-focus-color: #039ED3;");
 		}
 		// nickname NOT
 		else
 		{
 			this.buttonCNR.setDisable(true);
 			this.labelErrorNicknameS.setVisible(true);
+			this.textFieldNicknameS.setStyle("-fx-text-box-border: red; -fx-focus-color: red;");
 		}
 	}
 	@FXML public void increaseMinRoom(MouseEvent event)
@@ -456,27 +459,44 @@ public void initialize()
 			this.buttonJER.setDisable(false);
 			this.labelErrorNicknameC.setVisible(false);
 			this.labelErrorIP.setVisible(false);
+			// reset borders & focus (nickname)
+			this.textFieldNicknameC.setStyle("-fx-border-width: 0px; -fx-focus-color: #039ED3;");
+			// reset borders & focus (address)
+			this.textFieldIP.setStyle("-fx-border-width: 0px; -fx-focus-color: #039ED3;");			
 		}
 		// nickname OK & address NOT (nor empty)
 		else if(this.checkNickname(this.textFieldNicknameC.getText()) && !(checkIP(this.textFieldIP.getText()) || this.textFieldIP.getText().isEmpty()))
 		{
 			this.buttonJER.setDisable(true);
 			this.labelErrorNicknameC.setVisible(false);
 			this.labelErrorIP.setVisible(true);
+			// reset borders & focus (nickname)
+			this.textFieldNicknameC.setStyle("-fx-border-width: 0px; -fx-focus-color: #039ED3;");
+			// reset borders & focus (address)
+			this.textFieldIP.setStyle("-fx-text-box-border: red; -fx-focus-color: red;");		
 		}
 		// nickname NOT & address OK (or empty)
 		else if(!this.checkNickname(this.textFieldNicknameC.getText()) && (checkIP(this.textFieldIP.getText()) || this.textFieldIP.getText().isEmpty()))
 		{
 			this.buttonJER.setDisable(true);
 			this.labelErrorNicknameC.setVisible(true);
 			this.labelErrorIP.setVisible(false);
+			// red borders & focus (nickname)
+			this.textFieldNicknameC.setStyle("-fx-text-box-border: red; -fx-focus-color: red;");
+			// reset borders & focus (address)
+			this.textFieldIP.setStyle("-fx-border-width: 0px; -fx-focus-color: #039ED3;");
 		}
 		// nickname NOT & address NOT (nor empty)
 		else
 		{
 			this.buttonJER.setDisable(true);
 			this.labelErrorNicknameC.setVisible(true);
 			this.labelErrorIP.setVisible(true);
+			// red borders & focus (nickname)
+			this.textFieldNicknameC.setStyle("-fx-text-box-border: red; -fx-focus-color: red;");
+			// red borders & focus (address)
+			this.textFieldIP.setStyle("-fx-text-box-border: red; -fx-focus-color: red;");
+					
 		}
 	}
 	@FXML public void joinExistingRoom(ActionEvent event) 
@@ -690,12 +710,12 @@ else if(!this.checkNickname(this.textFieldNicknameC.getText()) && (checkIP(this.
 	}
 
 	// utilities
-	private boolean checkNickname(String text)
+	public boolean checkNickname(String text)
 	{
 		// if OK return true
 		return PATTERN_NICKNAME.matcher(text).matches() ? true : false;
 	}
-	private boolean checkIP(String text)
+	public boolean checkIP(String text)
 	{
 		// if OK return true
 		return PATTERN_IP.matcher(text).matches() ? true : false;

bin/networking/ServerStream.class

@@ -4,8 +4,9 @@
 
 public interface IServer {
 	public void sendChatMessage(String content);
-	public void sendKickUser(String nickname);
+	public void sendKickUser(String kickNickname);
 	public User sendBanUser(String banNickname);
+	//public boolean sendBanUser(String banNickname, String banAddress);
 	public boolean removeBan(String address);
 	public boolean checkCanStartGame();
 	public void sendClose();

src/controller/Controller.java

@@ -390,6 +390,23 @@ public User sendBanUser(String banNickname)
 		return result;
 	}
 
+	/*@Override
+	public boolean sendBanUser(String banNickname, String banAddress)
+	{
+		if(!this.controller.checkNickname(banNickname) || !this.controller.checkIP(banAddress))
+			return false;
+		
+		for(User u : this.bannedUsers)
+		{
+			if(banNickname.equals(u.getNickname()) && banAddress.equals(u.getAddress().getHostAddress()))
+		}
+		if(!this.bannedUsers.contains(this.bannedUsers.contains()))
+		{
+			return true;
+		}
+		return false;
+	}*/
+	
 	@Override
 	public boolean removeBan(String address)
 	{