DDC-3549: [GH-1292] Mark getSelectConditionStatementColumnSQL method as private #4367

Closed
opened 2026-01-22 14:40:21 +01:00 by admin · 2 comments
Owner

Originally created by @doctrinebot on GitHub (Jan 28, 2015).

Originally assigned to: @guilhermeblanco on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of goetas:

Url: https://github.com/doctrine/doctrine2/pull/1292

Message:

The Doctrine/\ORM\Persisters\BasicEntityPersister::getSelectConditionStatementColumnSQL method has been marked as private.

The current protected mark is a bit unuseful since everything is already handled into the public method getSelectConditionStatementSQL.

Originally created by @doctrinebot on GitHub (Jan 28, 2015). Originally assigned to: @guilhermeblanco on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of goetas: Url: https://github.com/doctrine/doctrine2/pull/1292 Message: The `Doctrine/\ORM\Persisters\BasicEntityPersister::getSelectConditionStatementColumnSQL` method has been marked as `private`. The current `protected` mark is a bit unuseful since everything is already handled into the public method `getSelectConditionStatementSQL`.
admin added the Improvement label 2026-01-22 14:40:21 +01:00
admin closed this issue 2026-01-22 14:40:21 +01:00
Author
Owner

@doctrinebot commented on GitHub (Jan 29, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1292] was merged:
https://github.com/doctrine/doctrine2/pull/1292

@doctrinebot commented on GitHub (Jan 29, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1292] was merged: https://github.com/doctrine/doctrine2/pull/1292
Author
Owner

@doctrinebot commented on GitHub (Jan 29, 2015):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Jan 29, 2015): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#4367