Make WordPress Core

Opened 5 months ago

Closed 5 months ago

Last modified 4 months ago

#62112 closed defect (bug) (fixed)

Twenty Nineteen : PHPCS Fixes

Reported by: pitamdey's profile pitamdey Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.7 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: docs, coding-standards Cc:

Description

In theme Twenty-Nineteen, I can see there are a few PHPCS errors, so I have applied a patch to fix them.

Attachments (3)

phpcs-fixes.patch (2.5 KB) - added by pitamdey 5 months ago.
Patch for this issue
62112.patch (2.3 KB) - added by viralsampat 5 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
62112.2.patch (2.5 KB) - added by viralsampat 4 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.

Download all attachments as: .zip

Change History (9)

@pitamdey
5 months ago

Patch for this issue

#1 @pitamdey
5 months ago

The PHPCS errors that were fixed are :

  • Doc comments are missing
  • The closing brace for the class must go on the next line after the body
  • There must be exactly one blank line after the file comment

#2 @SergeyBiryukov
5 months ago

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

In 59090:

Twenty Nineteen: Add missing documentation for helper function parameters.

Follow-up to [43808], [44149], [47214], [47242].

Props pitamdey.
Fixes #62112.

#3 @SergeyBiryukov
5 months ago

  • Milestone changed from Awaiting Review to 6.7

@viralsampat
5 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

#4 @SergeyBiryukov
5 months ago

In 59102:

Twenty Nineteen: Add more missing documentation for helper function parameters.

Follow-up to [59090].

Props viralsampat.
See #62112.

#5 @SergeyBiryukov
5 months ago

In 59103:

Twenty Nineteen: Remove trailing spaces in DocBlocks.

Follow-up to [59102].

Props mukesh27.
See #62112.

@viralsampat
4 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

#6 @sabernhardt
4 months ago

62112.2.patch proposes editing documentation in other themes, but those were already fixed last year (please update your files).

  • [56045] edited documentation for Twenty Twenty's filters.
  • [56118] documented the $current_screen and $is_IE variables in Twenty Twenty-One.
Note: See TracTickets for help on using tickets.