diff --git a/src/js/App.js b/src/js/App.js
index 4e0602c5..bf25aedc 100755
--- a/src/js/App.js
+++ b/src/js/App.js
@@ -127,7 +127,7 @@ class App extends React.Component{
if (this.props.play_state == 'playing'){
this.props.mopidyActions.pause()
this.props.uiActions.createNotification('pause', 'shortcut', 'shortcut')
- }else{
+ } else {
this.props.mopidyActions.play()
this.props.uiActions.createNotification('play', 'shortcut', 'shortcut')
}
diff --git a/src/js/components/ArtistSentence.js b/src/js/components/ArtistSentence.js
index 208eabc2..71d42e4a 100755
--- a/src/js/components/ArtistSentence.js
+++ b/src/js/components/ArtistSentence.js
@@ -27,7 +27,7 @@ export default class ArtistSentence extends React.Component{
var content = -
} else if (!artist.uri || this.props.nolinks){
var content = { artist.name }
- }else{
+ } else {
var content = { artist.name }
}
return (
diff --git a/src/js/components/ConfirmationButton.js b/src/js/components/ConfirmationButton.js
index 13fcbd87..650e618c 100755
--- a/src/js/components/ConfirmationButton.js
+++ b/src/js/components/ConfirmationButton.js
@@ -22,7 +22,7 @@ export default class ConfirmationButton extends React.Component{
if (this.state.confirming){
this.setState({ confirming: false });
this.props.onConfirm();
- }else{
+ } else {
this.setState({ confirming: true });
}
}
diff --git a/src/js/components/Dater.js b/src/js/components/Dater.js
index 13335eb9..f6bea4ba 100755
--- a/src/js/components/Dater.js
+++ b/src/js/components/Dater.js
@@ -100,7 +100,7 @@ export default class Dater extends React.Component{
return minutes + " minutes";
}else if (hours < 24){
return hours + " hours";
- }else{
+ } else {
return days + " days"
}
break
diff --git a/src/js/components/ProgressSlider.js b/src/js/components/ProgressSlider.js
index 497c4fbb..f432dbd7 100755
--- a/src/js/components/ProgressSlider.js
+++ b/src/js/components/ProgressSlider.js
@@ -21,7 +21,7 @@ class ProgressSlider extends React.Component{
var percent = (sliderX / sliderWidth ).toFixed(2);
if (this.props.connected && this.props.current_track){
- var destination_time = this.props.current_track.length * percent
+ var destination_time = this.props.current_track.duration * percent
this.props.mopidyActions.seek(destination_time )
this.setState({ animating: false })
}
@@ -30,7 +30,7 @@ class ProgressSlider extends React.Component{
render(){
var percent = 0
if (this.props.connected && this.props.current_track){
- percent = this.props.time_position / this.props.current_track.length
+ percent = this.props.time_position / this.props.current_track.duration
percent = percent * 100;
if (percent > 100 ) percent = 100
}
@@ -53,7 +53,7 @@ class ProgressSlider extends React.Component{
const mapStateToProps = (state, ownProps) => {
return {
- current_track: (typeof(state.core.current_track) !== 'undefined' && typeof(state.core.tracks) !== 'undefined' && typeof(state.core.tracks[state.core.current_track.uri]) !== 'undefined' ? state.core.tracks[state.core.current_track.uri] : null),
+ current_track: (state.core.current_track !== undefined && state.core.tracks !== undefined && state.core.tracks[state.core.current_track.uri] !== undefined ? state.core.tracks[state.core.current_track.uri] : null),
connected: state.mopidy.connected,
time_position: state.mopidy.time_position,
play_state: state.mopidy.play_state
diff --git a/src/js/components/RelatedArtists.js b/src/js/components/RelatedArtists.js
index e6a015ff..5d4a0679 100755
--- a/src/js/components/RelatedArtists.js
+++ b/src/js/components/RelatedArtists.js
@@ -23,7 +23,7 @@ export default class RelatedArtists extends React.Component{
{ artist.name }
)
- }else{
+ } else {
return (
diff --git a/src/js/components/Track.js b/src/js/components/Track.js
index b4e0e68c..b9c298e9 100755
--- a/src/js/components/Track.js
+++ b/src/js/components/Track.js
@@ -208,8 +208,7 @@ export default class Track extends React.Component{
)
track_columns.push(
- {track.duration_ms ? : null}
- {track.length ? : null}
+ {track.duration ? : null}
)
}
diff --git a/src/js/helpers.js b/src/js/helpers.js
index bff4510f..189330dd 100755
--- a/src/js/helpers.js
+++ b/src/js/helpers.js
@@ -27,7 +27,7 @@ export let sizedImages = function(images){
sizes.medium = image.url;
}else if (image.width < 1000){
sizes.large = image.url;
- }else{
+ } else {
sizes.huge = image.url;
}
@@ -44,7 +44,7 @@ export let sizedImages = function(images){
sizes.medium = image.url;
}else if (image.width < 1000){
sizes.large = image.url;
- }else{
+ } else {
sizes.huge = image.url;
}
@@ -464,7 +464,7 @@ export let sortItems = function (array, property, reverse = false){
if (typeof(a_value[a_property_split[i]]) === 'undefined'){
a_value = false
break
- }else{
+ } else {
a_value = a_value[a_property_split[i]]
}
}
@@ -491,7 +491,7 @@ export let sortItems = function (array, property, reverse = false){
if (a_value.toLowerCase() < b_value.toLowerCase()) return -1
return 0
- }else{
+ } else {
if (parseInt(a_value) > parseInt(b_value)) return 1
if (parseInt(a_value) < parseInt(b_value)) return -1
return 0
diff --git a/src/js/services/core/reducer.js b/src/js/services/core/reducer.js
index 897f39c2..6a51fd7f 100755
--- a/src/js/services/core/reducer.js
+++ b/src/js/services/core/reducer.js
@@ -71,10 +71,17 @@ export default function reducer(core = {}, action){
var tracks = Object.assign({}, core.tracks)
if (tracks[action.key]){
var track = Object.assign({}, tracks[action.key], action.track)
- }else{
+ } else {
var track = Object.assign({}, action.track)
}
+ // Standardise components
+ if (!track.track_number && track.track_no) track.track_number = track.track_no;
+ if (!track.disc_number && track.disc_no) track.disc_number = track.disc_no;
+ if (!track.duration && track.duration_ms) track.duration = track.duration_ms;
+ if (!track.duration && track.length) track.duration = track.length;
+ if (!track.date && track.release_date) track.date = track.release_date;
+
tracks[action.key] = track
return Object.assign({}, core, { tracks: tracks });
@@ -129,7 +136,7 @@ export default function reducer(core = {}, action){
if (categories[action.key]){
var category = Object.assign({}, categories[action.key], action.category)
- }else{
+ } else {
var category = Object.assign({}, action.category)
}
@@ -143,7 +150,7 @@ export default function reducer(core = {}, action){
var key = 'category:'+action.categories[i].id
if (categories[key]){
var category = Object.assign({}, categories[key], action.categories[i])
- }else{
+ } else {
var category = Object.assign({}, action.categories[i])
}
categories[key] = category
@@ -180,7 +187,7 @@ export default function reducer(core = {}, action){
if (albums[action.key]){
var album = Object.assign({}, albums[action.key], action.album)
- }else{
+ } else {
var album = Object.assign({}, action.album)
}
@@ -237,7 +244,7 @@ export default function reducer(core = {}, action){
}
var artist = Object.assign({}, artists[action.key], action.artist)
- }else{
+ } else {
var artist = Object.assign({}, action.artist)
}
@@ -284,7 +291,7 @@ export default function reducer(core = {}, action){
if (users[action.key]){
var user = Object.assign({}, users[action.key], action.user)
- }else{
+ } else {
var user = Object.assign({}, action.user)
}
@@ -340,7 +347,7 @@ export default function reducer(core = {}, action){
tracks: tracks
}
)
- }else{
+ } else {
var merged_playlist = Object.assign({}, action.playlist)
}
@@ -461,7 +468,7 @@ export default function reducer(core = {}, action){
case 'LIBRARY_PLAYLISTS_LOADED':
if (core.library_playlists){
var library_playlists = [...core.library_playlists, ...action.uris]
- }else{
+ } else {
var library_playlists = action.uris
}
@@ -500,7 +507,7 @@ export default function reducer(core = {}, action){
// artists
if (core.search_results && core.search_results.artists_uris){
var artists_uris = core.search_results.artists_uris
- }else{
+ } else {
var artists_uris = []
}
if (action.artists_uris) artists_uris = [...artists_uris, ...action.artists_uris]
@@ -514,7 +521,7 @@ export default function reducer(core = {}, action){
// albums
if (core.search_results && core.search_results.albums_uris){
var albums_uris = core.search_results.albums_uris
- }else{
+ } else {
var albums_uris = []
}
if (action.albums_uris) albums_uris = [...albums_uris, ...action.albums_uris]
@@ -528,7 +535,7 @@ export default function reducer(core = {}, action){
// playlists
if (core.search_results && core.search_results.playlists_uris){
var playlists_uris = core.search_results.playlists_uris
- }else{
+ } else {
var playlists_uris = []
}
if (action.playlists_uris) playlists_uris = [...playlists_uris, ...action.playlists_uris]
@@ -542,7 +549,7 @@ export default function reducer(core = {}, action){
// tracks
if (core.search_results && core.search_results.tracks){
var tracks = core.search_results.tracks
- }else{
+ } else {
var tracks = []
}
if (action.tracks) tracks = [...tracks, ...action.tracks]
diff --git a/src/js/services/lastfm/actions.js b/src/js/services/lastfm/actions.js
index 038608c2..53703bb0 100755
--- a/src/js/services/lastfm/actions.js
+++ b/src/js/services/lastfm/actions.js
@@ -63,7 +63,7 @@ export function getArtist(uri, artist, mbid = false){
return (dispatch, getState) => {
if (mbid){
var params = 'method=artist.getInfo&mbid='+mbid
- }else{
+ } else {
artist = encodeURIComponent(artist );
var params = 'method=artist.getInfo&artist='+artist
}
@@ -95,7 +95,7 @@ export function getAlbum(artist, album, mbid = false){
if (mbid){
var params = 'method=album.getInfo&mbid='+mbid
- }else{
+ } else {
artist = encodeURIComponent(artist )
album = encodeURIComponent(album )
var params = 'method=album.getInfo&album='+album+'&artist='+artist
diff --git a/src/js/services/mopidy/middleware.js b/src/js/services/mopidy/middleware.js
index 95e7743c..5ef15817 100755
--- a/src/js/services/mopidy/middleware.js
+++ b/src/js/services/mopidy/middleware.js
@@ -50,7 +50,7 @@ const MopidyMiddleware = (function(){
store.dispatch(mopidyActions.getTimePosition())
// otherwise we just assume to add 1000ms every 1000ms of play time
- }else{
+ } else {
store.dispatch(mopidyActions.setTimePosition(store.getState().mopidy.time_position + 1000 ))
}
@@ -133,7 +133,7 @@ const MopidyMiddleware = (function(){
if (method in ws[model]){
var mopidyObject = ws[model][method]
var property = method;
- }else{
+ } else {
var mopidyObject = ws[model]
var property = model;
}
@@ -1178,7 +1178,7 @@ const MopidyMiddleware = (function(){
var playlist = Object.assign({}, response)
if (playlist.tracks){
playlist.tracks = [...playlist.tracks, ...tracks]
- }else{
+ } else {
playlist.tracks = tracks
}
@@ -1628,7 +1628,7 @@ const MopidyMiddleware = (function(){
if (!artist.images || artist.images.length <= 0){
if (artist.musicbrainz_id){
store.dispatch(lastfmActions.getArtist(artist.uri, false, artist.musicbrainz_id ) )
- }else{
+ } else {
store.dispatch(lastfmActions.getArtist(artist.uri, artist.name.replace('&','and') ) )
}
}
diff --git a/src/js/services/pusher/middleware.js b/src/js/services/pusher/middleware.js
index 9998fe63..b5f4b121 100755
--- a/src/js/services/pusher/middleware.js
+++ b/src/js/services/pusher/middleware.js
@@ -259,7 +259,7 @@ const PusherMiddleware = (function(){
if (response.upgrade_successful){
store.dispatch(uiActions.createNotification('Upgrade complete') )
- }else{
+ } else {
store.dispatch(uiActions.createNotification('Upgrade failed, please upgrade manually','bad') )
}
diff --git a/src/js/services/spotify/actions.js b/src/js/services/spotify/actions.js
index 852b4c2c..952278de 100755
--- a/src/js/services/spotify/actions.js
+++ b/src/js/services/spotify/actions.js
@@ -703,14 +703,14 @@ export function following(uri, method = 'GET'){
case 'album':
if (method == 'GET'){
endpoint = 'me/albums/contains/?ids='+ helpers.getFromUri('albumid', uri)
- }else{
+ } else {
endpoint = 'me/albums/?ids='+ helpers.getFromUri('albumid', uri)
}
break
case 'artist':
if (method == 'GET'){
endpoint = 'me/following/contains?type=artist&ids='+ helpers.getFromUri('artistid', uri)
- }else{
+ } else {
endpoint = 'me/following?type=artist&ids='+ helpers.getFromUri('artistid', uri)
data = {}
}
@@ -718,7 +718,7 @@ export function following(uri, method = 'GET'){
case 'user':
if (method == 'GET'){
endpoint = 'me/following/contains?type=user&ids='+ helpers.getFromUri('userid', uri)
- }else{
+ } else {
endpoint = 'me/following?type=user&ids='+ helpers.getFromUri('userid', uri)
data = {}
}
@@ -726,7 +726,7 @@ export function following(uri, method = 'GET'){
case 'playlist':
if (method == 'GET'){
endpoint = 'users/'+ helpers.getFromUri('userid',uri) +'/playlists/'+ helpers.getFromUri('playlistid',uri) +'/followers/contains?ids='+ getState().spotify.me.id
- }else{
+ } else {
endpoint = 'users/'+ helpers.getFromUri('userid',uri) +'/playlists/'+ helpers.getFromUri('playlistid',uri) +'/followers'
}
break
diff --git a/src/js/views/Album.js b/src/js/views/Album.js
index 9fc70cec..62de1321 100755
--- a/src/js/views/Album.js
+++ b/src/js/views/Album.js
@@ -66,7 +66,7 @@ class Album extends React.Component{
case 'spotify':
if (props.album && props.album.tracks && props.album.artists_uris){
console.info('Loading album from index')
- }else{
+ } else {
this.props.spotifyActions.getAlbum(props.params.uri );
}
break;
diff --git a/src/js/views/Artist.js b/src/js/views/Artist.js
index b4f9874d..c794b7c4 100755
--- a/src/js/views/Artist.js
+++ b/src/js/views/Artist.js
@@ -60,7 +60,7 @@ class Artist extends React.Component{
case 'spotify':
if (props.artist && props.artist.albums_uris && props.artist.related_artists_uris){
console.info('Loading spotify artist from index')
- }else{
+ } else {
this.props.spotifyActions.getArtist(props.params.uri, true);
}
break
diff --git a/src/js/views/Track.js b/src/js/views/Track.js
index fcd3b1d7..3e081c9f 100755
--- a/src/js/views/Track.js
+++ b/src/js/views/Track.js
@@ -72,7 +72,7 @@ class Track extends React.Component{
case 'spotify':
if (props.track){
console.info('Loading track from index')
- }else{
+ } else {
this.props.spotifyActions.getTrack(props.params.uri );
}
break;
@@ -145,13 +145,12 @@ class Track extends React.Component{
{track.date ? : null}
{track.explicit ? EXPLICIT : null}
- {track.disc_no ? Disc {track.disc_no} : null}
{track.disc_number ? Disc {track.disc_number} : null}
- {track.track_no ? , track {track.track_no} : null}
- {track.track_number ? , track {track.track_number} : null}
+ {track.disc_number && track.track_number ? , : null}
+ {track.track_number ? Track {track.track_number} : null}
- {track.duration_ms ? : null}
+ {track.duration ? : null}
{track.length ? : null}
diff --git a/src/js/views/library/LibraryAlbums.js b/src/js/views/library/LibraryAlbums.js
index 06dabbb6..2000c3ee 100755
--- a/src/js/views/library/LibraryAlbums.js
+++ b/src/js/views/library/LibraryAlbums.js
@@ -173,7 +173,7 @@ class LibraryAlbums extends React.Component{
}
)
- }else{
+ } else {
return (
this.handleContextMenu(e,item)}
diff --git a/src/js/views/library/LibraryArtists.js b/src/js/views/library/LibraryArtists.js
index 7368e1e9..5b3e656d 100755
--- a/src/js/views/library/LibraryArtists.js
+++ b/src/js/views/library/LibraryArtists.js
@@ -111,7 +111,7 @@ class LibraryArtists extends React.Component{
link_prefix={global.baseURL+"artist/"} />
)
- }else{
+ } else {
return (
)
- }else{
+ } else {
return (