From be1e1e7ba070514256377dc43c95fcf733d5bc34 Mon Sep 17 00:00:00 2001
From: advplyr <advplyr@protonmail.com>
Date: Sun, 13 Mar 2022 17:33:50 -0500
Subject: [PATCH] New data model update stats page and routes, update users
 page

---
 client/components/stats/PreviewIcons.vue | 14 ++++-----
 client/pages/config/library-stats.vue    | 26 ++++++++--------
 client/pages/config/stats.vue            | 14 ++++-----
 client/pages/config/users/_id.vue        |  6 ++--
 server/ApiController.js                  |  8 ++---
 server/controllers/LibraryController.js  | 18 +++++------
 server/utils/libraryHelpers.js           | 38 ++++++++++++------------
 7 files changed, 62 insertions(+), 62 deletions(-)

diff --git a/client/components/stats/PreviewIcons.vue b/client/components/stats/PreviewIcons.vue
index 918c1294..d96f0593 100644
--- a/client/components/stats/PreviewIcons.vue
+++ b/client/components/stats/PreviewIcons.vue
@@ -5,7 +5,7 @@
         <path fill="currentColor" d="M9 3V18H12V3H9M12 5L16 18L19 17L15 4L12 5M5 5V18H8V5H5M3 19V21H21V19H3Z" />
       </svg>
       <div class="px-2">
-        <p class="text-4xl md:text-5xl font-bold">{{ totalBooks }}</p>
+        <p class="text-4xl md:text-5xl font-bold">{{ totalItems }}</p>
         <p class="font-book text-xs md:text-sm text-white text-opacity-80">Books in Library</p>
       </div>
     </div>
@@ -13,7 +13,7 @@
     <div class="flex px-4">
       <span class="material-icons text-7xl">show_chart</span>
       <div class="px-1">
-        <p class="text-4xl md:text-5xl font-bold">{{ totalAudiobookHours }}</p>
+        <p class="text-4xl md:text-5xl font-bold">{{ totalHours }}</p>
         <p class="font-book text-xs md:text-sm text-white text-opacity-80">Overall Hours</p>
       </div>
     </div>
