Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DDC-2704] - merge inherited transient properties - merge properties into uninitialized proxies#1272

Merged
Ocramius merged 20 commits into
doctrine:masterfrom
Ocramius:hotfix/DDC-2704-merge-inherited-transient-properties
Jan 25, 2015
Merged

[DDC-2704] - merge inherited transient properties - merge properties into uninitialized proxies#1272
Ocramius merged 20 commits into
doctrine:masterfrom
Ocramius:hotfix/DDC-2704-merge-inherited-transient-properties

Conversation

@Ocramius
Copy link
Copy Markdown
Member

@doctrinebot
Copy link
Copy Markdown

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-3524

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

Comment thread lib/Doctrine/ORM/UnitOfWork.php Outdated
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This may be replaced with a static utility if the classmetadata changes are not required

…d class metadata instance will result in a failure
…nitialized class metadata instance will result in a failure
…class metadata may not hold reflection class after wakeup)
…n metadata API when fetching reflection properties for a class
@Ocramius Ocramius force-pushed the hotfix/DDC-2704-merge-inherited-transient-properties branch from 10c7978 to 28e0da4 Compare January 24, 2015 13:32
@Ocramius Ocramius removed the WIP label Jan 24, 2015
Ocramius added a commit that referenced this pull request Jan 25, 2015
…d-transient-properties

[DDC-2704] - merge inherited transient properties - merge properties into uninitialized proxies
@Ocramius Ocramius merged commit d024193 into doctrine:master Jan 25, 2015
@Ocramius Ocramius deleted the hotfix/DDC-2704-merge-inherited-transient-properties branch January 25, 2015 04:55
@digitalkaoz
Copy link
Copy Markdown

@Ocramius it still seems to fail for properties which where "injected" by Traits

@digitalkaoz
Copy link
Copy Markdown

making them protected fixes this, but its not something i like

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants