Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing php coding #3961

Merged
merged 2 commits into from
Jun 20, 2014
Merged

Fixing php coding #3961

merged 2 commits into from
Jun 20, 2014

Conversation

mvhirsch
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets n/a

There is a syntax error in those arrays, so I fixed it.
I also added the configuration-block macro as it seems to be missing on that page.

@wouterj
Copy link
Member

wouterj commented Jun 19, 2014

👍 thank you!

@weaverryan
Copy link
Member

Nice catch! Thanks Michael!

@weaverryan weaverryan merged commit 68a8ab1 into symfony:2.3 Jun 20, 2014
weaverryan added a commit that referenced this pull request Jun 20, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

Fixing php coding

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | n/a

There is a syntax error in those arrays, so I fixed it.
I also added the configuration-block macro as it seems to be missing on that page.

Commits
-------

68a8ab1 added block macro
f2aad67 fixed small php syntax error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants