Skip to content

Commit

Permalink
Merge pull request #40 from kafene/master
Browse files Browse the repository at this point in the history
When an optional parameter has no devault value available, set it required
  • Loading branch information
rantonmattei committed Nov 24, 2015
2 parents 065eaed + 8accb36 commit a3b0a68
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ public function getReflected()
protected function configureOptions(OptionsResolver $optionsResolver)
{
foreach ($this->constructorArgs as $name => $param) {
if ($param->isOptional()) {
if ($param->isOptional() && $param->isDefaultValueAvailable()) {
$optionsResolver->setDefault($name, $param->getDefaultValue());
} else {
$optionsResolver->setRequired($name);
Expand Down

0 comments on commit a3b0a68

Please sign in to comment.