diff --git a/src/controllers/itemDetails/index.js b/src/controllers/itemDetails/index.js index 18388a6459a8..b9c607d83ae2 100644 --- a/src/controllers/itemDetails/index.js +++ b/src/controllers/itemDetails/index.js @@ -386,25 +386,17 @@ function getArtistLinksHtml(artists, serverId, context) { const html = []; const numberOfArtists = artists.length; - if (numberOfArtists < 10) { - for (const artist of artists) { - const href = appRouter.getRouteUrl(artist, { - context: context, - itemType: 'MusicArtist', - serverId: serverId - }); - html.push('' + escapeHtml(artist.Name) + ''); - } - } else { - for (let i = 0; i < 10; i++) { - const artist = artists[i]; - const href = appRouter.getRouteUrl(artist, { - context: context, - itemType: 'MusicArtist', - serverId: serverId - }); - html.push('' + escapeHtml(artist.Name) + ''); - } + for (let i = 0; i < Math.min(numberOfArtists, 10); i++) { + const artist = artists[i]; + const href = appRouter.getRouteUrl(artist, { + context, + itemType: 'MusicArtist', + serverId + }); + html.push('' + escapeHtml(artist.Name) + ''); + } + + if (numberOfArtists > 10) { const remainingNumberOfArtists = numberOfArtists - 10; html.push(`${globalize.translate('AndOtherArtists', remainingNumberOfArtists)}`); }