From e6978019a002e59db8dc312b9c2caa3c2ee9d9fe Mon Sep 17 00:00:00 2001 From: "(quasar) nebula" Date: Sat, 5 Mar 2022 20:35:39 -0400 Subject: fix cover artist info, chronology links --- src/data/things.js | 5 ++++- src/page/album.js | 32 ++++++++++++++++---------------- src/page/flash.js | 10 +++++----- src/page/track.js | 8 ++++---- 4 files changed, 29 insertions(+), 26 deletions(-) diff --git a/src/data/things.js b/src/data/things.js index bc4cf3a..901442d 100644 --- a/src/data/things.js +++ b/src/data/things.js @@ -477,8 +477,11 @@ Album.propertyDescriptors = { // Expose only - // Previously known as: (album).artists artistContribs: Thing.common.dynamicContribs('artistContribsByRef'), + coverArtistContribs: Thing.common.dynamicContribs('coverArtistContribsByRef'), + trackCoverArtistContribs: Thing.common.dynamicContribs('trackCoverArtistContribsByRef'), + wallpaperArtistContribs: Thing.common.dynamicContribs('wallpaperArtistContribsByRef'), + bannerArtistContribs: Thing.common.dynamicContribs('bannerArtistContribsByRef'), commentatorArtists: Thing.common.commentatorArtists(), diff --git a/src/page/album.js b/src/page/album.js index 1ebfdec..23a974c 100644 --- a/src/page/album.js +++ b/src/page/album.js @@ -74,10 +74,10 @@ export function write(album, {wikiData}) { duration: albumDuration, color: album.color, cover: serializeCover(album, getAlbumCover), - artists: serializeContribs(album.artists || []), - coverArtists: serializeContribs(album.coverArtists || []), - wallpaperArtists: serializeContribs(album.wallpaperArtists || []), - bannerArtists: serializeContribs(album.bannerArtists || []), + artistContribs: serializeContribs(album.artistContribs || []), + coverArtistContribs: serializeContribs(album.coverArtistContribs || []), + wallpaperArtistContribs: serializeContribs(album.wallpaperArtistContribs || []), + bannerArtistContribs: serializeContribs(album.bannerArtistContribs || []), groups: serializeGroupsForAlbum(album), trackGroups: album.trackGroups?.map(trackGroup => ({ name: trackGroup.name, @@ -137,26 +137,26 @@ export function write(album, {wikiData}) {

${strings('albumPage.title', {album: album.name})}

${[ - album.artists && strings('releaseInfo.by', { - artists: getArtistString(album.artists, { + album.artistContribs.length && strings('releaseInfo.by', { + artists: getArtistString(album.artistContribs, { showContrib: true, showIcons: true }) }), - album.coverArtists && strings('releaseInfo.coverArtBy', { - artists: getArtistString(album.coverArtists, { + album.coverArtistContribs.length && strings('releaseInfo.coverArtBy', { + artists: getArtistString(album.coverArtistContribs, { showContrib: true, showIcons: true }) }), - album.wallpaperArtists && strings('releaseInfo.wallpaperArtBy', { - artists: getArtistString(album.wallpaperArtists, { + album.wallpaperArtistContribs.length && strings('releaseInfo.wallpaperArtBy', { + artists: getArtistString(album.wallpaperArtistContribs, { showContrib: true, showIcons: true }) }), - album.bannerArtists && strings('releaseInfo.bannerArtBy', { - artists: getArtistString(album.bannerArtists, { + album.bannerArtistContribs.length && strings('releaseInfo.bannerArtBy', { + artists: getArtistString(album.bannerArtistContribs, { showContrib: true, showIcons: true }) @@ -399,14 +399,14 @@ export function generateAlbumNavLinks(album, currentTrack, { export function generateAlbumChronologyLinks(album, currentTrack, {generateChronologyLinks}) { return [ currentTrack && generateChronologyLinks(currentTrack, { - contribKey: 'artists', - getThings: artist => [...artist.tracks.asArtist, ...artist.tracks.asContributor], + contribKey: 'artistContribs', + getThings: artist => [...artist.tracksAsArtist, ...artist.tracksAsContributor], headingString: 'misc.chronology.heading.track' }), generateChronologyLinks(currentTrack || album, { - contribKey: 'coverArtists', + contribKey: 'coverArtistContribs', dateKey: 'coverArtDate', - getThings: artist => [...artist.albums.asCoverArtist, ...artist.tracks.asCoverArtist], + getThings: artist => [...artist.albumsAsCoverArtist, ...artist.tracksAsCoverArtist], headingString: 'misc.chronology.heading.coverArt' }) ].filter(Boolean).join('\n'); diff --git a/src/page/flash.js b/src/page/flash.js index 8f6e4f8..02b20c5 100644 --- a/src/page/flash.js +++ b/src/page/flash.js @@ -55,15 +55,15 @@ export function write(flash, {wikiData}) { ...flash.urls ?? [] ].map(url => fancifyFlashURL(url, flash))) })}

`} - ${flash.tracks?.length && fixWS` + ${flash.featuredTracks && fixWS`

Tracks featured in ${flash.name.replace(/\.$/, '')}: