Skip to content

Commit a735d61

Browse files
committed
Removed fetchedFrom from other fetchers
1 parent 919bf51 commit a735d61

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

src/rpc/format.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,9 @@ const fetchers = {
3939
if (result.data.resultCount > 0 && result.data.results[0] !== undefined)
4040
{
4141
return {
42-
fetchedFrom: "Apple Music",
42+
artworkFrom: "Apple Music",
4343
artworkUrl: result.data.results[0].artworkUrl100,
44+
joinFrom: "Apple Music",
4445
joinUrl: result.data.results[0].trackViewUrl
4546
};
4647
}
@@ -69,8 +70,9 @@ const fetchers = {
6970
if (result.items.length > 0)
7071
{
7172
return {
72-
fetchedFrom: "Youtube",
73+
artworkFrom: undefined,
7374
artworkUrl: undefined,
75+
joinFrom: "Youtube",
7476
joinUrl: result.items[0].url
7577
};
7678
}
@@ -81,18 +83,14 @@ const fetchers = {
8183
* Safe wrapper for calling a fetcher in try-catch block
8284
* @param {string} fetcherName name of the fetcher to use
8385
* @param {Object} metadata VLC metadata
84-
* @returns {!{artworkFrom: string, artworkUrl: string, joinFrom: string, joinUrl: string}}
86+
* @returns {!{artworkFrom: ?string, artworkUrl: ?string, joinFrom: ?string, joinUrl: ?string}}
8587
*/
8688
async function fetchSafely(fetcherName, metadata)
8789
{
8890
let returnValue;
8991
try
9092
{
9193
returnValue = await fetchers[fetcherName](metadata);
92-
if (returnValue && fetcherName !== "staticoverrides")
93-
{
94-
returnValue.artworkFrom = returnValue.joinFrom = returnValue.fetchedFrom;
95-
}
9694
}
9795
catch (err)
9896
{
@@ -109,7 +107,7 @@ async function fetchSafely(fetcherName, metadata)
109107
* @param {string} preferredArtworkProvider name of preferred artwork fetcher
110108
* @param {string} preferredJoinProvider name of preferred join fetcher
111109
* @param {Object} metadata VLC metadata
112-
* @returns {!{artworkFrom: string, artworkUrl: string, joinFrom: string, joinUrl: string}}
110+
* @returns {!{artworkFrom: ?string, artworkUrl: ?string, joinFrom: ?string, joinUrl: ?string}}
113111
*/
114112
async function combinedFetch(preferredArtworkProvider, preferredJoinProvider, metadata)
115113
{

src/rpc/musichoardersFetcher.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ class MusicHoardersFetcher
8686
* Fetch artwork and join URLs if not cached, otherwise return cached
8787
* @param {string} service service name, e.g. "spotify"
8888
* @param {Object} metadata VLC metadata
89-
* @returns {?{fetchedFrom: string, artworkUrl: string, joinUrl: string}}
89+
* @returns {?{artworkFrom: ?string, artworkUrl: ?string, joinFrom: ?string, joinUrl: ?string}}
9090
*/
9191
async fetch(service, metadata)
9292
{
@@ -152,8 +152,9 @@ class MusicHoardersFetcher
152152
for (let serviceKey in bestResults)
153153
{
154154
bestResults[serviceKey] = {
155-
fetchedFrom: MusicHoardersFetcher.#services[bestResults[serviceKey].album.source],
155+
artworkFrom: MusicHoardersFetcher.#services[bestResults[serviceKey].album.source],
156156
artworkUrl: bestResults[serviceKey].album.smallCoverUrl,
157+
joinFrom: MusicHoardersFetcher.#services[bestResults[serviceKey].album.source],
157158
joinUrl: bestResults[serviceKey].album.releaseInfo.url,
158159
};
159160
};

0 commit comments

Comments
 (0)