« get me outta code hell

data: withReverseContributionList: drop 'mode' option - hsmusic-wiki - HSMusic - static wiki software cataloguing collaborative creation
about summary refs log tree commit diff
path: root/src/data/composite
diff options
context:
space:
mode:
author(quasar) nebula <qznebula@protonmail.com>2024-06-02 11:04:13 -0300
committer(quasar) nebula <qznebula@protonmail.com>2024-06-12 17:26:54 -0300
commit1ade1832435c0f03c625def19ec8a955c56cb943 (patch)
tree24b02eaf9bea6243ea371b8fb15a431c17d449a6 /src/data/composite
parentf65746589e42f1dd74673247ff2d9ca8718c8f82 (diff)
data: withReverseContributionList: drop 'mode' option
Diffstat (limited to 'src/data/composite')
-rw-r--r--src/data/composite/things/artist/artistTotalDuration.js2
-rw-r--r--src/data/composite/wiki-data/withReverseContributionList.js28
-rw-r--r--src/data/composite/wiki-properties/reverseContributionList.js7
3 files changed, 2 insertions, 35 deletions
diff --git a/src/data/composite/things/artist/artistTotalDuration.js b/src/data/composite/things/artist/artistTotalDuration.js
index 54204918..ff709f28 100644
--- a/src/data/composite/things/artist/artistTotalDuration.js
+++ b/src/data/composite/things/artist/artistTotalDuration.js
@@ -14,7 +14,6 @@ export default templateCompositeFrom({
     withReverseContributionList({
       data: 'trackData',
       list: input.value('artistContribs'),
-      mode: input.value('contributions'),
     }).outputs({
       '#reverseContributionList': '#contributionsAsArtist',
     }),
@@ -22,7 +21,6 @@ export default templateCompositeFrom({
     withReverseContributionList({
       data: 'trackData',
       list: input.value('contributorContribs'),
-      mode: input.value('contributions'),
     }).outputs({
       '#reverseContributionList': '#contributionsAsContributor',
     }),
diff --git a/src/data/composite/wiki-data/withReverseContributionList.js b/src/data/composite/wiki-data/withReverseContributionList.js
index 99d0c73b..b7f7a95b 100644
--- a/src/data/composite/wiki-data/withReverseContributionList.js
+++ b/src/data/composite/wiki-data/withReverseContributionList.js
@@ -1,9 +1,6 @@
 // Analogous implementation for withReverseReferenceList, for contributions.
 // This is mostly duplicate code and both should be ported to the same
-// underlying data form later on. Unique to contributions, the 'mode' option
-// controls whether the things themselves, for which the artist is credited,
-// are exposed (the default), or the actual contribution objects representing
-// the relationship itself, instead.
+// underlying data form later on.
 //
 // This implementation uses a global cache (via WeakMap) to attempt to speed
 // up subsequent similar accesses.
@@ -13,7 +10,6 @@
 // is used, a fresh cache will always be created.
 
 import {input, templateCompositeFrom} from '#composite';
-import {is} from '#validators';
 
 import {exitWithoutDependency, raiseOutputWithoutDependency}
   from '#composite/control-flow';
@@ -32,11 +28,6 @@ export default templateCompositeFrom({
   inputs: {
     data: inputWikiData({allowMixedTypes: false}),
     list: input({type: 'string'}),
-
-    mode: input({
-      validate: is('things', 'contributions'),
-      defaultValue: 'things',
-    }),
   },
 
   outputs: ['#reverseContributionList'],
@@ -90,25 +81,10 @@ export default templateCompositeFrom({
         }
 
         return continuation({
-          ['#contributions']:
+          ['#reverseContributionList']:
             cache.get(data).get(myself) ?? [],
         });
       },
     },
-
-    {
-      dependencies: ['#contributions', input('mode')],
-      compute: (continuation, {
-        ['#contributions']: contributions,
-        [input('mode')]: mode,
-      }) => continuation({
-        ['#reverseContributionList']:
-          (mode === 'contributions'
-            ? contributions
-         : mode === 'things'
-            ? contributions.map(contrib => contrib.thing)
-            : []),
-      }),
-    },
   ],
 });
diff --git a/src/data/composite/wiki-properties/reverseContributionList.js b/src/data/composite/wiki-properties/reverseContributionList.js
index ce219377..7f3f9c81 100644
--- a/src/data/composite/wiki-properties/reverseContributionList.js
+++ b/src/data/composite/wiki-properties/reverseContributionList.js
@@ -1,5 +1,4 @@
 import {input, templateCompositeFrom} from '#composite';
-import {is} from '#validators';
 
 import {exposeDependency} from '#composite/control-flow';
 import {inputWikiData, withReverseContributionList} from '#composite/wiki-data';
@@ -12,18 +11,12 @@ export default templateCompositeFrom({
   inputs: {
     data: inputWikiData({allowMixedTypes: false}),
     list: input({type: 'string'}),
-
-    mode: input({
-      validate: is('things', 'contributions'),
-      defaultValue: 'things',
-    }),
   },
 
   steps: () => [
     withReverseContributionList({
       data: input('data'),
       list: input('list'),
-      mode: input('mode'),
     }),
 
     exposeDependency({dependency: '#reverseContributionList'}),