Revert "Completed implementation of HED validator"

hed-standard · Dec 23, 2024 · d430b66 · d430b66
1 parent afe0e3e
commit d430b66
Show file tree

Hide file tree

Showing 74 changed files with 15,096 additions and 6,480 deletions.
diff --git a/bids/schema.js b/bids/schema.js
@@ -2,7 +2,7 @@ import castArray from 'lodash/castArray'
 import semver from 'semver'
 
 import { buildSchemas } from '../schema/init'
-import { IssueError } from '../common/issues/issues'
+import { generateIssue, IssueError } from '../common/issues/issues'
 import { SchemaSpec, SchemasSpec } from '../schema/specs'
 
 const alphabeticRegExp = new RegExp('^[a-zA-Z]+$')

diff --git a/bids/types/basic.js b/bids/types/basic.js
@@ -1,5 +1,4 @@
 import { BidsHedIssue } from './issues'
-import { generateIssue } from '../../common/issues/issues'
 
 /**
  * A BIDS file.
@@ -37,27 +36,15 @@ export class BidsFile {
     return false
   }
 
-  /**
-   * Validate this validator's tsv file
-   *
-   * @param {Schemas} schemas
-   * @returns {BidsIssue[]} Any issues found during validation of this TSV file.
-   */
-  validate(schemas) {
+  validate(hedSchemas) {
     if (!this.hasHedData()) {
       return []
-    }
-    if (!schemas) {
-      BidsHedIssue.fromHedIssue(
-        generateIssue('genericError', {
-          message: 'BIDS file HED validation requires a HED schema, but the schema received was null.',
-        }),
-        { path: this.file.file, relativePath: this.file.file },
-      )
+    } else if (hedSchemas === null) {
+      return null
     }
 
     try {
-      const validator = new this.validatorClass(this, schemas)
+      const validator = new this.validatorClass(this, hedSchemas)
       return validator.validate()
     } catch (internalError) {
       return BidsHedIssue.fromHedIssues(internalError, this.file)

diff --git a/bids/types/dataset.js b/bids/types/dataset.js
@@ -1,3 +1,5 @@
+import { fallbackDatasetDescription } from './json'
+
 export class BidsDataset {
   /**
    * The dataset's event file data.
@@ -20,7 +22,7 @@ export class BidsDataset {
    */
   datasetRootDirectory
 
-  constructor(eventData, sidecarData, datasetDescription, datasetRootDirectory = null) {
+  constructor(eventData, sidecarData, datasetDescription = fallbackDatasetDescription, datasetRootDirectory = null) {
     this.eventData = eventData
     this.sidecarData = sidecarData
     this.datasetDescription = datasetDescription

diff --git a/bids/types/issues.js b/bids/types/issues.js
@@ -38,7 +38,7 @@ export class BidsIssue {
    * Determine if any of the passed issues are errors.
    *
    * @param {BidsIssue[]} issues A list of issues.
-   * @returns {boolean} Whether any of the passed issues are errors (rather than warnings).
+   * @return {boolean} Whether any of the passed issues are errors (rather than warnings).
    */
   static anyAreErrors(issues) {
     return issues.some((issue) => issue.isError())