Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Yaml] remove deprecated features#22770
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
xabbuh commentedMay 19, 2017
| Q | A |
|---|---|
| Branch? | master |
| Bug fix? | no |
| New feature? | no |
| BC breaks? | yes |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | |
| License | MIT |
| Doc PR |
e6c9a2d to6f4ad01Compare38d9b3d to020178bCompare020178b to5d584deCompare
nicolas-grekas left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
👍
GuilhemN left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
👍
| } | ||
| } | ||
| if (':' !==$key && (!isset($mapping[$i +1]) || !in_array($mapping[$i +1],array('',',','[',']','{','}'),true))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
':' !== $key should be removed, right?
| } | ||
| // key | ||
| $key =self::parseScalar($mapping,$flags,array(':',''),$i,false,array(),true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The last argument should be removed, isn't it? (and the according logic in parseScalar)
5d584de toefe2337Comparefabpot commentedMay 24, 2017
Thank you@xabbuh. |
This PR was merged into the 4.0-dev branch.Discussion----------[Yaml] remove deprecated features| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MIT| Doc PR |Commits-------efe2337 [Yaml] remove deprecated features