Merge pull request #6374 from surveyjs/features/6373-multipletextitem…

…-name Make multipletextitem name property requried and unique fix #6373
surveyjs · Jun 19, 2023 · eb70a7b · eb70a7b
2 parents 3be49f0 + c609340
commit eb70a7b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/question_multipletext.ts b/src/question_multipletext.ts
@@ -661,7 +661,7 @@ export class QuestionMultipleTextModel extends Question
 Serializer.addClass(
   "multipletextitem",
   [
-    "name",
+    { name: "!name", isUnique: true },
     "isRequired:boolean",
     { name: "placeholder", alternativeName: "placeHolder", serializationProperty: "locPlaceholder" },
     {

diff --git a/tests/jsonobjecttests.ts b/tests/jsonobjecttests.ts
@@ -2981,4 +2981,9 @@ QUnit.test("overridingProperty test", function (assert) {
   assert.equal(Serializer.findProperty("question", "visible").overridingProperty, "visibleIf", "visible property check");
   assert.equal(Serializer.findProperty("question", "readOnly").overridingProperty, "enableIf", "readOnly property check");
   assert.equal(Serializer.findProperty("question", "isRequired").overridingProperty, "requiredIf", "isRequired property check");
+});
+QUnit.test("multipletextitem, name property should be required and unique", function (assert) {
+  const prop = Serializer.findProperty("multipletextitem", "name");
+  assert.equal(prop.isRequired, true, "name property is required");
+  assert.equal(prop.isUnique, true, "name property is unique");
 });