Automattic / Automattic/jetpack
RP & Twenty Fourteen Compat: Remove the_content filters for widget
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
Twenty Fourteen has an Ephemera widget that is used for inserting posts of a certain type in a sidebar. The widget is calling `the_content` ( https://core.trac.wordpress.org/browser/trunk/src/wp-content/themes/twentyfourteen/inc/widgets.php#L193 ) which is calling our `the_content` filters (specifically Likes, Related Posts, Sharing).
Since having the same ID multiple times breaks HTML5 validation and potentially confusing for Related Posts, we should remove our filters as needed for output within this widget.
See https://core.trac.wordpress.org/ticket/37023 cc: @lamdayap
Contributor guide
Research direction
Start with the Twenty Fourteen Ephemera widget in inc/widgets.php around line 193 and trace the Likes, Related Posts, and Sharing filters triggered by its the_content call. Done means those filters are omitted as needed for this widget's output, avoiding duplicate IDs and the reported compatibility problems.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100