#51050 closed defect (bug) (fixed)
Docs: Add docblock for get_the_archive_title() filter
Reported by: | justinahinon | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.5.1 | Priority: | normal |
Severity: | minor | Version: | 5.5 |
Component: | Themes | Keywords: | has-patch 2nd-opinion |
Focuses: | docs | Cc: |
Description
In [48294] a span element was introduced to wrap around the get_the_archive_title() filter. Details for this changes is needed in the filter docblock.
Attachments (1)
Change History (8)
#1
@
4 years ago
- Focuses docs added
- Keywords has-patch added
- Milestone changed from Awaiting Review to 5.5.1
#2
follow-up:
↓ 5
@
4 years ago
- Focuses template removed
- Keywords 2nd-opinion added
- Milestone changed from 5.5.1 to Awaiting Review
#5
in reply to:
↑ 2
@
4 years ago
Replying to audrasjb:
I guess you're rather talking about
get_the_archive_title
function.
Yes Jb. I was referring to the function https://developer.wordpress.org/reference/functions/get_the_archive_title.
Thanks for the patch.
Note: See
TracTickets for help on using
tickets.
Thanks for opening the ticket, @justinahinon,
By the way, this filter is actually properly documented: https://developer.wordpress.org/reference/hooks/get_the_archive_title/
"@since 5.5.0 Added the
$prefix
and$original_title
parameters."I guess you're rather talking about
get_the_archive_title
function.In 51050.diff, I added an @since mention, but not completely sure it's necessary as the filter is already documented. I'd defer on @SergeyBiryukov ’s judgement for that :)