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 deprecation warnings and improve property handling #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Mallon94
Copy link

  • 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
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.

1 participant