Fixed issue: Proper error message when you save a question with wrong…

… code
LimeSurvey · Jan 9, 2020 · 897cdf8 · 897cdf8
1 parent 6fd8cee
commit 897cdf8
Show file tree

Hide file tree

Showing 7 changed files with 2,594 additions and 2,592 deletions.
diff --git a/assets/packages/questioneditor/build.min/css/main.css b/assets/packages/questioneditor/build.min/css/main.css
diff --git a/assets/packages/questioneditor/build.min/js/questionedit.js b/assets/packages/questioneditor/build.min/js/questionedit.js
diff --git a/assets/packages/questioneditor/build.min/js/questionedit.js.map b/assets/packages/questioneditor/build.min/js/questionedit.js.map
diff --git a/assets/packages/questioneditor/build/js/questionedit.js b/assets/packages/questioneditor/build/js/questionedit.js
diff --git a/assets/packages/questioneditor/build/js/questionedit.js.map b/assets/packages/questioneditor/build/js/questionedit.js.map
diff --git a/assets/packages/questioneditor/src/App.vue b/assets/packages/questioneditor/src/App.vue
@@ -388,7 +388,7 @@ export default {
                         this.loading = false;
                         this.$log.error(rejected);
                         if(rejected.data != undefined) {
-                            window.LS.notifyFader(rejected.data.message, 'well-lg bg-danger text-center', {timeout: 5500});
+                            window.LS.notifyFader(rejected.data.message, 'well-lg bg-danger text-center', undefined, {timeout: 5500});
                         }
                     }
                 )

diff --git a/assets/packages/questioneditor/src/storage/getters.js b/assets/packages/questioneditor/src/storage/getters.js
@@ -25,8 +25,9 @@ export default {
 
     hasTitleSet: (state) => {
         const isNotEmpty = !isEmpty(state.currentQuestion.title);
-        const startingWithALetter =  /[a-zA-Z]/.test(state.currentQuestion.title.substring(0,1));
-        return isNotEmpty && startingWithALetter;
+        const startingWithALetter = /^[a-z]/i.test(state.currentQuestion.title);
+        const onlyLettersAndNumbers = /^[a-z0-9]+$/i.test(state.currentQuestion.title);
+        return isNotEmpty && startingWithALetter && onlyLettersAndNumbers;
     },
     hasIndividualSubquestionTitles: (state) => {
         return reduce(