8000 [YAML] weird sequence-in-mapping error · Issue #8077 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[YAML] weird sequence-in-mapping error #8077

New issue

Ha 8000 ve a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
indeyets opened this issue May 17, 2013 · 6 comments
Closed

[YAML] weird sequence-in-mapping error #8077

indeyets opened this issue May 17, 2013 · 6 comments
Labels
Bug Status: Needs Review Yaml

Comments

@indeyets
Copy link

This YAML code is parsed just fine:

---
a:
- b: c

But this code:

---
a:
-
  b: c

results in Symfony\Component\Yaml\Exception\ParseException with message You cannot define a sequence item when in a mapping

I believe, that both examples are correct
cordoval added a commit to cordoval/symfony that referenced this issue Dec 5, 2013
@cordoval
reproducing bug symfony#8077 yaml sample should be equivalent
a29a350
@cordoval cordoval mentioned this issue Dec 5, 2013
Closed
@cordoval
Copy link
Contributor
cordoval commented Dec 7, 2013

keep in mind #8562 (comment) @indeyets

@wouterj
Copy link
Member
wouterj commented Feb 19, 2015

In #8562, @fabpot says:

Closing this one as I'm not going to support this syntax... except if someone provides a full patch

Should we keep it open for the "someone that provides a full patch" or should we mark this as one of the many Yaml syntaxes that the component isn't going to support?

@marcojanssen
Copy link

@fabpot a full patch as in the entire yaml spec? Or am i misreading the comment

@fabpot
Copy link
Member
fabpot commented Oct 5, 2015

Closing as we indeed do not support the whole spec and we will never be able to.

@fabpot fabpot closed this as completed Oct 5, 2015
@xabbuh xabbuh added this to the 3.1 milestone Feb 9, 2016
@xabbuh
Copy link
Member
xabbuh commented Feb 9, 2016

reopening as I think this should be easy to solve (at least I think so and will take a look into it)

@xabbuh xabbuh reopened this Feb 9, 2016
@xabbuh xabbuh added the Bug label May 22, 2016
@xabbuh xabbuh mentioned this issue May 22, 2016
Merged
@xabbuh
Copy link
Member
xabbuh commented May 22, 2016

see #18840 for a fix

fabpot added a commit that referenced this issue May 23, 2016
@fabpot
bug #18840 [Yaml] properly handle unindented collections (xabbuh)
7830fa7 
This PR was merged into the 2.3 branch.

Discussion
----------

[Yaml] properly handle unindented collections

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

Commits
-------

717e1a9 [Yaml] properly handle unindented collections
@fabpot fabpot closed this as completed May 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Needs Review Yaml
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@indeyets @fabpot @cordoval @wouterj @xabbuh @marcojanssen @carsonbot
0