« get me outta code hell

data, test: boldfaceArtist -> artistDisplayText - hsmusic-wiki - HSMusic - static wiki software cataloguing collaborative creation
about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
author(quasar) nebula <qznebula@protonmail.com>2023-11-14 23:36:37 -0400
committer(quasar) nebula <qznebula@protonmail.com>2023-11-14 23:36:37 -0400
commitf754a8d9187e435a761db31b5053aa2e7ba22e13 (patch)
tree0c221afa834b794aa3de92bceb093e80c087d4b2 /src
parent362dc0619b93d74ad34df1bfbfd9ebc632fa5156 (diff)
data, test: boldfaceArtist -> artistDisplayText
Diffstat (limited to 'src')
-rw-r--r--src/data/composite/wiki-data/withParsedCommentaryEntries.js21
-rw-r--r--src/util/wiki-data.js12
2 files changed, 14 insertions, 19 deletions
diff --git a/src/data/composite/wiki-data/withParsedCommentaryEntries.js b/src/data/composite/wiki-data/withParsedCommentaryEntries.js
index 9e33cdac..7b1c9484 100644
--- a/src/data/composite/wiki-data/withParsedCommentaryEntries.js
+++ b/src/data/composite/wiki-data/withParsedCommentaryEntries.js
@@ -87,7 +87,7 @@ export default templateCompositeFrom({
       prefix: input.value('#entries'),
       properties: input.value([
         'artistReference',
-        'boldfaceArtist',
+        'artistDisplayText',
         'annotation',
         'date',
       ]),
@@ -105,15 +105,10 @@ export default templateCompositeFrom({
       '#resolvedReferenceList': '#entries.artist',
     }),
 
-    {
-      dependencies: ['#entries.boldfaceArtist'],
-      compute: (continuation, {
-        ['#entries.boldfaceArtist']: boldfaceArtist,
-      }) => continuation({
-        ['#entries.boldfaceArtist']:
-          boldfaceArtist.map(boldface => boldface ? true : false),
-      }),
-    },
+    fillMissingListItems({
+      list: '#entries.artistDisplayText',
+      fill: input.value(null),
+    }),
 
     fillMissingListItems({
       list: '#entries.annotation',
@@ -133,7 +128,7 @@ export default templateCompositeFrom({
     {
       dependencies: [
         '#entries.artist',
-        '#entries.boldfaceArtist',
+        '#entries.artistDisplayText',
         '#entries.annotation',
         '#entries.date',
         '#entries.body',
@@ -141,7 +136,7 @@ export default templateCompositeFrom({
 
       compute: (continuation, {
         ['#entries.artist']: artist,
-        ['#entries.boldfaceArtist']: boldfaceArtist,
+        ['#entries.artistDisplayText']: artistDisplayText,
         ['#entries.annotation']: annotation,
         ['#entries.date']: date,
         ['#entries.body']: body,
@@ -149,7 +144,7 @@ export default templateCompositeFrom({
         ['#parsedCommentaryEntries']:
           stitchArrays({
             artist,
-            boldfaceArtist,
+            artistDisplayText,
             annotation,
             date,
             body,
diff --git a/src/util/wiki-data.js b/src/util/wiki-data.js
index 5ab01225..75a141d3 100644
--- a/src/util/wiki-data.js
+++ b/src/util/wiki-data.js
@@ -629,9 +629,10 @@ export function sortFlashesChronologically(data, {
 
 // Specific data utilities
 
-// Matches heading details from commentary data in roughly the format:
+// Matches heading details from commentary data in roughly the formats:
 //
 //    <i>artistReference:</i> (annotation, date)
+//    <i>artistReference|artistDisplayText:</i> (annotation, date)
 //
 // where capturing group "annotation" can be any text at all, except that the
 // last entry (past a comma or the only content within parentheses), if parsed
@@ -643,16 +644,15 @@ export function sortFlashesChronologically(data, {
 //   * "12/25/2019" - one or two number digits, a slash, one or two number
 //     digits, a slash, and two to four number digits
 //
-// The artist reference can optionally be boldface (in <b></b>), which will be
-// captured as non-null in "boldfaceArtist". Otherwise it is all the characters
-// between <i> and </i> and is captured in "artistReference" and is either the
-// name of an artist or an "artist:directory"-style reference.
+// Capturing group "artistReference" is all the characters between <i> and </i>
+// (apart from the pipe and "artistDisplayText" text, if present), and is either
+// the name of an artist or an "artist:directory"-style reference.
 //
 // This regular expression *doesn't* match bodies, which will need to be parsed
 // out of the original string based on the indices matched using this.
 //
 export const commentaryRegex =
-  /^<i>(?<boldfaceArtist><b>)?(?<artistReference>.+):(?:<\/b>)?<\/i>(?: \((?<annotation>(?:.*?(?=[,)]))*?)(?:,? ?(?<date>[0-9]{1,2} [^,]*[0-9]{4,4}|[0-9]{1,2}\/[0-9]{1,2}\/[0-9]{2,4}))?\))?/gm;
+  /^<i>(?<artistReference>.+?)(?:\|(?<artistDisplayText>.+))?:<\/i>(?: \((?<annotation>(?:.*?(?=[,)]))*?)(?:,? ?(?<date>[a-zA-Z]+ [0-9]{1,2}, [0-9]{4,4}|[0-9]{1,2} [^,]*[0-9]{4,4}|[0-9]{1,2}\/[0-9]{1,2}\/[0-9]{2,4}))?\))?/gm;
 
 export function filterAlbumsByCommentary(albums) {
   return albums