From 3bbeea69c6356edd5792ac7898b9623c2e1519de Mon Sep 17 00:00:00 2001 From: "(quasar) nebula" Date: Sat, 5 Mar 2022 21:34:59 -0400 Subject: non-jpg full res art links, artist avatars --- src/data/cacheable-object.js | 2 +- src/data/things.js | 20 ++++++++++++++++++++ src/misc-templates.js | 2 +- src/page/album.js | 13 +++++++------ src/page/artist.js | 9 +++++---- src/page/track.js | 2 +- src/upd8.js | 11 +++++++++++ src/url-spec.js | 9 ++++----- src/util/wiki-data.js | 12 +++++++----- 9 files changed, 57 insertions(+), 23 deletions(-) diff --git a/src/data/cacheable-object.js b/src/data/cacheable-object.js index 9928095..4afb036 100644 --- a/src/data/cacheable-object.js +++ b/src/data/cacheable-object.js @@ -169,7 +169,7 @@ export default class CacheableObject { const oldValue = this.#propertyUpdateValues[property]; if (newValue === undefined) { - throw new ValueError(`Properties cannot be set to undefined`); + throw new TypeError(`Properties cannot be set to undefined`); } if (newValue === oldValue) { diff --git a/src/data/things.js b/src/data/things.js index c93b885..16970af 100644 --- a/src/data/things.js +++ b/src/data/things.js @@ -448,6 +448,9 @@ Album.propertyDescriptors = { } }, + coverArtFileExtension: Thing.common.fileExtension('jpg'), + trackCoverArtFileExtension: Thing.common.fileExtension('jpg'), + wallpaperStyle: Thing.common.simpleString(), wallpaperFileExtension: Thing.common.fileExtension('jpg'), @@ -605,6 +608,20 @@ Track.propertyDescriptors = { } }, + coverArtFileExtension: { + flags: {update: true, expose: true}, + + update: {validate: isFileExtension}, + + expose: { + dependencies: ['albumData'], + transform: (coverArtFileExtension, { albumData, [Track.instance]: track }) => ( + coverArtFileExtension ?? + Track.findAlbum(track, albumData)?.trackCoverArtFileExtension ?? + true) + } + }, + // Previously known as: (track).aka originalReleaseTrackByRef: Thing.common.singleReference(Track), @@ -779,6 +796,9 @@ Artist.propertyDescriptors = { urls: Thing.common.urls(), contextNotes: Thing.common.simpleString(), + hasAvatar: Thing.common.flag(false), + avatarFileExtension: Thing.common.fileExtension('jpg'), + aliasNames: { flags: {update: true, expose: true}, update: { diff --git a/src/misc-templates.js b/src/misc-templates.js index eb12496..300fd8a 100644 --- a/src/misc-templates.js +++ b/src/misc-templates.js @@ -182,7 +182,7 @@ export function getThemeString(color, additionalVariables = []) { } export function getAlbumStylesheet(album, {to}) { return [ - album.wallpaperArtistContribs && fixWS` + album.wallpaperArtistContribs.length && fixWS` body::before { background-image: url("${to('media.albumWallpaper', album.directory, album.wallpaperFileExtension)}"); ${album.wallpaperStyle} diff --git a/src/page/album.js b/src/page/album.js index 6478cce..d7447a4 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), - artistContribs: serializeContribs(album.artistContribs || []), - coverArtistContribs: serializeContribs(album.coverArtistContribs || []), - wallpaperArtistContribs: serializeContribs(album.wallpaperArtistContribs || []), - bannerArtistContribs: serializeContribs(album.bannerArtistContribs || []), + 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, @@ -98,6 +98,7 @@ export function write(album, {wikiData}) { fancifyURL, generateChronologyLinks, generateCoverLink, + getAlbumCover, getAlbumStylesheet, getArtistString, getLinkThemeString, @@ -120,7 +121,7 @@ export function write(album, {wikiData}) { `--album-directory: ${album.directory}` ]), - banner: album.bannerArtistContribs && { + banner: album.bannerArtistContribs.length && { dimensions: album.bannerDimensions, path: ['media.albumBanner', album.directory, album.bannerFileExtension], alt: strings('misc.alt.albumBanner'), @@ -130,7 +131,7 @@ export function write(album, {wikiData}) { main: { content: fixWS` ${generateCoverLink({ - path: ['media.albumCover', album.directory], + src: getAlbumCover(album), alt: strings('misc.alt.albumCover'), tags: album.artTags })} diff --git a/src/page/artist.js b/src/page/artist.js index 6a46560..dae0e19 100644 --- a/src/page/artist.js +++ b/src/page/artist.js @@ -272,6 +272,7 @@ export function write(artist, {wikiData}) { fancifyURL, generateCoverLink, generateInfoGalleryLinks, + getArtistAvatar, getArtistString, link, strings, @@ -290,7 +291,7 @@ export function write(artist, {wikiData}) { main: { content: fixWS` ${artist.hasAvatar && generateCoverLink({ - path: ['localized.artistAvatar', artist.directory], + src: getArtistAvatar(artist), alt: strings('misc.alt.artistAvatar') })}

${strings('artistPage.title', {artist: name})}

@@ -369,9 +370,9 @@ export function write(artist, {wikiData}) { track: link.track(track) }) : `${strings('artistPage.creditList.entry.album.' + { - wallpaperArtists: 'wallpaperArt', - bannerArtists: 'bannerArt', - coverArtists: 'coverArt' + wallpaperArtistContribs: 'wallpaperArt', + bannerArtistContribs: 'bannerArt', + coverArtistContribs: 'coverArt' }[key])}`), ...props })) diff --git a/src/page/track.js b/src/page/track.js index 264195d..9a3307b 100644 --- a/src/page/track.js +++ b/src/page/track.js @@ -149,7 +149,7 @@ export function write(track, {wikiData}) { // disabled for now! shifting banner position per height of page is disorienting /* - banner: album.bannerArtistContribs && { + banner: album.bannerArtistContribs.length && { classes: ['dim'], dimensions: album.bannerDimensions, path: ['media.albumBanner', album.directory, album.bannerFileExtension], diff --git a/src/upd8.js b/src/upd8.js index 38027ba..e946ca2 100755 --- a/src/upd8.js +++ b/src/upd8.js @@ -159,6 +159,7 @@ import { getAlbumCover, getAlbumListTag, getAllTracks, + getArtistAvatar, getArtistCommentary, getArtistNumContributions, getFlashCover, @@ -867,6 +868,9 @@ const processAlbumDocument = makeProcessDocument(Album, { coverArtistContribsByRef: 'Cover Artists', trackCoverArtistContribsByRef: 'Default Track Cover Artists', + coverArtFileExtension: 'Cover Art File Extension', + trackCoverArtFileExtension: 'Track Art File Extension', + wallpaperArtistContribsByRef: 'Wallpaper Artists', wallpaperStyle: 'Wallpaper Style', wallpaperFileExtension: 'Wallpaper File Extension', @@ -923,6 +927,7 @@ const processTrackDocument = makeProcessDocument(Track, { urls: 'URLs', coverArtDate: 'Cover Art Date', + coverArtFileExtension: 'Cover Art File Extension', dateFirstReleased: 'Date First Released', hasCoverArt: 'Has Cover Art', hasURLs: 'Has URLs', @@ -947,6 +952,8 @@ const processArtistDocument = makeProcessDocument(Artist, { directory: 'Directory', urls: 'URLs', + hasAvatar: 'Has Avatar', + avatarFileExtension: 'Avatar File Extension', aliasNames: 'Aliases', @@ -3301,6 +3308,10 @@ async function main() { to }); + bound.getArtistAvatar = bindOpts(getArtistAvatar, { + to + }); + bound.generateChronologyLinks = bindOpts(generateChronologyLinks, { link: bound.link, linkAnythingMan: bound.linkAnythingMan, diff --git a/src/url-spec.js b/src/url-spec.js index 3a77858..b9366b0 100644 --- a/src/url-spec.js +++ b/src/url-spec.js @@ -70,13 +70,12 @@ const urlSpec = { root: '', path: '<>', - albumCover: 'album-art/<>/cover.jpg', + albumCover: 'album-art/<>/cover.<>', albumWallpaper: 'album-art/<>/bg.<>', albumBanner: 'album-art/<>/banner.<>', - trackCover: 'album-art/<>/<>.jpg', - artistAvatar: 'artist-avatar/<>.jpg', - flashArt: 'flash-art/<>.jpg', - flashArtGif: 'flash-art/<>.gif' // Hack! Sorry not sorry. ::::) + trackCover: 'album-art/<>/<>.<>', + artistAvatar: 'artist-avatar/<>.<>', + flashArt: 'flash-art/<>.<>', } } }; diff --git a/src/util/wiki-data.js b/src/util/wiki-data.js index 36c6145..870c0b0 100644 --- a/src/util/wiki-data.js +++ b/src/util/wiki-data.js @@ -98,7 +98,7 @@ export function filterAlbumsByCommentary(albums) { } export function getAlbumCover(album, {to}) { - return to('media.albumCover', album.directory); + return to('media.albumCover', album.directory, album.coverArtFileExtension); } export function getAlbumListTag(album) { @@ -143,9 +143,7 @@ export function getArtistCommentary(artist, {justEverythingMan}) { } export function getFlashCover(flash, {to}) { - return (flash.jiff - ? to('media.flashArtGif', flash.directory) - : to('media.flashArt', flash.directory)); + return to('media.flashArt', flash.directory, flash.coverArtFileExtension); } export function getFlashLink(flash) { @@ -162,10 +160,14 @@ export function getTrackCover(track, {to}) { if (!track.hasCoverArt) { return getAlbumCover(track.album, {to}); } else { - return to('media.trackCover', track.album.directory, track.directory); + return to('media.trackCover', track.album.directory, track.directory, track.coverArtFileExtension); } } +export function getArtistAvatar(artist, {to}) { + return to('media.artistAvatar', artist.directory, artist.avatarFileExtension); +} + // Big-ass homepage row functions export function getNewAdditions(numAlbums, {wikiData}) { -- cgit 1.3.0-6-gf8a5