Skip to content

Fix deprecation warnings and improve property handling#63

Closed
Mallon94 wants to merge 2 commits into
ipinfo:masterfrom
Mallon94:patch-1
Closed

Fix deprecation warnings and improve property handling#63
Mallon94 wants to merge 2 commits into
ipinfo:masterfrom
Mallon94:patch-1

Conversation

@Mallon94

Copy link
Copy Markdown
  • Explicitly declare the $ipinfo and $no_except properties to avoid dynamic creation which is deprecated as of PHP 8.2.
  • Update the docblock for the $filter property to accurately represent its type as callable.
  • Ensure that all class properties are initialized properly to prevent runtime errors and adhere to stricter PHP standards.

- Explicitly declare the `$ipinfo` and `$no_except` properties to avoid dynamic creation which is deprecated as of PHP 8.2.
- Update the docblock for the `$filter` property to accurately represent its type as `callable`.
- Ensure that all class properties are initialized properly to prevent runtime errors and adhere to stricter PHP standards.
all illuminate/support 11
@max-ipinfo

Copy link
Copy Markdown
Contributor

Hello @Mallon94 , I'm picking up our Laravel library again and just found out about your PR. Sorry for the lack of response!

I've just released v2.7.0 with Laravel v11 support. Could you rebase and send me your PR with the latest changes?

@silvanocerza

Copy link
Copy Markdown
Contributor

Superseded by #77.

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.

3 participants