[5.4] MSPB-366: Validate if extesion already exist on users managemen…

…t section
2600hz · Jun 11, 2024 · b18d940 · b18d940
1 parent 4665cc1
commit b18d940
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 4 deletions.
diff --git a/i18n/en-US.json b/i18n/en-US.json
@@ -481,6 +481,7 @@
 		"add": "Add User",
 		"addExtension": "Add an extension to this user",
 		"addNewExtension": "Extension number: ",
+		"extensionAlreadyExist": "Already exists",
 		"addFromSpare": "Add from Spare Numbers",
 		"addFromSpareDevices": "Add from Spare Devices",
 		"buy": "Buy Numbers",

diff --git a/submodules/users/users.js b/submodules/users/users.js
@@ -833,6 +833,21 @@ define(function(require) {
 				listExtensions.append(newLineTemplate);
 
 				existingExtensions.push(nextExtension);
+
+				newLineTemplate.find('.extesion-already-exist').hide();
+
+				newLineTemplate.find('.input-extension ').on('keyup', function() {
+					var val = $(this).val();
+
+					if (existingExtensions.indexOf(val) >= 0) {
+						newLineTemplate.find('.extesion-already-exist').show();
+						template.find('.save-extensions').attr('disabled', true)
+						return;
+					}
+
+					newLineTemplate.find('.extesion-already-exist').hide();
+					template.find('.save-extensions').attr('disabled', false)
+				});
 			});
 
 			template.on('click', '.remove-extension', function() {
@@ -854,6 +869,7 @@ define(function(require) {
 
 				if (index > -1) {
 					existingExtensions.splice(index, 1);
+					template.find('.save-extensions').attr('disabled', false)
 				}
 
 				$(this).parents('.item-row').remove();
@@ -4896,7 +4912,7 @@ define(function(require) {
 					}, function(devices) {
 						callback(null, devices);
 					});
-				}
+				},
 			}, function(err, results) {
 				callback && callback(results);
 			});

diff --git a/submodules/users/users.scss b/submodules/users/users.scss
@@ -456,6 +456,15 @@
 	line-height: 32px;
 }
 
+#users_container .list-wrapper .item-row .extesion-already-exist {
+	margin: 0;
+	padding: 0;
+	font-size: 12px;
+	font-style: italic;
+	color: #ee1212;
+	font-weight: 600;
+}
+
 #users_container .list-wrapper .empty-search-row,
 #users_container .list-wrapper .empty-row {
 	display: none;

diff --git a/submodules/users/views/newExtension.html b/submodules/users/views/newExtension.html
@@ -1,7 +1,8 @@
 <div class="item-row larger">
     <div class="number">
-		{{ i18n.users.addNewExtension }}
-		<input type="text" class="input-extension" value="{{recommendedExtension}}"></input>
-		<a class="cancel-extension-link monster-link"><i class="monster-red fa fa-times"></i></a>
+			{{ i18n.users.addNewExtension }}
+			<input type="text" class="input-extension" value="{{recommendedExtension}}"></input>
+			<a class="cancel-extension-link monster-link"><i class="monster-red fa fa-times"></i></a>
     </div>
+		<p class="extesion-already-exist">{{ i18n.users.extensionAlreadyExist }}</p>
 </div>