8000 [DoctrineBridge] Fixed caching in DoctrineType when "choices" or "preferred_choices" is passed by webmozart · Pull Request #6216 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[DoctrineBridge] Fixed caching in DoctrineType when "choices" or "preferred_choices" is passed #6216

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

Merged
merged 1 commit into from
Dec 7, 2012

Conversation

webmozart
Copy link
Contributor

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: #6190
Todo: -
License of the code: MIT
Documentation PR: n/a

@craue
Copy link
Contributor
craue commented Dec 6, 2012

👍

@lstrojny
Copy link
Contributor
lstrojny commented Dec 6, 2012

What about a test?

@@ -1,6 +1,11 @@
CHANGELOG
=========

2.1.5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be removed as this is done when we release

@webmozart
Copy link
Contributor Author

Removed CHANGELOG entries and added tests.

@craue
Copy link
Contributor
craue commented Dec 7, 2012

I'm not sure if @fabpot is objected to the changelog entry entirely or just the version number. What is the segfault about for PHP 5.4 in Travis?

fabpot added a commit that referenced this pull request Dec 7, 2012
This PR was merged into the 2.1 branch.

Commits
-------

ca5d9ac [DoctrineBridge] Fixed caching in DoctrineType when "choices" or "preferred_choices" is passed

Discussion
----------

[DoctrineBridge] Fixed caching in DoctrineType when "choices" or "preferred_choices" is passed

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: #6190
Todo: -
License of the code: MIT
Documentation PR: n/a

---------------------------------------------------------------------------

by craue at 2012-12-06T18:31:43Z

:+1:

---------------------------------------------------------------------------

by lstrojny at 2012-12-06T18:35:04Z

What about a test?

---------------------------------------------------------------------------

by bschussek at 2012-12-07T12:39:51Z

Removed CHANGELOG entries and added tests.

---------------------------------------------------------------------------

by craue at 2012-12-07T13:14:02Z

I'm not sure if @fabpot is objected to the changelog entry entirely or just the version number. What is the segfault about for PHP 5.4 in Travis?
@fabpot fabpot merged commit ca5d9ac into symfony:2.1 Dec 7, 2012
Reviewers
No reviews
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0