From 8f2e8599543999921f582afa8d0ea0ede00ca7ae Mon Sep 17 00:00:00 2001 From: teticio Date: Thu, 16 Sep 2021 10:57:58 +0100 Subject: [PATCH] cosmetic --- backend/main.py | 2 +- migrate_db.py | 1 + src/App.js | 2 +- src/components/AddTrack.js | 2 +- src/components/CreatePlaylist.js | 4 ++-- src/components/LatestPlaylists.js | 2 +- src/components/MostUploadedPlaylists.js | 2 +- src/components/RemovablePlaylist.js | 2 +- src/components/SearchPlaylists.js | 2 +- src/components/Settings.js | 2 +- src/components/ShowPlaylist.js | 17 ++++++++++------- src/components/Spotify.js | 2 +- src/components/TopPlaylists.js | 2 +- 13 files changed, 23 insertions(+), 19 deletions(-) diff --git a/backend/main.py b/backend/main.py index b73223d..76f5df2 100644 --- a/backend/main.py +++ b/backend/main.py @@ -381,7 +381,7 @@ def update_playlist_id(playlist: schemas.PlaylistId, @app.post('/api/v1/update_playlist_uploads') def update_playlist_uploads(playlist: schemas.PlaylistUploads, - db: Session = Depends(get_db)): + db: Session = Depends(get_db)): """Update number of times playlist has been uploaded to Spotify. Args: diff --git a/migrate_db.py b/migrate_db.py index 5da998e..09a6fa7 100644 --- a/migrate_db.py +++ b/migrate_db.py @@ -14,6 +14,7 @@ logging.basicConfig(format='%(levelname)s:%(message)s', level=logging.INFO) + def copy_objects(db_from, db_to, cls): """Copy all items in a table from one databse to another. diff --git a/src/App.js b/src/App.js index c52d854..3b0b903 100644 --- a/src/App.js +++ b/src/App.js @@ -65,7 +65,7 @@ function App() { setPlaylist(playlist); navigate('/playlist'); }} - onSettings={(waypoints) => { + onSettings={waypoints => { setWaypoints(waypoints); navigate('/settings'); }} diff --git a/src/components/AddTrack.js b/src/components/AddTrack.js index d39c09c..0479306 100644 --- a/src/components/AddTrack.js +++ b/src/components/AddTrack.js @@ -17,7 +17,7 @@ export default function AddTrack({ numTracks = 0, spotify = null, onAdd = f => f
setCurrentId(id)} + onSelect={id => setCurrentId(id)} onSearch={() => { setSearches(searches => { if (searches + 1 !== 0) setSpinner(true); diff --git a/src/components/CreatePlaylist.js b/src/components/CreatePlaylist.js index efec9cd..eab682b 100644 --- a/src/components/CreatePlaylist.js +++ b/src/components/CreatePlaylist.js @@ -27,13 +27,13 @@ export default function CreatePlaylist({ { + onAdd={id => { setWaypoints({ 'track_ids': _waypoints.track_ids.concat(id) }); }} /> - { + { setWaypoints({ 'track_ids': _waypoints.track_ids.filter((element, index) => index !== id) }); }} /> {_waypoints.track_ids.length > 0 ? diff --git a/src/components/LatestPlaylists.js b/src/components/LatestPlaylists.js index d82a136..a774fa3 100644 --- a/src/components/LatestPlaylists.js +++ b/src/components/LatestPlaylists.js @@ -19,7 +19,7 @@ export default function LatestPlaylists({ spotify }) { useEffect(() => { getLatestPlaylists(topN) - .then((playlists) => { + .then(playlists => { setPlaylists(playlists); }).catch(error => console.error('Error:', error)); }, [topN]); diff --git a/src/components/MostUploadedPlaylists.js b/src/components/MostUploadedPlaylists.js index ab5e9bf..0b41a63 100644 --- a/src/components/MostUploadedPlaylists.js +++ b/src/components/MostUploadedPlaylists.js @@ -19,7 +19,7 @@ export default function MostUploadedPlaylists({ spotify }) { useEffect(() => { getMostUploadedPlaylists(topN) - .then((playlists) => { + .then(playlists => { setPlaylists(playlists); }).catch(error => console.error('Error:', error)); }, [topN]); diff --git a/src/components/RemovablePlaylist.js b/src/components/RemovablePlaylist.js index 1534017..088b5f7 100644 --- a/src/components/RemovablePlaylist.js +++ b/src/components/RemovablePlaylist.js @@ -9,7 +9,7 @@ export default function RemovablePlaylist({ track_ids = [], onRemove = f => f }) key={i} track_id={track_id} uuid={i} - onRemove={(uuid) => onRemove(uuid)} + onRemove={uuid => onRemove(uuid)} />
))} diff --git a/src/components/SearchPlaylists.js b/src/components/SearchPlaylists.js index 670b6aa..b8b70db 100644 --- a/src/components/SearchPlaylists.js +++ b/src/components/SearchPlaylists.js @@ -29,7 +29,7 @@ export default function SearchPlaylists({ spotify }) { setPlaylists([]); } else { searchPlaylists(actualSearchString, topN) - .then((playlists) => { + .then(playlists => { setPlaylists(playlists); }).catch(error => console.error('Error:', error)); } diff --git a/src/components/Settings.js b/src/components/Settings.js index 6de073d..7695622 100644 --- a/src/components/Settings.js +++ b/src/components/Settings.js @@ -8,7 +8,7 @@ export default function Settings({ size, creativity, noise, onChange = f => f, o const [_size, setSize] = useState(size); const [_creativity, setCreativity] = useState(creativity); const [_noise, setNoise] = useState(noise); - const setValidSize = (size) => setSize(Math.max(Math.min(size, 100), 1)) + const setValidSize = size => setSize(Math.max(Math.min(size, 100), 1)) const update = () => onChange(_size, _creativity, _noise); useEffect(() => () => update()); diff --git a/src/components/ShowPlaylist.js b/src/components/ShowPlaylist.js index 9b3c8a5..c199a01 100644 --- a/src/components/ShowPlaylist.js +++ b/src/components/ShowPlaylist.js @@ -104,13 +104,16 @@ export default function ShowPlaylist({ playlist, onClose = f => f, spotify = nul
{rateIt ? - { - updatePlaylistRating( - playlist.id, - (rating + playlist.av_rating) / (playlist.num_ratings + 1), - playlist.num_ratings + 1 - ).catch(error => console.error('Error:', error)); - }} /> : + { + updatePlaylistRating( + playlist.id, + (rating + playlist.av_rating) / (playlist.num_ratings + 1), + playlist.num_ratings + 1 + ).catch(error => console.error('Error:', error)); + }} + /> : setRateIt(true)}> diff --git a/src/components/Spotify.js b/src/components/Spotify.js index da66ebb..15a6162 100644 --- a/src/components/Spotify.js +++ b/src/components/Spotify.js @@ -76,7 +76,7 @@ export default class Spotify extends SpotifyWebApi { } await this.replaceTracksInPlaylist( playlist.id, - track_ids.map((track_id) => `spotify:track:${track_id}`) + track_ids.map(track_id => `spotify:track:${track_id}`) ); return playlist; } diff --git a/src/components/TopPlaylists.js b/src/components/TopPlaylists.js index 23653d3..26ce559 100644 --- a/src/components/TopPlaylists.js +++ b/src/components/TopPlaylists.js @@ -19,7 +19,7 @@ export default function TopPlaylists({ spotify }) { useEffect(() => { getTopPlaylists(topN) - .then((playlists) => { + .then(playlists => { setPlaylists(playlists); }).catch(error => console.error('Error:', error)); }, [topN]);