WordPress.org

Make WordPress Core

Opened 6 months ago

Last modified 8 weeks ago

#48350 reviewing enhancement

Stricter docs for WP_Http::processHeaders()

Reported by: marekdedic Owned by: SergeyBiryukov
Milestone: 5.5 Priority: normal
Severity: minor Version: 5.2
Component: HTTP API Keywords: good-first-bug has-patch needs-refresh
Focuses: docs Cc:

Description

I believe the function could better document the parameter and return types. From a look at the source, maybe something like this:

@param string|string[] $headers
@param string          $url

@return array {
    @type array $response {
        @type int    $code
        @type string $message
    }
    @type string|string[]  $newheaders
    @type WP_Http_Cookie[] $cookies
}

+ comments obviously.

Attachments (1)

48350.diff (1.5 KB) - added by valchovski 5 months ago.

Download all attachments as: .zip

Change History (9)

#1 @johnbillion
6 months ago

  • Keywords needs-patch good-first-bug added

#2 @desrosj
6 months ago

  • Milestone changed from Awaiting Review to 5.4

@valchovski
5 months ago

#3 @valchovski
5 months ago

  • Keywords has-patch added; needs-patch removed

Heya!

Attached a file containing expanded documentation for processHeaders().

Looking forward to any feedback!

#4 @SergeyBiryukov
5 months ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#5 @davidbaumwald
2 months ago

@SergeyBiryukov Are you still reviewing this one for 5.4?

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.


8 weeks ago

#7 @davidbaumwald
8 weeks ago

  • Keywords needs-refresh added
  • Milestone changed from 5.4 to Future Release

The most recent patch fails against trunk. This ticket still needs a refreshed patch, and with 5.4 Beta 1 landing tomorrow, this is being moved to Future Release. If any maintainer or committer feels this can be included in 5.4 or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

#8 @SergeyBiryukov
8 weeks ago

  • Milestone changed from Future Release to 5.5
Note: See TracTickets for help on using tickets.