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

Adding bug 63462 test #229

Closed
wants to merge 1 commit into from
Closed

Conversation

Ocramius
Copy link
Contributor

@Ocramius Ocramius commented Nov 8, 2012

This PR introduces a failing test for bug 63462.

Any access to private/protected properties that were unset triggers __get, __set or __isset twice.

This does not happen with public or non-existing properties though.

@lstrojny
Copy link
Contributor

lstrojny commented Jan 6, 2013

Could you please mark the test as XFAIL? Thanks.

@Ocramius
Copy link
Contributor Author

Ocramius commented Jan 6, 2013

@lstrojny heya, what exactly do you mean by XFAIL (sorry, not aware of how the test runner works)

@Ocramius
Copy link
Contributor Author

Ocramius commented Jan 6, 2013

To be more clear, I think that Ocramius:issue/bug-63462 should only be merged into a branch that already contains the fix

@lstrojny
Copy link
Contributor

lstrojny commented Jan 6, 2013

@Ocramius XFAIL is a section that allows committing failing tests without failing the build

--XFAIL--
Expected to fail as bug 12345 is not yet fixed

@Ocramius
Copy link
Contributor Author

Ocramius commented Jan 6, 2013

Will do ASAP then

@lstrojny
Copy link
Contributor

lstrojny commented Jan 6, 2013

One last thing: could you squash the commits into one?

@Ocramius
Copy link
Contributor Author

Ocramius commented Jan 6, 2013

@lstrojny done

@php-pulls
Copy link

Comment on behalf of lstrojny at php.net:

Merged. Thanks!

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.

3 participants