Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Extracted OptionsResolver component out of Form #3968

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
May 15, 2012
Merged

Extracted OptionsResolver component out of Form #3968

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b9d053e
[Form] Moved Options classes to new OptionsParser component
webmozart Apr 17, 2012
04522ca
[OptionsParser] Added method replaceDefaults()
webmozart Apr 23, 2012
256b708
[OptionsParser] Renamed OptionsParser to OptionsResolver
webmozart May 10, 2012
95454f5
[OptionsResolver] Fixed typos
webmozart May 10, 2012
876fd9b
[OptionsResolver] Implemented fluid interface
webmozart May 11, 2012
9c76750
[OptionsResolver] Fixed doc and block nesting
webmozart May 11, 2012
6ce68b1
[OptionsResolver] Removed reference to non-existing property
webmozart May 11, 2012
16f7d20
[OptionsResolver] Improved implementation and clarity of the Options …
webmozart May 11, 2012
2b46975
[OptionsResolver] Fixed Options::replace() method
webmozart May 11, 2012
d60626e
[OptionsResolver] Fixed clear() and remove() method in Options class
webmozart May 14, 2012
1c5f6c7
[OptionsResolver] Fixed issues mentioned in the PR comments
webmozart May 14, 2012
95727ff
[OptionsResolver] Updated PHP requirements to 5.3.3
webmozart May 15, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[OptionsResolver] Fixed Options::replace() method
  • Loading branch information
@webmozart
webmozart committed May 14, 2012
commit 2b46975e322b55e255f5cc7bb1148c8d90a53458
3 changes: 2 additions & 1 deletion src/Symfony/Component/OptionsResolver/Options.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,9 @@ public function replace(array $options)
}

$this->options = array();

foreach ($options as $option => $value) {
$this->options[$option] = $value;
$this->set($option, $value);
}
}

Expand Down
17 changes: 17 additions & 0 deletions src/Symfony/Component/OptionsResolver/Tests/OptionsTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -198,4 +198,21 @@ public function testFailForCyclicDependencies()

$this->options->get('foo');
}

public function testReplaceClearsAndSets()
{
$this->options->set('one', '1');

$this->options->replace(array(
'two' => '2',
'three' => function (Options $options) {
return '2' === $options['two'] ? '3' : 'foo';
}
));

$this->assertEquals(array(
'two' => '2',
'three' => '3',
), $this->options->all());
}
}