Problem/Motivation

PHP 8.4 deprecation notices:
Implicitly marking parameter $param as nullable is deprecated, the explicit nullable type must be used instead

Steps to reproduce

Use PHP 8.4

CommentFileSizeAuthor
#2 geolocation-php84-3546951-2.patch14.09 KBaalin

Issue fork geolocation-3546951

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

aalin created an issue. See original summary.

aalin’s picture

aalin commented
Status: Active » Needs review
StatusFileSize
new geolocation-php84-3546951-2.patch14.09 KB

Patch added

paulguy’s picture

paulguy commented

I just posted a similar patch for another module, happy to see this one is already fixed.

Applying patch from #2 do remove the deprecated warning, thanks !

berdir made their first commit to this issue’s fork.

berdir opened merge request !121

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Needs review » Reviewed & tested by the community
bserem’s picture

bserem commented

Bumping for a new 3.x release. This is required from localgov installations and it's one of the few blockers for updating cleanly to PHP 8.4

bserem’s picture

bserem commented

Another bump

christianadamski’s picture

christianadamski commented

Give it a few more days..

christianadamski’s picture

christianadamski commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.