fix: 🐛 Fixed an error in confirmation modal

#62

src/components/editor/EditorOperationPanel.vue

@@ -61,13 +61,16 @@ export default {
   },
   methods: {
     async onClickShuffle() {
+      const confirmed = await this.askForConfirmation({
+        headline: 'Shuffle',
+        question: 'Are you sure you want to shuffle your playlist?',
+        positive: 'Shuffle',
+        negative: 'Cancel',
+      });
+
+      if (!confirmed) return;
+
       try {
-        await this.askForConfirmation({
-          headline: 'Shuffle',
-          question: 'Are you sure you want to shuffle your playlist?',
-          positive: 'Shuffle',
-          negative: 'Cancel',
-        });
         await this.rearrangePlaylistWith({
           rearranger: RandomShuffle,
         });

src/components/editor/SortConfiguration.vue

@@ -92,12 +92,15 @@ export default {
     },
     async sort() {
       this.showModal = false;
-      await this.askForConfirmation({
+      const confirmed = await this.askForConfirmation({
         headline: 'Sort',
         question: 'Are you sure you want to sort your playlist?',
         positive: 'Sort',
         negative: 'Cancel',
       });
+
+      if (!confirmed) return;
+
       await this.rearrangePlaylistWith({
         rearranger: this.sortModeMap[this.sortMode],
         options: this.options,

src/store/app/actions.js

@@ -42,9 +42,9 @@ export default {
   },
 
   askForConfirmation({ commit }, confirmation = {}) {
-    const promise = new Promise((resolve, reject) => {
+    const promise = new Promise((resolve) => {
       commit('setOnConfirmationAccept', resolve);
-      commit('setOnConfirmationDecline', reject);
+      commit('setOnConfirmationDecline', resolve);
     });
 
     commit('setPendingConfirmation', {
@@ -54,20 +54,21 @@ export default {
       negative: 'No',
       ...confirmation,
     });
+
     return promise;
   },
 
   async acceptConfirmation({ commit, state }) {
     const { onConfirmationAccept } = state;
-    onConfirmationAccept(); // resolve promise
+    onConfirmationAccept(true); // resolve promise
     commit('setPendingConfirmation', null);
     commit('setOnConfirmationAccept', null);
     commit('setOnConfirmationDecline', null);
   },
 
   declineConfirmation({ commit, state }) {
     const { onConfirmationDecline } = state;
-    onConfirmationDecline(); // reject promise
+    onConfirmationDecline(false); // resolve promise
     commit('setPendingConfirmation', null);
     commit('setOnConfirmationAccept', null);
     commit('setOnConfirmationDecline', null);