Fix: input type text -> number for player enter code

src/web/js/lib/account_validator.js

@@ -73,11 +73,11 @@ AccountValidator.prototype.isValidPlayerPW = function(textPW) {
 
 AccountValidator.prototype.messageForInvalidPlayerPW = function(textPW) {
 	if(textPW.length < Number(AccountValidator.PLAYER_PW_LENGTH_MINIMUM))
-		return "학생 입장 번호는 " + AccountValidator.PLAYER_PW_LENGTH_MINIMUM + "글자 이상의 숫자로만 입력해 주세요.";
+		return "학생 입장 번호는 " + AccountValidator.PLAYER_PW_LENGTH_MINIMUM + "자리 이상의 숫자로 입력해 주세요.";
 	else if(textPW.length > Number(AccountValidator.PLAYER_PW_LENGTH_MAXIMUM))
-		return "학생 입장 번호는 " + AccountValidator.PLAYER_PW_LENGTH_MAXIMUM + "글자 이하의 숫자로만 입력해 주세요.";
+		return "학생 입장 번호는 " + AccountValidator.PLAYER_PW_LENGTH_MAXIMUM + "자리 이하의 숫자로 입력해 주세요.";
 	else
-		return "학생 입장 번호는 2~6자리 숫자로 입력해 주세요.";
+		return "학생 입장 번호는 2~8자리 숫자로 입력해 주세요.";
 }
 
 

src/web/js/player_list_manager.js

@@ -149,11 +149,26 @@ PlayerListManager.prototype.editPlayer = function(inputButton) {
 
 	var name = $(inputButton).siblings(".player_name");
 	var playerName = name.val();
-	// console.log(playerName);
+	console.log(playerName);
 
 	var enterCode = $(inputButton).siblings(".player_entercode");
 	var playerEnterCode = enterCode.val();
-	// console.log(playerEnterCode);
+	console.log(playerEnterCode);
+
+	if(!accountValidator.isValidPlayerID(playerName)) {
+		// console.log("Error - playerName : " + playerName);
+		onErrorPlayerName(accountValidator.messageForInvalidPlayerID(playerName));
+		return;
+	} else if(!accountValidator.isValidPlayerPW(playerEnterCode)) {
+		// console.log("Error - enterCode : " + enterCode);
+		// onErrorPlayerPW(accountValidator.messageForInvalidPlayerPW(playerEnterCode));
+		showErrorMessage(accountValidator.messageForInvalidPlayerPW(playerEnterCode), "error");
+
+		$(enterCode).focus();
+		$(enterCode).select();
+
+		return;
+	}
 
 	sendRequestServer(	//url, sendParams, onSuccess, onFail, isDebugMode);
 		"./../server/player/edit_player.php",

src/web/module/maestro_section_add_player.html

@@ -65,14 +65,14 @@ function addPlayer() {
 function onErrorPlayerName(message) {
 	showErrorMessage(message, "error");
 
-	$("#player_name").val("");
+	$("#player_name").select();
 	$("#player_name").focus();
 }
 
 function onErrorPlayerPW(message) {
 	showErrorMessage(message, "error");
 
-	$("#entercode").val("");
+	$("#entercode").select();
 	$("#entercode").focus();
 }
 
@@ -98,7 +98,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group row">
 				<label for="entercode" class="col-sm-3 col-form-label">입장 번호</label>
 				<div class="col-sm-9">
-					<input type="text" class="form-control col-sm-9" id="entercode" placeholder="입장 번호 입력">
+					<input type="number" class="form-control col-sm-9" id="entercode" min="10" step="1" maxlength="8" placeholder="입장 번호 입력">
 					<small class="form-text text-muted">2~8자리의 숫자</small>
 					<button type="button" class="col-sm-7 btn btn-primary btn-lg mt-3" onClick="addPlayer()">추가</button>
 				</div>
@@ -158,7 +158,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test1">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -166,7 +166,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test2">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -174,7 +174,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test3">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -182,7 +182,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test4">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -190,7 +190,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test5">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -198,7 +198,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test6">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -206,7 +206,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test7">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -214,7 +214,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test8">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -222,7 +222,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test9">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -230,7 +230,7 @@ function onErrorPlayerPW(message) {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test10">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="✎">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>

src/web/module/maestro_section_search.html

@@ -75,7 +75,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test1">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -83,7 +83,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test2">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -91,7 +91,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test3">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -99,7 +99,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test4">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -107,7 +107,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test5">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -115,7 +115,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test6">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -123,7 +123,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test7">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -131,7 +131,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test8">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -139,7 +139,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test9">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>
@@ -147,7 +147,7 @@ function searchPlayer() {
 			<div class="form-group player-list row mx-2 mb-1">
 				<input type="text" class="player_id d-none" value="test10">
 				<input type="text" class="player_name col-sm" value="test">
-				<input type="text" class="player_entercode col-sm" value="test">
+				<input type="number" class="player_entercode col-sm" min="10" step="1" maxlength="8" value="test">
 				<input type="button" class="player_edit col-sm-1" value="+">
 				<input type="button" class="player_delete col-sm-1" value="-">
 			</div>