Replace finally with catch-then for compatibility (#321)

Author: stefandesu

src/MappingsApp.vue

@@ -574,7 +574,7 @@ export default {
         }).catch(error => {
           console.warn("Error fetching mapping schemes:", error)
           this.concordances = []
-        }).finally(() => {
+        }).then(() => {
           // If there are no concordances, jump to second tab.
           if (this.concordances.length == 0 && this.tab == 0) {
             this.tab = 1

src/components/MappingBrowser.vue

@@ -1130,7 +1130,7 @@ export default {
 
       }
 
-      Promise.all(promises).finally(() => {
+      Promise.all(promises).then(() => {
         if (this.loadingId == loadingId) {
           // Reset loading ID
           this.loading = 0
@@ -1225,7 +1225,9 @@ export default {
         }
         // Refresh list of mappings/suggestions.
         this.$store.commit("mapping/setRefresh", { registry: _.get(this.currentRegistry, "uri") })
-      }).finally(() => {
+      }).catch(error => {
+        console.error("MappingBrowser - error in removeMapping:", error)
+      }).then(() => {
         this.loadingGlobal = false
       })
     },
@@ -1275,7 +1277,9 @@ export default {
           this.alert(message, null, "danger")
         }
         return mapping
-      }).finally(() => {
+      }).catch(error => {
+        console.error("MappingBrowser - error in saveMapping:", error)
+      }).then(() => {
         this.loadingGlobal = false
         // Refresh list of mappings/suggestions.
         this.$store.commit("mapping/setRefresh", { registry: _.get(this.currentRegistry, "uri") })

src/components/MappingEditor.vue

@@ -440,7 +440,9 @@ export default {
         if (this.clearOnSave) {
           this.clearMapping()
         }
-      }).finally(() => {
+      }).catch(error => {
+        console.error("MappingEditor - error in saveMapping:", error)
+      }).then(() => {
         this.loadingGlobal = false
         this.$store.commit("mapping/setRefresh", { registry: _.get(this.$store.getters.getCurrentRegistry, "uri") })
       })
@@ -477,7 +479,9 @@ export default {
         } else {
           this.alert(this.$t("alerts.mappingNotDeleted"), null, "danger")
         }
-      }).finally(() => {
+      }).catch(error => {
+        console.error("MappingEditor - error in deleteOriginalMapping:", error)
+      }).then(() => {
         this.loadingGlobal = false
       })
       return true

src/providers/local-mappings-provider.js

@@ -165,9 +165,10 @@ class LocalMappingsProvider extends BaseProvider {
       localMappings = localMappings.map(mapping => jskos.minifyMapping(mapping))
       return localforage.setItem(this.localStorageKey, localMappings).then(() => {
         return mapping
-      }).catch(() => {
+      }).catch(error => {
+        console.error("local-mappings - error in saveMapping:", error)
         return null
-      }).finally(mapping => {
+      }).then(mapping => {
         done()
         return mapping
       })
@@ -193,9 +194,10 @@ class LocalMappingsProvider extends BaseProvider {
       localMappings = localMappings.map(mapping => jskos.minifyMapping(mapping))
       return localforage.setItem(this.localStorageKey, localMappings).then(() => {
         return mapping
-      }).catch(() => {
+      }).catch(error => {
+        console.error("local-mappings - error in removeMapping:", error)
         return null
-      }).finally(mapping => {
+      }).then(mapping => {
         done()
         return mapping
       })