DDC-597: Add check for public properties in Validate-Mapping command? #732

Closed
opened 2026-01-22 12:48:26 +01:00 by admin · 6 comments
Owner

Originally created by @doctrinebot on GitHub (May 17, 2010).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user @beberlei:

There should also be checks for public properties on the validate-mapping command, so that people can be warned about possible sources of bugs.

Originally created by @doctrinebot on GitHub (May 17, 2010). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user @beberlei: There should also be checks for public properties on the validate-mapping command, so that people can be warned about possible sources of bugs.
admin closed this issue 2026-01-22 12:48:26 +01:00
Author
Owner

@doctrinebot commented on GitHub (May 20, 2010):

Comment created by shurakai:

I've committed a first proposal: dd6d0ee0cc

Let me know whether you're satisfied with this solution or not.

@doctrinebot commented on GitHub (May 20, 2010): Comment created by shurakai: I've committed a first proposal: http://github.com/Shurakai/doctrine2/commit/dd6d0ee0cc19571acb70c33dbcfa4518c2387b6c Let me know whether you're satisfied with this solution or not.
Author
Owner

@doctrinebot commented on GitHub (May 20, 2010):

Comment created by @hobodave:

Pushed this branch to doctrine/orm for Christian http://github.com/doctrine/orm/tree/DDC-597

@doctrinebot commented on GitHub (May 20, 2010): Comment created by @hobodave: Pushed this branch to doctrine/orm for Christian http://github.com/doctrine/orm/tree/[DDC-597](http://www.doctrine-project.org/jira/browse/DDC-597)
Author
Owner

@doctrinebot commented on GitHub (May 21, 2010):

Comment created by shurakai:

Fixed.

@doctrinebot commented on GitHub (May 21, 2010): Comment created by shurakai: Fixed.
Author
Owner

@doctrinebot commented on GitHub (May 21, 2010):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (May 21, 2010): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Jun 8, 2010):

Comment created by romanb:

Has this been merged into master yet? I'm asking because the branch still exists. If it has not been merged yet, we should do that.

@doctrinebot commented on GitHub (Jun 8, 2010): Comment created by romanb: Has this been merged into master yet? I'm asking because the branch still exists. If it has not been merged yet, we should do that.
Author
Owner

@doctrinebot commented on GitHub (Jun 8, 2010):

Comment created by shurakai:

It has not been merged into master yet.

@doctrinebot commented on GitHub (Jun 8, 2010): Comment created by shurakai: It has not been merged into master yet.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#732