Allow to define Enum nodes with 1 single element #15433
Conversation
javiereguiluz
commented
Aug 2, 2015
javiereguiluz
commented
| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | yes |
| BC breaks? | yes |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #15402 |
| License | MIT |
| Doc PR | - |
There was a problem hiding this comment.
I would keep a validation to forbid empty arrays (which does not make sense as it makes the config unusable)
There was a problem hiding this comment.
I don't know. What is the usual Symfony practice?
|
@javiereguiluz Can you add a note in the UPGRADE file? |
|
Thank you @javiereguiluz. |
There was a problem hiding this comment.
the UPGRADE file is about showing required/recommended changes because of BC breaks of deprecations.
this is just adding a new feature in a BC way, so a line in the changelog would be enough. Adding it in the UPGRADE file just makes the file less focused
…buh) This PR was merged into the 2.8 branch. Discussion ---------- [Config] move feature description to changelog file | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #15433 | License | MIT | Doc PR | Commits ------- e1818bd move feature description to changelog file
|
@javiereguiluz: This actually fixes the issue in the |
|
@ogizanagi :'( thanks for reporting this error. I've created a new issue to fix it: #17774 |
…es with one element (ogizanagi) This PR was merged into the 2.8 branch. Discussion ---------- [Config] Fix EnumNodeDefinition to allow building enum nodes with one element | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #15433, #17774 | License | MIT | Doc PR | - Commits ------- e9111e4 [Config] Fix EnumNodeDefinition to allow building enum nodes with one element
6 participants
