DDC-2992: [GH-958] wrong access modifiers "private" instead of "protected" #3721

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

Originally created by @doctrinebot on GitHub (Feb 20, 2014).

Originally assigned to: @Ocramius on GitHub.

Jira issue originally created by user @doctrinebot:

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

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

Message:

In configuration is parameter:

<parameter key="doctrine.orm.metadata.xml.class">Doctrine\ORM\Mapping\Driver\SimplifiedXmlDriver</parameter>

that suggest that is designed to override in private bundles but when you try to do that it's nessesary to repeat entire body of methods in own class because of "private" access modifiers, i suggest to change them to 'protected' because without that it's makes extending this class more difficult and not clear to understand what is going on inside of this classes.

... I suggest to change this modifiers not only in this class...

Originally created by @doctrinebot on GitHub (Feb 20, 2014). Originally assigned to: @Ocramius on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of grytysek: Url: https://github.com/doctrine/doctrine2/pull/958 Message: In configuration is parameter: ``` <parameter key="doctrine.orm.metadata.xml.class">Doctrine\ORM\Mapping\Driver\SimplifiedXmlDriver</parameter> ``` that suggest that is designed to override in private bundles but when you try to do that it's nessesary to repeat entire body of methods in own class because of "private" access modifiers, i suggest to change them to 'protected' because without that it's makes extending this class more difficult and not clear to understand what is going on inside of this classes. ... I suggest to change this modifiers not only in this class...
admin added the Bug label 2026-01-22 14:26:16 +01:00
admin closed this issue 2026-01-22 14:26:16 +01:00
Author
Owner

@doctrinebot commented on GitHub (Feb 21, 2014):

Comment created by @doctrinebot:

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

@doctrinebot commented on GitHub (Feb 21, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-958] was closed: https://github.com/doctrine/doctrine2/pull/958
Author
Owner

@doctrinebot commented on GitHub (Feb 21, 2014):

Issue was closed with resolution "Won't Fix"

@doctrinebot commented on GitHub (Feb 21, 2014): Issue was closed with resolution "Won't Fix"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#3721