Skip to content

Commit

Permalink
Move implementation of secondTreeOptions() for core/staging out of th…
Browse files Browse the repository at this point in the history
…e base Source class

It's only applicable to the CoreSource and CoreStagingSource.
  • Loading branch information
tsibley committed Apr 8, 2021
1 parent 610c877 commit 10e081f
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/sources.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ class Source {
return new Narrative(this, pathParts);
}

// The computation of these globals should move here.
// eslint-disable-next-line no-unused-vars
secondTreeOptions(path) {
return (global.availableDatasets.secondTreeOptions[this.name] || {})[path] || [];
return [];
}

availableDatasets() {
Expand Down Expand Up @@ -128,6 +128,10 @@ class CoreSource extends Source {
}

// The computation of these globals should move here.
secondTreeOptions(path) {
return (global.availableDatasets.secondTreeOptions[this.name] || {})[path] || [];
}

availableDatasets() {
return global.availableDatasets[this.name] || [];
}
Expand Down

0 comments on commit 10e081f

Please sign in to comment.