Skip to content

Deprecation notice in ee/ExpressionEngine/Addons/stats/mod.stats.php, line 167 #2505

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.

Sign up for GitHub

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

Jump to bottom
Closed
bnetz opened this issue Nov 11, 2022 · 3 comments · Fixed by #2509
Closed

Deprecation notice in ee/ExpressionEngine/Addons/stats/mod.stats.php, line 167 #2505

bnetz opened this issue Nov 11, 2022 · 3 comments · Fixed by #2509
Assignees
@intoeetive

Comments

@bnetz
Copy link

bnetz commented Nov 11, 2022

After updating to EE7 + PHP8 we get the following notice:

Deprecated
preg_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated
ee/ExpressionEngine/Addons/stats/mod.stats.php, line 167
Severity: E_DEPRECATED

Environment Details:

  • EE 7.2
  • PHP 8.1.12
intoeetive added a commit that referenced this issue Nov 14, 2022
@intoeetive
Resolved #2505 where Stats module could show PHP error when using PHP…
31c4160 
… 8.1
@intoeetive intoeetive mentioned this issue Nov 14, 2022
Merged
@intoeetive intoeetive self-assigned this Nov 14, 2022
matthewjohns0n added a commit that referenced this issue Nov 21, 2022
@matthewjohns0n
Merge pull request #2509 from ExpressionEngine/bug/mod-stats-php-81
e34573c 
Resolved #2505 where Stats module could show PHP error when using PHP 8.1
@bnetz
Copy link
Author

bnetz commented Nov 27, 2022

Problem still persists after upgrading to 7.2.1

@bnetz
Copy link
Author

bnetz commented Nov 29, 2022

Solved with 7.2.2, thanks!

@matthewjohns0n
Copy link
Member

Awesome, glad to hear it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@intoeetive intoeetive

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@intoeetive @matthewjohns0n @bnetz