Added error handling; Made querystring helper

client/components/cards/ItemUploadCard.vue

@@ -65,8 +65,10 @@
 
 <script>
 import Path from 'path'
+import apiRequestHelpers from '@/mixins/apiRequestHelpers'
 
 export default {
+  mixins: [apiRequestHelpers],
   props: {
     item: {
       type: Object,
@@ -132,27 +134,36 @@ export default {
       }
 
       this.isFetchingMetadata = true
+      this.error = ''
 
       try {
-        const searchQueryString = `title=${this.itemData.title}&author=${this.itemData.author}&provider=${this.provider}`
+        const searchQueryString = this.buildQuerystring({
+          title: this.itemData.title,
+          author: this.itemData.author,
+          provider: this.provider
+        })
         const [bestCandidate, ..._rest] = await this.$axios.$get(`/api/search/books?${searchQueryString}`)
 
-        this.itemData = {
+        if (bestCandidate) {
-          ...this.itemData,
+          this.itemData = {
-          title: bestCandidate?.title,
+            ...this.itemData,
-          author: bestCandidate?.author,
+            title: bestCandidate?.title,
-          series: (bestCandidate?.series || [])[0]?.series
+            author: bestCandidate?.author,
+            series: (bestCandidate?.series || [])[0]?.series
+          }
+        } else {
+          this.error = this.$strings.ErrorUploadFetchMetadataNoResults
         }
       } catch (e) {
         console.error('Failed', e)
-        // TODO: do something with the error?
+        this.error = this.$strings.ErrorUploadFetchMetadataAPI
       } finally {
         this.isFetchingMetadata = false
       }
     },
     getData() {
       if (!this.itemData.title) {
-        this.error = 'Must have a title'
+        this.error = this.$strings.ErrorUploadLacksTitle
         return null
       }
       this.error = ''

client/mixins/apiRequestHelpers.js

@@ -0,0 +1,12 @@
+export default {
+  methods: {
+    buildQuerystring(obj, opts = { includePrefix: false }) {
+      let querystring = Object
+        .entries(obj)
+        .map(([key, val]) => `${encodeURIComponent(key)}=${encodeURIComponent(val)}`)
+        .join('&')
+
+      return (opts.includePrefix ? '?' : '').concat(querystring)
+    }
+  }
+}

client/strings/en-us.json

@@ -87,6 +87,9 @@
   "ButtonUserEdit": "Edit user {0}",
   "ButtonViewAll": "View All",
   "ButtonYes": "Yes",
+  "ErrorUploadFetchMetadataAPI": "Error fetching metadata",
+  "ErrorUploadFetchMetadataNoResults": "Could not fetch metadata - try updating title and/or author",
+  "ErrorUploadLacksTitle": "Must have a title",
   "HeaderAccount": "Account",
   "HeaderAdvanced": "Advanced",
   "HeaderAppriseNotificationSettings": "Apprise Notification Settings",