Improving logic behind vote and avoiding dups.

chrux · Jul 24, 2019 · c8fa58a · c8fa58a
1 parent c15deed
commit c8fa58a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/resources/js/components/FormQuestionComponent.vue b/resources/js/components/FormQuestionComponent.vue
@@ -51,7 +51,7 @@
               class="btn btn-primary btn-lg btn-block"
               type="button"
               :disabled="!isValid"
-              @click="newQuestion">{{ $t('Create') }}</button>
+              @click.once="newQuestion">{{ $t('Create') }}</button>
           </div>
         </div>
       </div>

diff --git a/resources/js/components/SurveyInstanceComponent.vue b/resources/js/components/SurveyInstanceComponent.vue
@@ -45,7 +45,7 @@
             type="button"
             class="btn btn-primary btn-lg btn-block"
             :disabled="!isValid"
-            @click="vote"
+            @click.once="vote"
             >{{ $t('Send') }}</button>
         </div>
       </div>
@@ -76,11 +76,11 @@ export default {
     },
     toggleOption(e) {
       const maxCheck = this.question.maxCheck || 1;
-      if (this.selectedOptions.length === maxCheck) {
-        return;
-      }
 
       if (e.selected) {
+        if (this.selectedOptions.length === maxCheck) {
+          return;
+        }
         this.selectedOptions.push(e.option.id);
       } else {
         this.selectedOptions = this.selectedOptions.filter(id => id !== e.option.id);