Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent constructor override when constructor is final #166

Merged
merged 2 commits into from Jan 26, 2015

Conversation

nicolasThal
Copy link
Contributor

During my test, I wanted to mock a class with a final constructor like this one https://github.com/doctrine/annotations/blob/master/lib/Doctrine/Common/Annotations/Annotation.php which has a final constructor.

I have added a test for this case.

What are your thoughts about this issue?

Regards

@MaximeThoonsen
Copy link

+1

4 similar comments
@juchi
Copy link

juchi commented Jan 26, 2015

+1

@KennyDurand
Copy link

+1

@nicbaz
Copy link

nicbaz commented Jan 26, 2015

👍

@jbeurel
Copy link

jbeurel commented Jan 26, 2015

👍

mlively added a commit that referenced this pull request Jan 26, 2015
Prevent constructor override when constructor is final
@mlively mlively merged commit bf565aa into phake:master Jan 26, 2015
@mlively
Copy link
Collaborator

mlively commented Jan 26, 2015

I knew I was missing something :). Thanks

mlively pushed a commit that referenced this pull request Jan 26, 2015
@mlively
Copy link
Collaborator

mlively commented Jan 26, 2015

This is now available in v2.0.0-beta4 and v1.0.7

@nicolasThal nicolasThal deleted the final_constructor_override branch January 26, 2015 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants