Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#56596 closed defect (bug) (fixed)

Docs: Default parameter not mentioned on strip_core_block_namespace

Reported by: rakibwordpress's profile rakibwordpress Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.1 Priority: normal
Severity: normal Version: 6.0.1
Component: Editor Keywords: has-patch changes-requested
Focuses: docs Cc:

Description (last modified by sabernhardt)

In blocks.php function inline document $block_name by default is null and this is not mentioned on this function strip_core_block_namespace

Change History (6)

This ticket was mentioned in PR #3272 on WordPress/wordpress-develop by rakibwebdev.


3 years ago
#1

  • Keywords has-patch added; needs-patch removed

This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

Trac ticket:

#2 @ironprogrammer
3 years ago

  • Component changed from Security to Text Changes
  • Keywords changes-requested added

Welcome to Trac, @rakibwordpress, and thank you for your ticket submission and follow up PR! 🙌🏻🙌🏻

For the PR, I would recommend checking out the WordPress PHP Documentation Standards for details on how the @param tag should be formatted. For example, I think you should be able to simplify this to Default null. (Please note punctuation, too -- these tags convert to official documentation, so the rules are very strict.)

Last edited 3 years ago by ironprogrammer (previous) (diff)

#3 @sabernhardt
3 years ago

  • Component changed from Text Changes to Editor
  • Description modified (diff)
  • Focuses docs added

#4 @SergeyBiryukov
3 years ago

  • Milestone changed from Awaiting Review to 6.1

#5 @SergeyBiryukov
3 years ago

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

In 54520:

Docs: Clarify default values for a few block function parameters.

This applies to:

  • $block_name in strip_core_block_namespace()
  • $allowed_html in filter_block_content()
  • $pagination_type in get_comments_pagination_arrow()

Follow-up to [46896], [48794], [53138], [54181].

Props rakibwordpress, ironprogrammer, sabernhardt, SergeyBiryukov.
Fixes #56596.

@SergeyBiryukov commented on PR #3272:


3 years ago
#6

Thanks for the PR! Merged in r54520.

Note: See TracTickets for help on using tickets.