Skip to content

Commit 51ed855

Browse files
committed
Fix discogs enhance not mutating + sleep
1 parent f3e5a50 commit 51ed855

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

src/server/config.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ type Config struct {
1313
ServiceUrl string `json:"service_url"`
1414
DiscogsAPIKey string `json:"discogs_api_key"`
1515
JWTSecret string `json:"jwt_secret"`
16-
AllowOrigin string `json:"allow_origin`
16+
AllowOrigin string `json:"allow_origin"`
1717
}
1818

1919
func loadConfigFromFile(file string) *Config {

src/server/discogs/dicogs.go

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ import (
44
"encoding/json"
55
"fmt"
66
"io"
7-
"log"
87
"net/http"
98
"slices"
109
"strconv"
10+
"time"
1111

1212
"github.com/mightynerd/hit/db"
1313
)
@@ -75,7 +75,7 @@ func (config *DiscogsConfig) GetEarliestReleaseYear(artist string, track string)
7575
request, err := http.NewRequest("GET", fmt.Sprintf("https://api.discogs.com/database/search?type=release&artist=%s&track=%s", artist, track), nil)
7676

7777
if err != nil {
78-
log.Fatal(err)
78+
return 0, err
7979
}
8080

8181
request.Header.Add("User-Agent", "github.com/mightynerd")
@@ -109,7 +109,7 @@ func (config *DiscogsConfig) GetEarliestReleaseYear(artist string, track string)
109109
}
110110

111111
if len(years) < 1 {
112-
return 0, fmt.Errorf("No results found")
112+
return 0, fmt.Errorf("no results found for artist '%s', track '%s'", artist, track)
113113
}
114114

115115
slices.Sort(years)
@@ -118,16 +118,20 @@ func (config *DiscogsConfig) GetEarliestReleaseYear(artist string, track string)
118118
}
119119

120120
func (config *DiscogsConfig) EnhanceYears(tracks *([]db.Track)) {
121-
for _, track := range *tracks {
121+
for i, track := range *tracks {
122122
dcYear, err := config.GetEarliestReleaseYear(track.Artist, track.Title)
123+
time.Sleep(1 * time.Second)
123124
if err != nil {
125+
fmt.Println("Enhance error", err)
124126
continue
125127
}
126128

127-
fmt.Printf("Track %s, org year %d, dc year %d\n", track.Title, track.Year, dcYear)
129+
fmt.Printf("Track '%s', artist '%s', org year %d, dc year %d\n", track.Title, track.Artist, track.Year, dcYear)
128130

129131
if track.Year > dcYear {
130-
track.Year = dcYear
132+
fmt.Println("DC year is better for above")
133+
(*tracks)[i].Year = dcYear
131134
}
135+
132136
}
133137
}

0 commit comments

Comments
 (0)