Skip to content

Commit

Permalink
Merge pull request #111 from anare/master
Browse files Browse the repository at this point in the history
Fixing OptionsResolver with field disabled in callbacks
  • Loading branch information
yohang committed Dec 23, 2015
2 parents 5c00313 + fe98909 commit dc53d48
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ protected function removeDisabledCallbacks(array $config)
if ($callback['disabled']) {
unset($config['callbacks'][$position][$i]);
}
unset($config['callbacks'][$position][$i]['disabled']);
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,10 +86,10 @@ private function getExpectedConfig()
),
'callbacks' => array(
'before' => array(
'callback1' => array('on' => '1_to_2', 'do' => array('@my.listener.service', 'on1To2'), 'disabled' => false)
'callback1' => array('on' => '1_to_2', 'do' => array('@my.listener.service', 'on1To2'))
),
'after' => array(
'callback2' => array('from' => '-state3', 'to' => array('state2', 'state3'), 'do' => array('@my.listener.service', 'on1To2'), 'disabled' => false)
'callback2' => array('from' => '-state3', 'to' => array('state2', 'state3'), 'do' => array('@my.listener.service', 'on1To2'))
)
)
);
Expand Down

0 comments on commit dc53d48

Please sign in to comment.