Remove redundant nesting and statements

This commit is contained in:
aditya-K2
2023-12-28 05:12:19 +05:30
parent 78ec99fa82
commit 970b2804e3

View File

@@ -63,9 +63,9 @@ func (c *CoverArt) RefreshState() {
SendNotification(err.Error()) SendNotification(err.Error())
return return
} }
if state != nil { if state != nil && state.Item != nil &&
if state.Item != nil { len(state.Item.Album.Images) > 0 {
if len(state.Item.Album.Images) > 0 {
file := fileName(state.Item.Album) file := fileName(state.Item.Album)
// Download Image if doesn't Exits // Download Image if doesn't Exits
@@ -75,23 +75,19 @@ func (c *CoverArt) RefreshState() {
defer f.Close() defer f.Close()
if err != nil { if err != nil {
go func() { go func() {
msg <- fmt.Sprintf("Error Downloading Image: %s", err.Error()) msg <- fmt.Sprintf("Following error occured while creating file: %s", err.Error())
}() }()
return return
} }
if len(state.Item.Album.Images) == 0 {
SendNotification("No Cover Art found for album: %s!", state.Item.Album.Name)
return
}
err = state.Item.Album.Images[0].Download(f) err = state.Item.Album.Images[0].Download(f)
if err != nil { if err != nil {
go func() { go func() {
msg <- fmt.Sprintf("Error Downloading Image: %s", err.Error()) msg <- fmt.Sprintf("Following error occured while downloading the image: %s", err.Error())
}() }()
return return
} }
go func() { go func() {
msg <- "Image Downloaded Succesfully!" msg <- "Image downloaded succesfully!"
}() }()
} }
@@ -109,8 +105,7 @@ func (c *CoverArt) RefreshState() {
return return
} }
c.image = im c.image = im
}
}
} }
cm.Unlock() cm.Unlock()
} }