Merge pull request #89 from mirpedrol/empty-params

fix error when a parameter is not present in the schema and evaluates to false

Author: ewels

CHANGELOG.md

@@ -1,8 +1,11 @@
 # nextflow-io/nf-validation: Changelog
 
-# Version 0.4.0
+## Version 0.4.0 (dev)
+
+### Bug fixes
 
 - Add parameters defined on the top level of the schema and within the definitions section as expected params ([#79](https://github.com/nextflow-io/nf-validation/pull/79))
+- Fix error when a parameter is not present in the schema and evaluates to false ([#89](https://github.com/nextflow-io/nf-validation/pull/89))
 - Changed the `schema_filename` option of `fromSamplesheet` to `parameters_schema` to make this option more clear to the user ([#91](https://github.com/nextflow-io/nf-validation/pull/91))
 
 ## Version 0.3.1

plugins/nf-validation/src/main/nextflow/validation/SchemaValidator.groovy

@@ -320,9 +320,9 @@ class SchemaValidator extends PluginExtensionPoint {
             def isCamelCaseBug = (specifiedParam.contains("-") && !expectedParams.contains(specifiedParam) && expectedParamsLowerCase.contains(specifiedParamLowerCase))
             if (!expectedParams.contains(specifiedParam) && !params_ignore.contains(specifiedParam) && !isCamelCaseBug) {
                 if (failUnrecognisedParams) {
-                    errors << "* --${specifiedParam}: ${paramsJSON[specifiedParam]}".toString()
+                    errors << "* --${specifiedParam}: ${params[specifiedParam]}".toString()
                 } else {
-                    warnings << "* --${specifiedParam}: ${paramsJSON[specifiedParam]}".toString()
+                    warnings << "* --${specifiedParam}: ${params[specifiedParam]}".toString()
                 }
             }
         }