Merge pull request #76 from mirpedrol/file-validation-if-false

Don't validate a file if the parameter evaluates to false
nextflow-io · Jul 12, 2023 · ef90632 · ef90632
2 parents 4fdbb3c + ee818b7
commit ef90632
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 ### Bug fixes
 
 - Don't check if path exists if param is not true ([#74](https://github.com/nextflow-io/nf-validation/pull/74))
+- Don't validate a file if the parameter evaluates to false ([#75](https://github.com/nextflow-io/nf-validation/pull/75))
 
 ## Version 0.3.0
 

diff --git a/docs/nextflow_schema/nextflow_schema_specification.md b/docs/nextflow_schema/nextflow_schema_specification.md
@@ -409,6 +409,10 @@ These schema files are typically stored in the pipeline `assets` directory, but
 }
 ```
 
+!!! note
+
+    If the parameter is set to `null`, `false` or an empty string, this validation is ignored. The file won't be validated.
+
 ## Numeric-specific keys
 
 ### `minimum`, `maximum`

diff --git a/plugins/nf-validation/src/main/nextflow/validation/SchemaValidator.groovy b/plugins/nf-validation/src/main/nextflow/validation/SchemaValidator.groovy
@@ -370,7 +370,7 @@ class SchemaValidator extends PluginExtensionPoint {
             def Map properties = (Map) group.value['properties']
             for (p in properties) {
                 def String key = (String) p.key
-                if (params[key] == null || params[key] == '') {
+                if (!params[key]) {
                     continue
                 }
                 def Map property = properties[key] as Map