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 deprecated usages on PHP 8.1 #1354

Merged
merged 1 commit into from
Nov 3, 2021
Merged

Fix deprecated usages on PHP 8.1 #1354

merged 1 commit into from
Nov 3, 2021

Conversation

cedric-anne
Copy link
Contributor

@cedric-anne cedric-anne commented Oct 6, 2021

Fixes following errors encountered on a PHP 8.1 RC environment.

==> Error E_DEPRECATED in ... generated by file /var/glpi/vendor/sabre/dav/lib/DAV/Xml/Element/Response.php on line 124:
ctype_digit(): Argument of type int will be interpreted as string in the future

@cedric-anne
Copy link
Contributor Author

Hi,

Can anyone review this PR ?

Regards

@cedric-anne
Copy link
Contributor Author

ping @phil-davis @staabm

@kesselb
Copy link
Contributor

kesselb commented Nov 2, 2021

Hey @cedric-anne 👋

please rebase your branch to have a recent CI run.

@cedric-anne
Copy link
Contributor Author

Hey @cedric-anne wave

please rebase your branch to have a recent CI run.

Hi,

Done.

@codecov
Copy link

codecov bot commented Nov 2, 2021

Codecov Report

Merging #1354 (30f30f7) into master (9c04ba8) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1354   +/-   ##
=========================================
  Coverage     97.31%   97.31%           
  Complexity     2806     2806           
=========================================
  Files           174      174           
  Lines          7975     7975           
=========================================
  Hits           7761     7761           
  Misses          214      214           
Impacted Files Coverage Δ
lib/DAV/Xml/Element/Response.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9c04ba8...30f30f7. Read the comment docs.

@phil-davis
Copy link
Contributor

phil-davis commented Nov 2, 2021

Note: issue #1361 - CI is currently failing on various new things found by the latest phpstan. That needs fixing first.

@phil-davis phil-davis self-assigned this Nov 3, 2021
@phil-davis
Copy link
Contributor

A GutHub workflow got:

Error: curl error 7 while downloading https://repo.packagist.org/packages.json: Failed to connect to repo.packagist.org port 443: Connection refused

I will try to rerun it.

@phil-davis phil-davis merged commit 38f1394 into sabre-io:master Nov 3, 2021
@cedric-anne cedric-anne deleted the fix/php8.1 branch November 3, 2021 09:49
@cedric-anne
Copy link
Contributor Author

Hi @phil-davis

I don't want to force your hand, but PHP 8.1.0 will be released on November 25th. I think a lot of people would like to be able to enjoy a stable release of sabre/dav, including these fixes, by then.

Regards

@phil-davis
Copy link
Contributor

@cedric-anne no problem! I am looking at the other pending PRs that probably can be merged. Whatever happens, I will make a release in the next few days with whatever is merged.

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

3 participants