@@ -61,8 +61,8 @@ export default {
     user() {
       return this.$store.state.user.user
     },
-    totalBooks() {
-      return this.libraryStats ? this.libraryStats.totalBooks : 0
+    totalItems() {
+      return this.libraryStats ? this.libraryStats.totalItems : 0
     },
     totalAuthors() {
       return this.libraryStats ? this.libraryStats.totalAuthors : 0
@@ -70,11 +70,11 @@ export default {
     numAudioTracks() {
       return this.libraryStats ? this.libraryStats.numAudioTracks : 0
     },
-    totalAudiobookDuration() {
+    totalDuration() {
       return this.libraryStats ? this.libraryStats.totalDuration : 0
     },
-    totalAudiobookHours() {
-      var totalHours = Math.round(this.totalAudiobookDuration / (60 * 60))
+    totalHours() {
+      var totalHours = Math.round(this.totalDuration / (60 * 60))
       return totalHours
     },
     totalSizePretty() {
diff --git a/client/pages/config/library-stats.vue b/client/pages/config/library-stats.vue
index 3f809c67..381c4987 100644
--- a/client/pages/config/library-stats.vue
+++ b/client/pages/config/library-stats.vue
@@ -11,12 +11,12 @@
         <template v-for="genre in top5Genres">
           <div :key="genre.genre" class="w-full py-2">
             <div class="flex items-end mb-1">
-              <p class="text-2xl font-bold">{{ Math.round((100 * genre.count) / totalBooks) }}&nbsp;%</p>
+              <p class="text-2xl font-bold">{{ Math.round((100 * genre.count) / totalItems) }}&nbsp;%</p>
               <div class="flex-grow" />
               <p class="text-base font-book text-white text-opacity-70">{{ genre.genre }}</p>
             </div>
             <div class="w-full rounded-full h-3 bg-primary bg-opacity-50 overflow-hidden">
-              <div class="bg-yellow-400 h-full rounded-full" :style="{ width: Math.round((100 * genre.count) / totalBooks) + '%' }" />
+              <div class="bg-yellow-400 h-full rounded-full" :style="{ width: Math.round((100 * genre.count) / totalItems) + '%' }" />
             </div>
           </div>
         </template>
@@ -39,14 +39,14 @@
         </template>
       </div>
       <div class="w-80 my-6 mx-auto">
-        <h1 class="text-2xl mb-4 font-book">Longest Audiobooks (hrs)</h1>
-        <p v-if="!top10LongestAudiobooks.length">No Audiobooks</p>
-        <template v-for="(ab, index) in top10LongestAudiobooks">
+        <h1 class="text-2xl mb-4 font-book">Longest Items (hrs)</h1>
+        <p v-if="!top10LongestItems.length">No Items</p>
+        <template v-for="(ab, index) in top10LongestItems">
           <div :key="index" class="w-full py-2">
             <div class="flex items-center mb-1">
               <p class="text-sm font-book text-white text-opacity-70 w-44 pr-2 truncate">{{ index + 1 }}.&nbsp;&nbsp;&nbsp;&nbsp;{{ ab.title }}</p>
               <div class="flex-grow rounded-full h-2.5 bg-primary bg-opacity-0 overflow-hidden">
-                <div class="bg-yellow-400 h-full rounded-full" :style="{ width: Math.round((100 * ab.duration) / longestAudiobookDuration) + '%' }" />
+                <div class="bg-yellow-400 h-full rounded-full" :style="{ width: Math.round((100 * ab.duration) / longestItemDuration) + '%' }" />
               </div>
               <div class="w-4 ml-3">
                 <p class="text-sm font-bold">{{ (ab.duration / 3600).toFixed(1) }}</p>
@@ -77,8 +77,8 @@ export default {
     user() {
       return this.$store.state.user.user
     },
-    totalBooks() {
-      return this.libraryStats ? this.libraryStats.totalBooks : 0
+    totalItems() {
+      return this.libraryStats ? this.libraryStats.totalItems : 0
     },
     genresWithCount() {
       return this.libraryStats ? this.libraryStats.genresWithCount : []
@@ -86,12 +86,12 @@ export default {
     top5Genres() {
       return this.genresWithCount.slice(0, 5)
     },
-    top10LongestAudiobooks() {
-      return this.libraryStats ? this.libraryStats.longestAudiobooks || [] : []
+    top10LongestItems() {
+      return this.libraryStats ? this.libraryStats.longestItems || [] : []
     },
-    longestAudiobookDuration() {
-      if (!this.top10LongestAudiobooks.length) return 0
-      return this.top10LongestAudiobooks[0].duration
+    longestItemDuration() {
+      if (!this.top10LongestItems.length) return 0
+      return this.top10LongestItems[0].duration
     },
     authorsWithCount() {
       return this.libraryStats ? this.libraryStats.authorsWithCount : []
diff --git a/client/pages/config/stats.vue b/client/pages/config/stats.vue
index 758827fd..a4f20410 100644
--- a/client/pages/config/stats.vue
+++ b/client/pages/config/stats.vue
@@ -59,8 +59,8 @@
 export default {
   data() {
     return {
-      listeningStats: null,
-      libraryStats: null
+      listeningStats: null
+      // libraryStats: null
     }
   },
   watch: {
@@ -103,11 +103,11 @@ export default {
   },
   methods: {
     async init() {
-      this.libraryStats = await this.$axios.$get(`/api/libraries/${this.currentLibraryId}/stats`).catch((err) => {
-        console.error('Failed to get library stats', err)
-        var errorMsg = err.response ? err.response.data || 'Unknown Error' : 'Unknown Error'
-        this.$toast.error(`Failed to get library stats: ${errorMsg}`)
-      })
+      // this.libraryStats = await this.$axios.$get(`/api/libraries/${this.currentLibraryId}/stats`).catch((err) => {
+      //   console.error('Failed to get library stats', err)
+      //   var errorMsg = err.response ? err.response.data || 'Unknown Error' : 'Unknown Error'
+      //   this.$toast.error(`Failed to get library stats: ${errorMsg}`)
+      // })
       this.listeningStats = await this.$axios.$get(`/api/me/listening-stats`).catch((err) => {
         console.error('Failed to load listening sesions', err)
         return []
diff --git a/client/pages/config/users/_id.vue b/client/pages/config/users/_id.vue
index c4726ca5..7b77915c 100644
--- a/client/pages/config/users/_id.vue
+++ b/client/pages/config/users/_id.vue
@@ -43,11 +43,11 @@
           </tr>
           <tr v-for="ab in userAudiobooks" :key="ab.audiobookId" :class="!ab.isRead ? '' : 'isRead'">
             <td>
-              <covers-book-cover :width="50" :audiobook="ab" :book-cover-aspect-ratio="bookCoverAspectRatio" />
+              <covers-book-cover :width="50" :library-item="ab" :book-cover-aspect-ratio="bookCoverAspectRatio" />
             </td>
             <td class="font-book">
-              <p>{{ ab.book ? ab.book.title : ab.audiobookTitle || 'Unknown' }}</p>
-              <p v-if="ab.book && ab.book.author" class="text-white text-opacity-50 text-sm font-sans">by {{ ab.book.author }}</p>
+              <p>{{ ab.media && ab.media.metadata ? ab.media.metadata.title : ab.audiobookTitle || 'Unknown' }}</p>
+              <p v-if="ab.media && ab.media.metadata && ab.media.metadata.authorName" class="text-white text-opacity-50 text-sm font-sans">by {{ ab.media.metadata.authorName }}</p>
             </td>
             <td class="text-center">{{ Math.floor(ab.progress * 100) }}%</td>
             <td class="text-center hidden sm:table-cell">
diff --git a/server/ApiController.js b/server/ApiController.js
index 458ffefb..a6b109cd 100644
--- a/server/ApiController.js
+++ b/server/ApiController.js
@@ -370,11 +370,11 @@ class ApiController {
     // User audiobook progress attach book details
     if (json.audiobooks && Object.keys(json.audiobooks).length) {
       for (const audiobookId in json.audiobooks) {
-        var audiobook = this.db.audiobooks.find(ab => ab.id === audiobookId)
-        if (!audiobook) {
-          Logger.error('[ApiController] Audiobook not found for users progress ' + audiobookId)
+        var libraryItem = this.db.libraryItems.find(li => li.id === audiobookId)
+        if (!libraryItem) {
+          Logger.error('[ApiController] Library item not found for users progress ' + audiobookId)
         } else {
-          json.audiobooks[audiobookId].book = audiobook.book.toJSON()
+          json.audiobooks[audiobookId].media = libraryItem.media.toJSONExpanded()
         }
       }
     }
diff --git a/server/controllers/LibraryController.js b/server/controllers/LibraryController.js
index 1ea4a1d7..911d3774 100644
--- a/server/controllers/LibraryController.js
+++ b/server/controllers/LibraryController.js
@@ -411,19 +411,19 @@ class LibraryController {
   }
 
   async stats(req, res) {
-    var audiobooksInLibrary = this.db.audiobooks.filter(ab => ab.libraryId === req.library.id)
+    var libraryItems = req.libraryItems
 
-    var authorsWithCount = libraryHelpers.getAuthorsWithCount(audiobooksInLibrary)
-    var genresWithCount = libraryHelpers.getGenresWithCount(audiobooksInLibrary)
-    var abDurationStats = libraryHelpers.getAudiobookDurationStats(audiobooksInLibrary)
+    var authorsWithCount = libraryHelpers.getAuthorsWithCount(libraryItems)
+    var genresWithCount = libraryHelpers.getGenresWithCount(libraryItems)
+    var durationStats = libraryHelpers.getItemDurationStats(libraryItems)
     var stats = {
-      totalBooks: audiobooksInLibrary.length,
+      totalItems: libraryItems.length,
       totalAuthors: Object.keys(authorsWithCount).length,
       totalGenres: Object.keys(genresWithCount).length,
-      totalDuration: abDurationStats.totalDuration,
-      longestAudiobooks: abDurationStats.longstAudiobooks,
-      numAudioTracks: abDurationStats.numAudioTracks,
-      totalSize: libraryHelpers.getAudiobooksTotalSize(audiobooksInLibrary),
+      totalDuration: durationStats.totalDuration,
+      longestItems: durationStats.longestItems,
+      numAudioTracks: durationStats.numAudioTracks,
+      totalSize: libraryHelpers.getLibraryItemsTotalSize(libraryItems),
       authorsWithCount,
       genresWithCount
     }
diff --git a/server/utils/libraryHelpers.js b/server/utils/libraryHelpers.js
index a5345be2..b80bda34 100644
--- a/server/utils/libraryHelpers.js
+++ b/server/utils/libraryHelpers.js
@@ -244,10 +244,10 @@ module.exports = {
     return seriesSortedByAddedAt.slice(0, limit)
   },
 
-  getGenresWithCount(audiobooks) {
+  getGenresWithCount(libraryItems) {
     var genresMap = {}
-    audiobooks.forEach((ab) => {
-      var genres = ab.book.genres || []
+    libraryItems.forEach((li) => {
+      var genres = li.media.metadata.genres || []
       genres.forEach((genre) => {
         if (genresMap[genre]) genresMap[genre].count++
         else
@@ -260,15 +260,15 @@ module.exports = {
     return Object.values(genresMap).sort((a, b) => b.count - a.count)
   },
 
-  getAuthorsWithCount(audiobooks) {
+  getAuthorsWithCount(libraryItems) {
     var authorsMap = {}
-    audiobooks.forEach((ab) => {
-      var authors = ab.book.authorFL ? ab.book.authorFL.split(', ') : []
+    libraryItems.forEach((li) => {
+      var authors = li.media.metadata.authors || []
       authors.forEach((author) => {
-        if (authorsMap[author]) authorsMap[author].count++
+        if (authorsMap[author.id]) authorsMap[author.id].count++
         else
-          authorsMap[author] = {
-            author,
+          authorsMap[author.id] = {
+            author: author.name,
             count: 1
           }
       })
@@ -276,26 +276,26 @@ module.exports = {
     return Object.values(authorsMap).sort((a, b) => b.count - a.count)
   },
 
-  getAudiobookDurationStats(audiobooks) {
-    var sorted = sort(audiobooks).desc(a => a.duration)
-    var top10 = sorted.slice(0, 10).map(ab => ({ title: ab.book.title, duration: ab.duration })).filter(ab => ab.duration > 0)
+  getItemDurationStats(libraryItems) {
+    var sorted = sort(libraryItems).desc(li => li.media.duration)
+    var top10 = sorted.slice(0, 10).map(li => ({ title: li.media.metadata.title, duration: li.media.duration })).filter(i => i.duration > 0)
     var totalDuration = 0
     var numAudioTracks = 0
-    audiobooks.forEach((ab) => {
-      totalDuration += ab.duration
-      numAudioTracks += ab.tracks.length
+    libraryItems.forEach((li) => {
+      totalDuration += li.media.duration
+      numAudioTracks += (li.media.tracks || []).length
     })
     return {
       totalDuration,
       numAudioTracks,
-      longstAudiobooks: top10
+      longestItems: top10
     }
   },
 
-  getAudiobooksTotalSize(audiobooks) {
+  getLibraryItemsTotalSize(libraryItems) {
     var totalSize = 0
-    audiobooks.forEach((ab) => {
-      totalSize += ab.size
+    libraryItems.forEach((li) => {
+      totalSize += li.media.size
     })
     return totalSize
   },