Skip to content

Don't pass null as second parameter to preg_match #190

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

Merged
merged 3 commits into from
Feb 14, 2022

Conversation

phpfui
Copy link
Contributor

@phpfui phpfui commented Feb 8, 2022

Found this on my website where I access git commits as a diff. PHPFUI/Instadoc click on the top two commits to see the error in the wild. Will be fixed automatically when this PR is published. Site updates nightly.

This is just an 8.1 issue.

There may be other places this happens, but I have only run into this one.

@phpfui
Copy link
Contributor Author

phpfui commented Feb 8, 2022

Curse you PHP 5.6!

@alexandresalome alexandresalome merged commit 3c2f67a into gitonomy:1.3 Feb 14, 2022
@alexandresalome
Copy link
Member

Thank you @phpfui !

@phpfui
Copy link
Contributor Author

phpfui commented Feb 14, 2022

Can we get a v1.3.4 tag for this? It is causing errors for me in production.

Thanks!

@alexandresalome
Copy link
Member

@phpfui done

@phpfui
Copy link
Contributor Author

phpfui commented Feb 17, 2022

@alexandresalome I still don't see a v1.3.4 version of this library. Did a tag not get pushed?

Thanks

@phpfui phpfui deleted the PHP8.1Fix branch February 28, 2022 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants