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

WP-r48241: Administration: Pass the result of set-screen-option fil… #589

Merged
merged 1 commit into from
Aug 8, 2020
Merged

WP-r48241: Administration: Pass the result of set-screen-option fil… #589

merged 1 commit into from
Aug 8, 2020

Conversation

mattyrob
Copy link
Collaborator

@mattyrob mattyrob commented Jul 2, 2020

…ter to the new set_screen_option_{$option} filter to ensure backward compatibility.

Rename the $keep parameter of both filters to $screen_option for clarity, update the documentation to better reflect its purpose.

WP:Props Chouby, sswells, SergeyBiryukov.
Fixes https://core.trac.wordpress.org/ticket/50392.


Merges https://core.trac.wordpress.org/changeset/48241 / WordPress/wordpress-develop@48f0e74279 to ClassicPress.

Description

Backport of upstream fix

Motivation and context

Fix for upstream ticket 50392

How has this been tested?

Backported through all supported branched of WP

Types of changes

  • Bug fix

…ter to the new `set_screen_option_{$option}` filter to ensure backward compatibility.

Rename the `$keep` parameter of both filters to `$screen_option` for clarity, update the documentation to better reflect its purpose.

WP:Props Chouby, sswells, SergeyBiryukov.
Fixes https://core.trac.wordpress.org/ticket/50392.

---

Merges https://core.trac.wordpress.org/changeset/48241 / WordPress/wordpress-develop@48f0e74279 to ClassicPress.
Copy link
Contributor

@nylen nylen left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for the PR!

@nylen nylen merged commit 2e9252e into ClassicPress:develop Aug 8, 2020
@mattyrob mattyrob deleted the merge/wp-r48241 branch August 8, 2020 08:34
@ClassyBot
Copy link
Contributor

This pull request has been mentioned on ClassicPress Forums. There might be relevant details there:

https://forums.classicpress.net/t/classicpress-1-2-0-rc1-release-notes/2479/1

@ClassyBot
Copy link
Contributor

This pull request has been mentioned on ClassicPress Forums. There might be relevant details there:

https://forums.classicpress.net/t/classicpress-1-2-0-release-notes/2516/1

@viktorix viktorix added the exclude For filtering purposes only. label Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude For filtering purposes only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants