You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Plugin Directory: Show closed plugins by their slug everywhere on the front end - #1190
Closed and disabled plugins show their slug instead of their name in the plugin header (https://meta.trac.wordpress.org/changeset/6469), but the <title>, og:title, and the Contributors & Developers section (“Display Widgets” is open source software…) still use the plugin name.
As suggested in the ticket, this moves the substitution into the_title and single_post_title filters in the theme, scoped to closed and disabled plugin posts. The filters are added on template_redirect, so they only apply to front-end page loads and leave wp-admin and the REST API alone. The header now reads the filtered title like everything else.
Testing
On a local Plugin Directory environment, close a plugin (or set a plugin post’s status to closed or disabled).
Visit its page, e.g. /plugins/<slug>/.
Check that the <h1 class="plugin-title">, the <title>, the og:title meta tag, and the “… is open source software” sentence in the Contributors & Developers section all show the slug.
Check that a published plugin still shows its name in all of those places, and that the closed plugin still shows its name in wp-admin.
Before, on production: https://wordpress.org/plugins/display-widgets/ has <h1>display-widgets</h1> but <title>Display Widgets – WordPress plugin | WordPress.org</title>, og:title="Display Widgets", and “Display Widgets” is open source software.
Checked in wp-env with closed, disabled, and published plugin posts: get_the_title(), the_title_attribute(), single_post_title(), and wp_get_document_title() return the slug for the first two and the name for the published one.
Closed or disabled plugin pages now display the plugin’s URL-friendly name as the title. Titles for other plugins remain unchanged. This keeps page titles consistent with the name used in the plugin’s web address when a plugin is no longer available.
… front end
Closed and disabled plugins already show their slug instead of their name
in the plugin header, see [6469].
The document title, og:title, and the Contributors & Developers section
still used the plugin name.
Filter the_title and single_post_title on the front end instead, so every
place that reads the title gets the slug, and drop the header's own
special case.
Fixes #8111.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No actionable comments were generated in the recent review.
Walkthrough
The theme adds title filters for plugin posts with status closed or disabled. Those filters return the post slug. The single-plugin template now obtains its title through get_the_title().
Changes: Plugin titles
Layer / File(s)
Summary
Register and apply plugin title filters wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/functions.php, wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/template-parts/plugin-single.php
The theme registers filters for the_title and single_post_title. For plugin posts with status closed or disabled, the filters return the post slug; other posts retain the supplied title. The single-plugin template now uses get_the_title() for its title.
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title clearly describes the main change: showing closed plugin slugs on the front end.
Docstring Coverage
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 1 functions across 2 files.
Finishing Touches
Generate unit tests (beta)
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.
The reason will be displayed to describe this comment to others. Learn more.
🟡 Changes recommended
The title-filter callback does not resolve the current post when its optional argument is absent, so disabled or closed plugins can still expose their names.
Updates the Plugin Directory theme so closed and disabled plugins consistently display their slug on the front end while preserving their names in wp-admin and the REST API.
Changes:
Adds front-end the_title and single_post_title filters for closed and disabled plugins.
Uses the filtered title for the plugin header.
Applies the slug consistently to document titles, Open Graph metadata, and contributor messaging.
The single_post_title callback currently receives no post argument from WordPress core and therefore does not apply the slug substitution. Resolve the queried post when the argument is absent.
Adds front-end title filters. Moderate issue: the single_post_title callback needs to resolve the queried post when no post argument is provided. (2 votes)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes https://meta.trac.wordpress.org/ticket/8111
Closed and disabled plugins show their slug instead of their name in the plugin header (https://meta.trac.wordpress.org/changeset/6469), but the
<title>,og:title, and the Contributors & Developers section (“Display Widgets” is open source software…) still use the plugin name.As suggested in the ticket, this moves the substitution into
the_titleandsingle_post_titlefilters in the theme, scoped to closed and disabled plugin posts. The filters are added ontemplate_redirect, so they only apply to front-end page loads and leave wp-admin and the REST API alone. The header now reads the filtered title like everything else.Testing
closedordisabled)./plugins/<slug>/.<h1 class="plugin-title">, the<title>, theog:titlemeta tag, and the “… is open source software” sentence in the Contributors & Developers section all show the slug.Before, on production: https://wordpress.org/plugins/display-widgets/ has
<h1>display-widgets</h1>but<title>Display Widgets – WordPress plugin | WordPress.org</title>,og:title="Display Widgets", and “Display Widgets” is open source software.Checked in wp-env with closed, disabled, and published plugin posts:
get_the_title(),the_title_attribute(),single_post_title(), andwp_get_document_title()return the slug for the first two and the name for the published one.🤖 Generated with Claude Code
Summary by CodeRabbit