Skip to content

Patch to support entity's static property. #295

Closed
wants to merge 1 commit into from

3 participants

@liverbool

Entity contains association mapping have trouble to debug with dumping object (entity) because association field will invoke proxy entity, in additional the association field can make as static and static can solve in this case. This patch make to support static member of the fields.

This pull to review.

@liverbool liverbool Patch to support entity's static property.
Entity contains association mapping have trouble to debug with dumping object (entity) because association field will invoke proxy entity, in additional the association field can make as static and static can solve in this case. This patch make to support static member of the fields.
a570efc
@doctrinebot

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/DCOM-211

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

@Ocramius
Doctrine member
Ocramius commented Sep 1, 2013

This introduces way too much overhead.

RuntimePublicReflectionProperty is only meant for hydration purposes, and it is a workaround for a known PHP bug.

@Ocramius Ocramius closed this Sep 1, 2013
@liverbool

RuntimePublicReflectionProperty used in many places, however how i can define something like associate field to static?

@Ocramius
Doctrine member
Ocramius commented Sep 2, 2013

@liverbool no, static properties are not considered by the data mappers in doctrine.

@liverbool

Ok, Accepted
:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.