[HttpKernel] Add DI extension configs as ressources when possible
Fix problem when 1 column identifier in propel is a string.

The ModelChoiceList thinks it can be used as the index of the ChoiceList, when it can only be used if it is an integer.
#2042 initial implementation of fatal error handler
fix phpdoc in ExecutionContextInterface
fixed previous merge
Fix namespace of Validator and BrowserKit Tests
merged branch franmomu/fix_validator_test_namespace (PR #6177)

This PR was submitted for the master branch but it was merged into the 2.1 branch instead (closes #6177).

Commits
-------

964074a Fix namespace of Validator and BrowserKit Tests

Discussion
----------

Fix namespace of Validator and BrowserKit Tests

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Todo: -
License of the code: MIT
made ExecutionContext more consistent
merged branch Tobion/getPropertyPath (PR #6178)

This PR was merged into the master branch.

Commits
-------

3d0c70e made ExecutionContext more consistent

Discussion
----------

[Validator] made ExecutionContext more consistent

BC break: no

The default should be an empty string instead of null because
1. a string is expected according to phpdoc
2. its more consistent with `validate($value, $groups = null, $subPath = '', $traverse = false, $deep = false)` and `validateValue($value, $constraints, $groups = null, $subPath = '')` that both have `''` as default for subPath

---------------------------------------------------------------------------

by bschussek at 2012-12-03T21:14:49Z

:+1:
Allow any callable to be passed to Command::setCode
merged branch igorw/callable-set-code (PR #6179)

This PR was submitted for the 2.1 branch but it was merged into the master branch instead (closes #6179).

Commits
-------

31911c2 [Console] Allow any callable to be passed to Command::setCode

Discussion
----------

[Console] Allow any callable to be passed to Command::setCode

Bug fix: no
Feature addition: kinda
Backwards compatibility break: no
Symfony2 tests pass: yes
License of the code: MIT
merged branch Tobion/patch-5 (PR #6176)

This PR was merged into the master branch.

Commits
-------

918bad6 fix phpdoc in ExecutionContextInterface

Discussion
----------

fix phpdoc in ExecutionContextInterface
Fixed DefaultValue for session.auto_start in NodeDefinition

This is just for consistency with the node type (Boolean)
merged branch franmomu/auto_start_default_value (PR #6155)

This PR was submitted for the master branch but it was merged into the 2.1 branch instead (closes #6155).

Commits
-------

88811f4 [FrameworkBundle] Fixed DefaultValue for session.auto_start in NodeDefinition

Discussion
----------

[FrameworkBundle] Fixed DefaultValue for session.auto_start in NodeDefinition

Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets:
Todo: -
License of the code: MIT
Documentation PR: -

This is just for consistency with the node type.

---------------------------------------------------------------------------

by fabpot at 2012-11-30T07:17:33Z

I don't see the need to change the default value as this is deprecated anyway.

---------------------------------------------------------------------------

by franmomu at 2012-11-30T07:35:35Z

I was trying to validate default values in Config Component and if I do this throws an exception
[WiP] Split urlmatcher for easier overriding
merged branch Crell/split-urlmatcher (PR #6100)

This PR was squashed before being merged into the master branch (closes #6100).

Commits
-------

0e3671b [WiP] Split urlmatcher for easier overriding

Discussion
----------

[WiP] Split urlmatcher for easier overriding

Based on discussion in https://github.com/symfony-cmf/Routing/pull/30, this PR splits the matchCollection() method of UrlMatcher into two methods.  The reason is to allow Symfony CMF and Drupal to override just one of them, while leaving the actual meat of the class intact.

Additionally, it switches $routes from private to protected for the same reason: It makes it possible for us to extend the class cleanly.

Marking as WIP in case further discussion in CMF suggests other/different changes, but review and a conceptual go/no-go would be appreciated now.

---------------------------------------------------------------------------

by dbu at 2012-11-25T12:57:46Z

i think this variant really just extracts part of the logic into a separate method whithout changing any behaviour or concept. it would help a lot for the cmf to have it this way so we can extend and tweak the logic. is this now good or anybody has more input?

---------------------------------------------------------------------------

by dbu at 2012-11-27T19:42:04Z

sorry for being pushy about this one, but we need to know if this change is ok or not, if we need to improve something. if there is some problem we did not think of, we have to find different solutions for the cmf/drupal matchers.

---------------------------------------------------------------------------

by fabpot at 2012-11-28T14:29:10Z

`PhpMatcherDumper` should probably also be updated to call the new `getAttributes()` method; but that won't be possible as the Route is not accessible when using this dumper. Adding a feature that can only be used by the standard `UrlMatcher` and not by the other matchers does not sound good to me.

---------------------------------------------------------------------------

by dbu at 2012-11-28T17:18:09Z

in the context of the cmf, our problem is that we have too many routes to hold in memory. i think the dumper is not of interest for that use case - @Crell correct me please if i am wrong. if we need any caching we would need to write our own dumper probably. but currently we just extend the UrlMatcher

---------------------------------------------------------------------------

by Crell at 2012-11-30T05:47:39Z

Correct. In both the CMF case and Drupal case we have our own dumpers, because the current ones don't work for us anyway.  (1000 routes and all that. :-) )  This isn't a new public method.  It's just a small refactor of UrlMatcher itself to make it easier to extend.  If you're using a dumped PhpMatcher, I don't know why you'd be using something like NestedMatcher in the first place.
[Routing] added a warning about UrlMatcher::getAttributes()
Update src/Symfony/Bridge/Propel1/Form/EventListener/TranslationCollectionFormListener.php

Remove duplicated semicolon
[Security] Move DigestDataTest.php inside the Security component
merged branch jamogon/patch-1 (PR #6181)

This PR was merged into the master branch.

Commits
-------

760aee0 Update src/Symfony/Bridge/Propel1/Form/EventListener/TranslationCollectionFormListener.php

Discussion
----------

Update src/Symfony/Bridge/Propel1/Form/EventListener/TranslationCollecti...

...onFormListener.php

Remove duplicated semicolon

---------------------------------------------------------------------------

by pborreli at 2012-12-03T22:41:12Z

:+1: