@museumwnf/viewer-core 2.6.0 → 2.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@museumwnf/viewer-core",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "description": "Application engine for MWNF websites: turns a dataset.config.js into a mounted Vue application.",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",

package/src/catalogue/fieldSearch.js

@@ -28,7 +28,7 @@ export function searchFields({ dynastyLabel } = {}) {
     location: (item, text) => [text.location, item.country_id],
     provenance: (item, text) => text.provenance,
     patron: (item, text) => text.patrons ?? text.initial_owner,
-    artist: (item, text) => [...(item.artist_names ?? []), text.architects],
+    artist: (item, text) => [...(item.artist_names ?? []), text.artist, text.architects],
     material: (item, text) => text.type,
     // The catch-all across the descriptive fields the others leave out.
     other: (item, text) => [

package/src/dxa/exhibition/useExhibitionSheet.js

@@ -27,7 +27,7 @@
 // dynasties stand in when it has none, and keep their own popouts
 // (decision D5, inventory-app#2233).
 
-import { artistLine } from '../../record/derivations.js'
+import { artistLine, recordArtist } from '../../record/derivations.js'
 
 /**
  * The exhibition's item-sheet spec, over its data layer's `data`.
@@ -61,7 +61,7 @@ export function useExhibitionSheet(data, config = {}) {
     { key: 'originalOwner', label: 'sheet.field.originalOwner', value: 'initial_owner' },
     { key: 'currentOwner', label: 'sheet.field.currentOwner', value: 'owner' },
     { key: 'date', label: 'sheet.field.date', value: 'dates' },
-    { key: 'artist', label: 'sheet.field.artists', value: (c) => artistLine((c.record.artist_names ?? []).join(', '), c.text, { style: 'line' }) },
+    { key: 'artist', label: 'sheet.field.artists', value: (c) => artistLine(recordArtist(c.record, c.text), c.text, { style: 'line' }) },
     { key: 'scribe', label: 'sheet.field.scribe', value: 'scriber' },
     { key: 'workshop', label: 'sheet.field.workshop', value: 'workshop' },
     { key: 'type', label: 'sheet.field.type', value: 'type' },

package/src/dxa/exhibition/useExhibitionThemes.js

@@ -2,6 +2,7 @@ import { computed } from 'vue'
 import { useI18n } from '../../i18n/index.js'
 import { entityRef } from '../../composables/useEntities.js'
 import { useCollectionTree } from '../../record/collectionTree.js'
+import { recordArtist } from '../../record/derivations.js'
 
 // An exhibition's themes: the tour its theme pages, its themes list and its
 // theme galleries read (inventory-app#2053). The six exhibitions each carried
@@ -240,7 +241,8 @@ export function useExhibitionThemes(data) {
       if (sheet.alternate_name) {
         fields.push({ label: `${t('sheet.field.alsoKnownAs')}:`, value: mdInline(sheet.alternate_name) })
       }
-      if (parent.artist_names?.length) fields.push({ label: '', value: parent.artist_names.join(', ') })
+      const artist = recordArtist(parent, sheet)
+      if (artist) fields.push({ label: '', value: artist })
       if (sheet.dates) fields.push({ label: '', value: sheet.dates })
       return fields
     }

package/src/dxa/gallery/useGallerySheet.js

@@ -27,7 +27,7 @@
 // dynasties stand in when it has none, and keep their own popouts
 // (decision D5, inventory-app#2233).
 
-import { artistLine } from '../../record/derivations.js'
+import { artistLine, recordArtist } from '../../record/derivations.js'
 
 /**
  * The gallery's item-sheet spec, over its data layer's `data`.
@@ -61,7 +61,7 @@ export function useGallerySheet(data, config = {}) {
     { key: 'originalOwner', label: 'sheet.field.originalOwner', value: 'initial_owner' },
     { key: 'currentOwner', label: 'sheet.field.currentOwner', value: 'owner' },
     { key: 'date', label: 'sheet.field.date', value: 'dates' },
-    { key: 'artist', label: 'sheet.field.artists', value: (c) => artistLine((c.record.artist_names ?? []).join(', '), c.text, { style: 'line' }) },
+    { key: 'artist', label: 'sheet.field.artists', value: (c) => artistLine(recordArtist(c.record, c.text), c.text, { style: 'line' }) },
     { key: 'scribe', label: 'sheet.field.scribe', value: 'scriber' },
     { key: 'workshop', label: 'sheet.field.workshop', value: 'workshop' },
     { key: 'type', label: 'sheet.field.type', value: 'type' },

package/src/index.js

@@ -39,7 +39,7 @@ export {
   glossaryEntries, glossaryTermsFor, glossaryTermsForText, sheetRows, useRecordSheet,
 } from './record/recordSheet.js'
 export {
-  artistLine, citation, relatedRecords, removeEndingPoint, searchGlossary, timelineLinkFor, useGlossaryPopup,
+  artistLine, citation, recordArtist, relatedRecords, removeEndingPoint, searchGlossary, timelineLinkFor, useGlossaryPopup,
   useRelatedRecords,
 } from './record/derivations.js'
 export { buildCollectionTree, collectionTreeFromThemes, useCollectionTree } from './record/collectionTree.js'

package/src/record/derivations.js

@@ -181,6 +181,16 @@ export function removeEndingPoint(text) {
 
 const lifePart = (value) => (typeof value === 'string' ? value.trim() : '')
 
+/**
+ * The artist the sheet names: the translation's own `artist` when it has one,
+ * else the record's `artist_names`. A Sharing History record has no artist
+ * entity, and legacy wrote its artist per language ("Unknown", "Inconnu"),
+ * so the package carries it on each translation (inventory-app#2233, C22).
+ */
+export function recordArtist(record, text) {
+  return lifePart(text?.artist) || (record?.artist_names ?? []).join(', ')
+}
+
 /**
  * The artist line as Markdown: `artist`, then the artist's life data the
  * record's translation carries (`artist_birthdate`, `artist_birthplace`,