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

Fix Null class for PHP 7 #25

Closed
wants to merge 1 commit into from
Closed

Conversation

Josh-Bruce
Copy link

No description provided.

@Josh-Bruce
Copy link
Author

This is a PR for issue #22 need it for a project 👍

@romanpitak romanpitak self-assigned this Jul 2, 2016
@DavidDowdall
Copy link

I have just had to make the same change on a project and I would guess moving forward this will be required more and more, this would be a good PR to merge in.

Great library though, thank you.

@rgrebenchuk
Copy link

Is there any plan to merge PR?

@fbenas
Copy link

fbenas commented May 10, 2017

I'm also in need of this merge. Anyone here think there's a chance of that? or perhaps people moved on to other packages?

@Josh-Bruce
Copy link
Author

@fbenas Have been waiting for this for nearly a year. Just ended up pointing the composer dependancy to my fork with the fix :)

@ngse
Copy link

ngse commented Mar 24, 2019

Assume this project is dead and the maintainer is no longer active, just encountered this issue and it's been nearly 2 years since this PR was opened. Shame!

@Josh-Bruce Josh-Bruce closed this Mar 26, 2020
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.

6 participants