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

Identifier is cached with wrong type #1568

Merged
merged 2 commits into from
Nov 25, 2015

Conversation

guiwoda
Copy link
Contributor

@guiwoda guiwoda commented Nov 24, 2015

Based on #1556, just added a functional test that reflects what was described on the original ticket.

@doctrinebot
Copy link

Hello,

thank you for creating this pull request. I have automatically opened an issue
on our Jira Bug Tracker for you. See the issue link:

http://www.doctrine-project.org/jira/browse/DDC-4016

We use Jira to track the state of pull requests and the versions they got
included in.

@guiwoda
Copy link
Contributor Author

guiwoda commented Nov 24, 2015

@guilhermeblanco let me know if this is what you needed to merge!

@guiwoda
Copy link
Contributor Author

guiwoda commented Nov 24, 2015

Saw the errors, I'm on it.

@guilhermeblanco guilhermeblanco self-assigned this Nov 25, 2015
guilhermeblanco added a commit that referenced this pull request Nov 25, 2015
Identifier is cached with wrong type
@guilhermeblanco guilhermeblanco merged commit 57b22bd into doctrine:master Nov 25, 2015
@guilhermeblanco
Copy link
Member

👍

@janlanger
Copy link
Contributor

@Ocramius @guilhermeblanco Will this changes be included any upcoming in 2.5 release? Or is 2.6 coming soon? :)

@Ocramius Ocramius added this to the 2.6.0 milestone Jan 5, 2016
@Ocramius Ocramius added the Bug label Jan 5, 2016
@Ocramius Ocramius modified the milestones: 2.5.4, 2.6.0 Jan 5, 2016
@Ocramius
Copy link
Member

Ocramius commented Jan 5, 2016

@janlanger I will backport this to 2.5.4, thanks for poking me :)

Ocramius added a commit that referenced this pull request Jan 5, 2016
@Ocramius
Copy link
Member

Ocramius commented Jan 5, 2016

Backported in aae43cb

@janlanger
Copy link
Contributor

Thanks!

@Ocramius
Copy link
Member

Ocramius commented Jan 5, 2016

@janlanger and released as 2.5.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants