WordPress.org
Get WordPress

Make WordPress Core

Opened 3 weeks ago

Closed 3 weeks ago

#64515 closed defect (bug) (fixed)

Duplicate @see keyword in PHPDoc for wp_should_load_separate_core_block_assets()

Reported by: marian1's profile marian1 Owned by: westonruter's profile westonruter
Milestone: 7.0 Priority: normal
Severity: trivial Version: 6.8
Component: Script Loader Keywords: has-patch good-first-bug
Focuses: docs Cc:

Description

The wp_should_load_separate_core_block_assets() function contains a malformed @see tag with the keyword duplicated (@see @see wp_should_load_...() instead of @see wp_should_load_...()).

Change History (5)

#1 @sabernhardt
3 weeks ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 7.0
  • Version set to 6.8

Good catch. That was part of [59823], and the ​Developer Resources page creates an extra link from it right now.

Would you like to make the patch in script-loader.php, or should I mark this as a good first bug?

This ticket was mentioned in ​PR #10744 on ​WordPress/wordpress-develop by ​@mukesh27.
3 weeks ago #2

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/64515

#3 @mukesh27
3 weeks ago

  • Keywords good-first-bug added

I’ve opened a draft PR in case any new contributors would like to review it. If someone prefers to open a similar PR themselves, feel free to close mine.

Happy contributing! 🎉

#4 @showravhasan
3 weeks ago

Reviewed PR 10744 on GitHub.

The fix correctly removes the duplicate @see tag. LGTM! 👍

Last edited 3 weeks ago by showravhasan (previous) (diff)

#5 @westonruter
3 weeks ago

  • Owner set to westonruter
  • Resolution set to fixed
  • Status changed from new to closed

In 61496:

Docs: Remove duplicated @see PHPDoc tag.

Follow-up to [59823].

Props marian1, showravhasan, sabernhardt, mukesh27.
See #61965, #64224.
Fixes #64515.

Note: See TracTickets for help on using tickets.