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

A few Pods 3.0.1 and PHP 8.1 Deprecated warnings (functions-view_template.php on line 174, Pods.php on line 741) #7171

Closed
oldrup opened this issue Sep 16, 2023 · 2 comments
Assignees
Labels
Keyword: Focus This is a major focus of a release Type: Bug
Milestone

Comments

@oldrup
Copy link

oldrup commented Sep 16, 2023

Description

On a test site, when updating to PHP 8.1, and with WP_DEBUN on, I get these deprecation warnings:
Deprecated: substr(): Passing null to parameter #1 ($string) of type string is deprecated in /home/removed/wp-content/plugins/pods/components/Templates/includes/functions-view_template.php on line 174

and

Deprecated : strlen(): Passing null to parameter #1 ($string) of type string is deprecated in /home/removed/wp-content/plugins/pods/classes/Pods.php on line 741

image

Is that something you can reproduce?

I'm using Pods Templates with MagicTags™. Please tell me, if I should put some effort into pinpointing exactly what query triggers the warning.

image

Version

3.0.1

Testing Instructions

No response

Screenshots / Screencast

No response

Possible Workaround

Downgrading to PHP 8.0 gets rid of the warnings.

Site Health Information

No response

Pods Package

No response

@sc0ttkclark
Copy link
Member

Will tackle this along with the strlen() issues in pick.php (line 2959) - https://wordpress.org/support/topic/tons-of-php-deprecated-strlen-messages-in-error-log-under-php-8-x/#post-17063480

@sc0ttkclark sc0ttkclark added this to the Pods 3.0.2 milestone Sep 19, 2023
@sc0ttkclark sc0ttkclark self-assigned this Sep 19, 2023
@sc0ttkclark sc0ttkclark added the Keyword: Focus This is a major focus of a release label Sep 19, 2023
@sc0ttkclark
Copy link
Member

Fixed via c30862a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Keyword: Focus This is a major focus of a release Type: Bug
Projects
None yet
Development

No branches or pull requests

2 participants