[Form] Added delete_empty option to allow proper emptyData handling of collections
feature #9773 [Form] Added delete_empty option to allow proper emptyData handling of collections (peterrehm)

This PR was squashed before being merged into the 2.5-dev branch (closes #9773).

Discussion
----------

[Form] Added delete_empty option to allow proper emptyData handling of collections

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | yes/no?
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #9375
| License       | MIT
| Doc PR        | symfony/symfony-docs#3338

This PR adresses the issue that if you have a form collection with the option required => false and you submit an empty collection so you will get a ArrayCollection that contains en element with the value null.

This behaviour was introduced with the following changes from symfony/symfony#3257

In addition to the above mentioned ticket there is also a description about the same issue here: http://stackoverflow.com/questions/19474872/symfony2-form-collection-allow-add-and-allow-delete-null-error-silex

With the changes of this PR the new option empty_data is introduced. With this option you will be able to ignore/delete such empty collections, so they will neither be validated nor appear as empty field in the result.

The option will remove/ignore such empty collections if you add them newly and if allow_add is enabled and
removes such empty collections only if allow_delete is enabled.

With setting required and empty_data accordingly it will be now flexible to customize to the outcome you want to achieve.

Thanks to @bschussek for the great work together - We have to discuss how to name this option so if delete
or ignore is the appropriate name.

Commits
-------

8bdb7a0 [Form] Added delete_empty option to allow proper emptyData handling of collections
fixed CS
[Form][2.4] added an option for multiple files upload (closes #1400)
feature #8224 [Form][2.4] added an option for multiple files upload (closes #1400) (bamarni)

This PR was squashed before being merged into the 2.5-dev branch (closes #8224).

Discussion
----------

[Form][2.4] added an option for multiple files upload (closes #1400)

| Q             | A
| ------------- | ---
| Bug fix?      | [no]
| New feature?  | [yes]
| BC breaks?    | [no]
| Deprecations? | [no]
| Tests pass?   | [yes]
| Fixed tickets | [#1400]
| License       | MIT
| Doc PR        | [todo]

Commits
-------

c8c6448 [Form][2.4] added an option for multiple files upload (closes #1400)
[Console] Added a way to set the process title
[DependencyInjection] Avoid call_user_func in dumped containers.
minor #9807 [DependencyInjection] Avoid call_user_func in dumped containers. (realityking)

This PR was merged into the 2.5-dev branch.

Discussion
----------

[DependencyInjection] Avoid call_user_func in dumped containers.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

This is the second commit from  #9432.

When using static methods for either the factory or as the configurator we can avoid using call_user_func and directly use the class:method notation. This is faster (about 5 times, but we're talking milliseconds here) but I think the resulting code is also much easier to read.

The code to use call_user_func has to remain in PhpDumper because in the uncompiled container they still get used.

Commits
-------

be1eaaa [DependencyInjection] Avoid call_user_func in dumped containers.
[HttpKernel] Remove FrameworkBundle dependency in BundleTest
feature #9780 [Console] Added a way to set the process title (lyrixx)

This PR was merged into the 2.5-dev branch.

Discussion
----------

[Console] Added a way to set the process title

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT

* I did not write test, because I want to RFC the idea before
* Do we want the process update its name when a sub command is executed by a master command ?

Commits
-------

204a25e [Console] Added a way to set the process title
minor #9802 [HttpKernel] Remove FrameworkBundle dependency in BundleTest (florianv)

This PR was merged into the 2.5-dev branch.

Discussion
----------

[HttpKernel] Remove FrameworkBundle dependency in BundleTest

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

 `HttpKernel` tests fatal due to missing dependency `AddConsoleCommandPass`.
Also removed unused import.

Commits
-------

20a064f [HttpKernel] Remove FrameworkBundle dependency in BundleTest
[DependencyInjection] fix a regression introduced in #9807

Always dump the fully qualified class name, to support dumping containers in a namespace.
bug #9812 [DependencyInjection] fix a regression introduced in #9807 (realityking)

This PR was merged into the 2.5-dev branch.

Discussion
----------

[DependencyInjection] fix a regression introduced in #9807

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

As @stof pointed out in #9807 we always need to dump the fully qualified class name, to support dumping containers in a namespace.

Commits
-------

0d78776 [DependencyInjection] fix a regression introduced in #9807
[DepdencyInjection] forgot to add definition of dumped container member variable parameters
feature #9668 [DepdencyInjection] forgot to add definition of dumped container member variable parameters (cordoval)

This PR was squashed before being merged into the 2.5-dev branch (closes #9668).

Discussion
----------

[DepdencyInjection] forgot to add definition of dumped container member variable parameters

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #5408
| License       | MIT
| Doc PR        | n/a

this fixes the allowing of access to parameters property of the dumped container

Commits
-------

5b02d3f [DepdencyInjection] forgot to add definition of dumped container member variable parameters
[EventDispatcher][HttpKernel] Move RegisterListenersPass from HttpKernel to EventDispatcher.
feature #9792 [EventDispatcher][HttpKernel] Move RegisterListenersPass from HttpKernel to EventDispatcher. (realityking)

This PR was merged into the 2.5-dev branch.

Discussion
----------

[EventDispatcher][HttpKernel] Move RegisterListenersPass from HttpKernel to EventDispatcher.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

I've been recently in the position to use the DependencyInjection Component and the EventDispatcher together but not HttpKernel. To make it easier to use these two together I propose moving RegisterListenersPass to the EventDispatcher component where the ContainerAwareEventDispatcher already resides.

Commits
-------

89b8e0a [EventDispatcher][HttpKernel] Move RegisterListenersPass from HttpKernel to EventDispatcher.