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

Bump Min WP Version to 6.4 #1287

Draft
wants to merge 2 commits into
base: trunk
Choose a base branch
from
Draft

Bump Min WP Version to 6.4 #1287

wants to merge 2 commits into from

Conversation

obenland
Copy link
Member

@obenland obenland commented Feb 7, 2025

Let's bump the min version to 6.4. The last three versions is what the Plugin Directory expects plugins to be tested with and it gets us guaranteed WP_HTML_Tag_Processor support. 6.4 is also the first version that supports PHP 8.3.

Based on https://fedi.wrm.sr/software/wordpress/versions, 126 installs are below 6.4, or around 1.47% of all ActivityPub installs.

Proposed changes:

  • Bumps min WP version to 6.4.
  • Removes now unnecessary class_exists checks and fallbacks.

Other information:

  • Have you written new tests for your changes, if applicable?

}

if (
! \function_exists( 'register_block_type_from_metadata' ) ||
Copy link
Member

Choose a reason for hiding this comment

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

let's keep that, so that it will still work with classicpress or the classic editor enabled.

Copy link
Member Author

Choose a reason for hiding this comment

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

The functions exist with the classic editor enabled. Should ClassicPress be a consideration?

*
* @return array The attachment IDs.
*/
protected function get_classic_editor_image_attachments( $max_images ) {
Copy link
Member

Choose a reason for hiding this comment

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

let's keep that, for people that still use the classic editor

Copy link
Member

Choose a reason for hiding this comment

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

Ah, this is only about the attachments 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

Good to remove?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants