Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#44939 closed enhancement (fixed)

Add public to WP_Rest_Server test class

Reported by: andizer's profile andizer Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.1 Priority: normal
Severity: normal Version:
Component: REST API Keywords: has-patch
Focuses: rest-api Cc:


I think it would be nice to be explicit about the access modifier, so I added public to the methods where no access modifier was given

Attachments (1)

44939.diff (2.0 KB) - added by andizer 5 years ago.

Download all attachments as: .zip

Change History (5)

5 years ago

#1 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 4.9.9
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#2 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 43652:

Tests: Add missing public access modifier to Tests_REST_Server methods.

Props andizer.
Fixes #44939.

#3 @SergeyBiryukov
5 years ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening for 4.9.9 consideration.

#4 @pento
5 years ago

  • Keywords fixed-major removed
  • Milestone changed from 4.9.9 to 5.1
  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.