Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#51441 closed defect (bug) (fixed)

Add brackets to @see tags

Reported by: ravipatel Owned by: SergeyBiryukov
Priority: low Milestone: 5.6
Component: General Version: 5.6
Severity: normal Keywords: has-patch
Cc: Focuses: docs, coding-standards

Description

Cannot be resolved to a type value fixed.

Attachments (3)

51441-general-template.php.patch​ (515 bytes ) - added by ravipatel 6 years ago.
cannot be resolved to a type - document chages
51441.patch​ (971 bytes ) - added by ravipatel 6 years ago.
Merge 2 files patches with same issue
51441-multiple-files.patch​ (3.5 KB ) - added by ravipatel 6 years ago.
Added with multiples files code

Download all attachments as: .zip

Change History (8)

@ravipatel
6 years ago

cannot be resolved to a type - document chages

@ravipatel
6 years ago

Merge 2 files patches with same issue

This ticket was mentioned in ​Slack in #core by ravi. ​View the logs.


6 years ago

#2 @davidbaumwald
6 years ago

  • Keywords needs-refresh added
  • Priority normal → low

@ravipatel Can you refresh these patches against the src? The patch does not include the file path and is more difficult to apply. For instance, shortcodes.php should be src/wp-includes/shortcodes.php.

Here's a Handbook page about creating the patch from src/ ​https://make.wordpress.org/core/handbook/tutorials/trac/submitting-a-patch/

@ravipatel
6 years ago

Added with multiples files code

#3 @davidbaumwald
6 years ago

  • Focuses docs added
  • Keywords needs-refresh removed

@ravipatel Third-party packages, like SimplePie, are imported to the codebase as-is, so they don't have to be updated to the WPCS. The other changes to other files look good though.

#4 @SergeyBiryukov
6 years ago

  • Milestone Awaiting Review → 5.6
  • Owner set to SergeyBiryukov
  • Status new → reviewing
  • Summary Cannot be resolved to a type. → Add brackets to @see tags

#5 @SergeyBiryukov
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 49092:

Docs: Add brackets to function names in some @see tags, for consistency.

Props ravipatel, davidbaumwald.
Fixes #51441.

Note: See TracTickets for help on using tickets.