bug #34707 [Validation][FrameworkBundle] Allow EnableAutoMapping to work without...
authorMaxime Steinhausser <maxime.steinhausser@elao.com>
Tue, 17 Dec 2019 08:15:10 +0000 (09:15 +0100)
committerMaxime Steinhausser <maxime.steinhausser@elao.com>
Tue, 17 Dec 2019 08:15:10 +0000 (09:15 +0100)
commit3efdd80704377f9479c15f0e77c862404c15de04
tree7a042766c0a728d023a9b0bf8da27a0a00dc11fc
parent4e44baf1bb354b0f777d5ecb227dbfdbad2cc240
parent00b46fa72ee4b6e4555abe5866853fa2087107e1
bug #34707 [Validation][FrameworkBundle] Allow EnableAutoMapping to work without auto-mapping namespaces (ogizanagi)

This PR was squashed before being merged into the 4.4 branch (closes #34707).

Discussion
----------

[Validation][FrameworkBundle] Allow EnableAutoMapping to work without auto-mapping namespaces

| Q             | A
| ------------- | ---
| Branch?       | 4.4 <!-- see below -->
| Bug fix?      | yes?
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | N/A <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | N/A

Currently, when no `framework.validation.auto_mapping` namespaces are configured, the `EnableAutoMapping` annotation has no effect on its own, because the loaders were removed by the fwb extension and the compiler pass.
Simply adding a namespace (even a foolish one) will make it work:

```yml
    validation:
        auto_mapping:
            '*': ~ # this does not really map anything, but allows `EnableAutoMapping` to work as loaders won't be removed.
```

So for those only wanting to use the auto-mapping feature by explicitly setting the `EnableAutoMapping` annotation, it'll be counter-intuitive, as it cannot work without declaring at least one namespace.

Commits
-------

00b46fa72e [Validation][FrameworkBundle] Allow EnableAutoMapping to work without auto-mapping namespaces