keboola / php-component

General library for PHP applications running in Keboola Connection environment
MIT License
0 stars 1 forks source link

Chybějící parameters v config.json #26

Closed ondrejhlavacek closed 6 years ago

ondrejhlavacek commented 6 years ago

ConfigDefinition.php::getParametersDefinition

        $parametersNode = parent::getParametersDefinition();
        // @formatter:off
        /** @noinspection NullPointerExceptionInspection */
        $parametersNode
            ->children()
                ->scalarNode('mask')
                    ->isRequired()
                ->end()
            ->end()
        ;
        // @formatter:on
        return $parametersNode;

Prázdný config.json

config.json

{
}

=> Appka proběhne

Prázdné paramters v config.json

config.json

{
  "parameters": {}
}

Appka hodí error


Čekal bych, že to hodí error pokaždý.

tomasfejfar commented 6 years ago

Imho bys musel na tom parameters node zavolat isRequired nejdřív. Jinak je nepovinný ;)

ondrejhlavacek commented 6 years ago

I see! A co bys doporučil? Mám tam poslat isRequired(), když pouštění bez parametrů nemá význam?

tomasfejfar commented 6 years ago

Jo, tak bych to udělal.

ondrejhlavacek commented 6 years ago

Super! Funguje to, díky.