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 php 8.1 deprecation #121

Merged
merged 2 commits into from
Jan 18, 2024
Merged

fix php 8.1 deprecation #121

merged 2 commits into from
Jan 18, 2024

Conversation

alikon
Copy link
Contributor

@alikon alikon commented Nov 8, 2023

Pull Request for Issue #114
reported also joomla/joomla-cms#42317

Summary of Changes

fix deprecated stripos() passing null

src/Web/WebClient.php Outdated Show resolved Hide resolved
@Chessy65
Copy link

Chessy65 commented Nov 8, 2023

Very thanks. No error text now. I hope, that this come in the new joomla standard version.

@Hackwar Hackwar merged commit 608393d into joomla-framework:3.x-dev Jan 18, 2024
1 check passed
@alikon alikon deleted the patch-1 branch January 18, 2024 18:25
@richard67
Copy link

@alikon Could you do the same PR here also for the 2.0-dev branch so we could make a release 2.0.5 to fix it in 4.,4-dev of the CMS, too? Let me know if you can do that or if I shall do it. Thanks in advance.

@alikon
Copy link
Contributor Author

alikon commented Feb 21, 2024

i'll try to do it
btw ...
if i'll run into problems
i'll ping you 😄

alikon added a commit to alikon/application that referenced this pull request Feb 22, 2024
@alikon
Copy link
Contributor Author

alikon commented Feb 22, 2024

see #122 for 2.0

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.

5 participants