Features/6679 allow enter empty space (#6687)

* Add functional test on entering empty space #6679 * Remove test.only #6679 * Rename the functional test #6679
surveyjs · Aug 10, 2023 · 31b1a7e · 31b1a7e
1 parent e3964b0
commit 31b1a7e
Show file tree

Hide file tree

Showing 10 changed files with 34 additions and 10 deletions.
diff --git a/packages/survey-angular-ui/src/components/dropdown/dropdown.component.ts b/packages/survey-angular-ui/src/components/dropdown/dropdown.component.ts
@@ -46,7 +46,7 @@ export class DropdownComponent extends BaseAngular implements OnInit {
       if (!!this.inputElementRef?.nativeElement) {
         const control: any = this.inputElementRef.nativeElement;
         const newValue = this.model.inputStringRendered;
-        if (!Helpers.isTwoValueEquals(newValue, control.value, false, true)) {
+        if (!Helpers.isTwoValueEquals(newValue, control.value, false, true, false)) {
           control.value = this.model.inputStringRendered || "";
         }
       }

diff --git a/packages/survey-vue3-ui/src/components/dropdown/Dropdown.vue b/packages/survey-vue3-ui/src/components/dropdown/Dropdown.vue
@@ -144,7 +144,9 @@ const updateInputDomElement = () => {
   if (inputElement.value) {
     const control: any = inputElement.value;
     const newValue = model.value.inputStringRendered;
-    if (!Helpers.isTwoValueEquals(newValue, control.value)) {
+    if (
+      !Helpers.isTwoValueEquals(newValue, control.value, false, true, false)
+    ) {
       control.value = model.value.inputStringRendered;
     }
   }

diff --git a/src/calculatedValue.ts b/src/calculatedValue.ts
@@ -145,7 +145,7 @@ export class CalculatedValue extends Base {
     if (!!this.expressionRunner) return;
     this.expressionRunner = new ExpressionRunner(this.expression);
     this.expressionRunner.onRunComplete = newValue => {
-      if (!Helpers.isTwoValueEquals(newValue, this.value, false, true)) {
+      if (!Helpers.isTwoValueEquals(newValue, this.value, false, true, false)) {
         this.setValue(newValue);
       }
       this.unlocCalculation();

diff --git a/src/question_file.ts b/src/question_file.ts
@@ -286,7 +286,7 @@ export class QuestionFileModel extends Question {
         if (status === "success") {
           var oldValue = this.value;
           if (Array.isArray(oldValue)) {
-            this.value = oldValue.filter((f) => !Helpers.isTwoValueEquals(f, content, true));
+            this.value = oldValue.filter((f) => !Helpers.isTwoValueEquals(f, content, true, false, false));
           } else {
             this.value = undefined;
           }

diff --git a/src/question_paneldynamic.ts b/src/question_paneldynamic.ts
@@ -152,7 +152,7 @@ export class QuestionPanelDynamicItem implements ISurveyData, ISurveyImpl {
   public setValue(name: string, newValue: any) {
     const oldItemData = this.data.getPanelItemData(this);
     const oldValue = !!oldItemData ? oldItemData[name] : undefined;
-    if (Helpers.isTwoValueEquals(newValue, oldValue, false, true)) return;
+    if (Helpers.isTwoValueEquals(newValue, oldValue, false, true, false)) return;
     this.data.setPanelItemData(this, name, Helpers.getUnbindValue(newValue));
     const questions = this.panel.questions;
     for (var i = 0; i < questions.length; i++) {

diff --git a/src/react/dropdown-base.tsx b/src/react/dropdown-base.tsx
@@ -193,7 +193,7 @@ export class SurveyQuestionDropdownBase<T extends Question> extends SurveyQuesti
     if (!!this.inputElement) {
       const control: any = this.inputElement;
       const newValue = this.question.dropdownListModel.inputStringRendered;
-      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true)) {
+      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true, false)) {
         control.value = this.question.dropdownListModel.inputStringRendered;
       }
     }

diff --git a/src/react/reactquestion_element.tsx b/src/react/reactquestion_element.tsx
@@ -282,7 +282,7 @@ export class SurveyQuestionUncontrolledElement<
   }
   updateValueOnEvent = (event: any) => {
     if (
-      !Helpers.isTwoValueEquals(this.questionBase.value, event.target.value, false, true)
+      !Helpers.isTwoValueEquals(this.questionBase.value, event.target.value, false, true, false)
     ) {
       this.setValueCore(event.target.value);
     }
@@ -297,7 +297,7 @@ export class SurveyQuestionUncontrolledElement<
     if (!!this.control) {
       const control: any = this.control;
       const newValue = this.getValueCore();
-      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true)) {
+      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true, false)) {
         control.value = this.getValue(newValue);
       }
     }

diff --git a/src/react/tagbox-filter.tsx b/src/react/tagbox-filter.tsx
@@ -29,7 +29,7 @@ export class TagboxFilterString extends SurveyElementBase<ITagboxFilterProps, an
     if (!!this.inputElement) {
       const control: any = this.inputElement;
       const newValue = this.model.inputStringRendered;
-      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true)) {
+      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true, false)) {
         control.value = this.model.inputStringRendered;
       }
     }

diff --git a/src/vue/components/dropdown/dropdown.vue b/src/vue/components/dropdown/dropdown.vue
@@ -152,7 +152,7 @@ export class DropdownComponent extends BaseVue {
     if (!!this.inputElement) {
       const control: any = this.inputElement;
       const newValue = this.model.inputStringRendered;
-      if (!Helpers.isTwoValueEquals(newValue, control.value)) {
+      if (!Helpers.isTwoValueEquals(newValue, control.value, false, true, false)) {
         control.value = this.model.inputStringRendered;
       }
     }

diff --git a/testCafe/questions/text.js b/testCafe/questions/text.js
@@ -219,4 +219,26 @@ frameworks.forEach((framework) => {
       .pressKey("backspace")
       .expect(characterCounter.textContent).eql("0/10");
   });
+  test("Allow Space As Answer", async (t) => {
+    await initSurvey(framework, {
+      questions: [
+        {
+          name: "name",
+          type: "text",
+          maxLength: 10,
+        }]
+    });
+    await ClientFunction(() => {
+      window.survey.getQuestionByName("name").allowSpaceAsAnswer = true;
+    })();
+
+    await t
+      .pressKey("space")
+      .click("input[value=Complete]");
+
+    const surveyResult = await getSurveyResult();
+    await t.expect(surveyResult).eql({
+      name: " ",
+    });
+  });
 });