Skip to content

Conversation

@szepeviktor
Copy link
Contributor

@rlankhorst Set PHP 7.2 everywhere.

$consent_category = apply_filters( 'wp_setcookie_category', wp_validate_consent_category( $consent_category ), $name, $value );
if ( empty( $consent_category ) ) {
_doing_it_wrong( 'wp_setcookie', esc_html__( 'Missing consent category. A functional, preferences, statistics-anonymous, statistics or marketing category should be passed when using wp_setcookie.', 'wp-consent-api' ), '5.6' );
_doing_it_wrong( 'wp_setcookie', esc_html__( 'Missing consent category. A functional, preferences, statistics-anonymous, statistics or marketing category should be passed when using wp_setcookie.', 'wp-consent-api' ) );
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rlankhorst rlankhorst merged commit d64b315 into WordPress:master Nov 10, 2025
@rlankhorst
Copy link
Collaborator

good point, but will be upgrading to 7.4 as minimum required. Pushing a MR for that now.

@szepeviktor szepeviktor deleted the php-ver branch November 10, 2025 11:27
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.

2 participants