From 2f8dc794def1c06089c3b7cf99adc72671acb779 Mon Sep 17 00:00:00 2001
From: Deluan <deluan@navidrome.org>
Date: Mon, 12 Oct 2020 21:28:59 -0400
Subject: [PATCH] Add and show Playlists sizes

---
 .github/FUNDING.yml                      |  2 +-
 model/playlist.go                        |  1 +
 persistence/playlist_track_repository.go | 10 ++++++----
 ui/src/playlist/PlaylistActions.js       |  8 +++++++-
 ui/src/playlist/PlaylistDetails.js       |  4 +++-
 5 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/.github/FUNDING.yml b/.github/FUNDING.yml
index d2a1001f..534cfbd1 100644
--- a/.github/FUNDING.yml
+++ b/.github/FUNDING.yml
@@ -3,8 +3,8 @@
 github: deluan
 patreon: # Replace with a single Patreon username
 open_collective: # Replace with a single Open Collective username
-liberapay: deluan
 ko_fi: deluan
+liberapay: deluan
 tidelift: # Replace with a single Tidelift platform-name/package-name e.g., npm/babel
 community_bridge: # Replace with a single Community Bridge project-name e.g., cloud-foundry
 issuehunt: # Replace with a single IssueHunt username
diff --git a/model/playlist.go b/model/playlist.go
index 6ce64bf1..a3595d6f 100644
--- a/model/playlist.go
+++ b/model/playlist.go
@@ -9,6 +9,7 @@ type Playlist struct {
 	Name      string     `json:"name"`
 	Comment   string     `json:"comment"`
 	Duration  float32    `json:"duration"`
+	Size      int64      `json:"size"`
 	SongCount int        `json:"songCount"`
 	Owner     string     `json:"owner"`
 	Public    bool       `json:"public"`
diff --git a/persistence/playlist_track_repository.go b/persistence/playlist_track_repository.go
index 52daeddd..3955fc24 100644
--- a/persistence/playlist_track_repository.go
+++ b/persistence/playlist_track_repository.go
@@ -140,19 +140,21 @@ func (r *playlistTrackRepository) Update(mediaFileIds []string) error {
 }
 
 func (r *playlistTrackRepository) updateStats() error {
-	// Get total playlist duration and count
-	statsSql := Select("sum(duration) as duration", "count(*) as count").From("media_file").
+	// Get total playlist duration, size and count
+	statsSql := Select("sum(duration) as duration", "sum(size) as size", "count(*) as count").
+		From("media_file").
 		Join("playlist_tracks f on f.media_file_id = media_file.id").
 		Where(Eq{"playlist_id": r.playlistId})
-	var res struct{ Duration, Count float32 }
+	var res struct{ Duration, Size, Count float32 }
 	err := r.queryOne(statsSql, &res)
 	if err != nil {
 		return err
 	}
 
-	// Update playlist's total duration and count
+	// Update playlist's total duration, size and count
 	upd := Update("playlist").
 		Set("duration", res.Duration).
+		Set("size", res.Size).
 		Set("song_count", res.Count).
 		Set("updated_at", time.Now()).
 		Where(Eq{"id": r.playlistId})
diff --git a/ui/src/playlist/PlaylistActions.js b/ui/src/playlist/PlaylistActions.js
index 64c0a287..0136d05b 100644
--- a/ui/src/playlist/PlaylistActions.js
+++ b/ui/src/playlist/PlaylistActions.js
@@ -16,10 +16,13 @@ import { playNext, addTracks, playTracks, shuffleTracks } from '../audioplayer'
 import { M3U_MIME_TYPE, REST_URL } from '../consts'
 import subsonic from '../subsonic'
 import PropTypes from 'prop-types'
+import { formatBytes } from '../common/SizeField'
+import { useMediaQuery } from '@material-ui/core'
 
 const PlaylistActions = ({ className, ids, data, record, ...rest }) => {
   const dispatch = useDispatch()
   const translate = useTranslate()
+  const isDesktop = useMediaQuery((theme) => theme.breakpoints.up('md'))
 
   const handlePlay = React.useCallback(() => {
     dispatch(playTracks(data, ids))
@@ -87,7 +90,10 @@ const PlaylistActions = ({ className, ids, data, record, ...rest }) => {
       </Button>
       <Button
         onClick={handleDownload}
-        label={translate('resources.album.actions.download')}
+        label={
+          translate('resources.album.actions.download') +
+          (isDesktop ? ` (${formatBytes(record.size)})` : '')
+        }
       >
         <CloudDownloadOutlinedIcon />
       </Button>
diff --git a/ui/src/playlist/PlaylistDetails.js b/ui/src/playlist/PlaylistDetails.js
index 1ef99dcf..c64f5cdb 100644
--- a/ui/src/playlist/PlaylistDetails.js
+++ b/ui/src/playlist/PlaylistDetails.js
@@ -2,7 +2,7 @@ import React from 'react'
 import { Card, CardContent, Typography } from '@material-ui/core'
 import { makeStyles } from '@material-ui/core/styles'
 import { useTranslate } from 'react-admin'
-import { DurationField } from '../common'
+import { DurationField, SizeField } from '../common'
 
 const useStyles = makeStyles((theme) => ({
   container: {
@@ -56,6 +56,8 @@ const PlaylistDetails = (props) => {
               })}
               {' · '}
               <DurationField record={record} source={'duration'} />
+              {' · '}
+              <SizeField record={record} source={'size'} />
             </span>
           ) : (
             <span>&nbsp;</span>