Skip to content

Fix bugs of ACF and featured media - #91

Open
ixkaito wants to merge 7 commits into
humanmade:masterfrom
ixkaito:fix-acf-and-featured-media
Open

Fix bugs of ACF and featured media#91
ixkaito wants to merge 7 commits into
humanmade:masterfrom
ixkaito:fix-acf-and-featured-media

Conversation

@ixkaito

@ixkaito ixkaito commented Sep 6, 2022

Copy link
Copy Markdown

This PR fixes the following.

  • wp_make_content_images_responsive() is deprecated and the wp_make_content_images_responsive hook was removed.
    • Use wp_filter_content_tags() and the wp_filter_content_tags hook instead.
  • Fix ACF bug with repeater fields
    • We are using the wrong hook to get the value. We have to get the value before formatted and overwrite it after formatted via acf/format_value/type={$type}.
  • Fix bugs of saving featured images via REST API (i.e. Block Editor)
    • We have to use the rest_pre_insert_{$post_type} hook to get the featured image ID

joiglifberg and others added 2 commits October 3, 2019 08:44
… previous image or file field data due to filter order.
- Update for core and
- Fix bugs of ACF Pro (or repeater field)
- Fix bugs of featured media
@ixkaito ixkaito mentioned this pull request Sep 6, 2022
@jonnyvaughan

Copy link
Copy Markdown

Thank you!

dsaha1656 and others added 5 commits July 30, 2025 08:40
Fix: Prevent PHP Warning when get_current_screen() returns null
Fix ACF bug with image and file fields in repeatable fields returning the previous image or file field data due to filter order.
Comment thread network-media-library.php
Comment on lines +408 to +409
remove_filter( 'the_content', 'wp_filter_content_tags' );
add_filter( 'the_content', __NAMESPACE__ . '\filter_content_tags' );

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

should we create some backwards compatibility here? wp_filter_content_tags is only available as of WordPress v5.5, and this repo says it's compatible to WordPress 4.9

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.

6 participants