Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 13 years ago

Closed 12 years ago

#26049 closed defect (bug) (fixed)

Hook Docs (21): wp-includes/class-http.php

Reported by:	kraftbj	Owned by:	DrewAPicture
Priority:	normal	Milestone:	3.9
Component:	HTTP API	Version:
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:	docs

Description (last modified by kpdesign)

Holding ticket for upcoming inline docs patch for wp-includes/class-http.php

Attachments (5)

26049-1.patch (1.5 KB ) - added by kraftbj 13 years ago.: Patch of do_action* hooks documented. TODO: Filters
26049-2.patch (8.6 KB ) - added by kraftbj 13 years ago.: All hooks in /wp-includes/class-http.php documented.
26049-3.patch (9.5 KB ) - added by kpdesign 13 years ago.: First pass on 26049-2.patch
26049.patch (10.1 KB ) - added by DrewAPicture 12 years ago.: 2nd pass
26049.1.patch (10.2 KB ) - added by kpdesign 12 years ago.: Third pass

Download all attachments as: .zip

Change History (15)

@kraftbj
13 years ago

Attachment 26049-1.patch added

Patch of do_action* hooks documented. TODO: Filters

#1 @kpdesign
13 years ago

Description modified (diff)
Owner set to kpdesign
Status new → reviewing
Summary Hook Docs: /wp-includes/class-http.php → Hook Docs: wp-includes/class-http.php

#2 @DrewAPicture
13 years ago

Keywords needs-patch added
Summary Hook Docs: wp-includes/class-http.php → Hook Docs (21): wp-includes/class-http.php

Still needs a complete patch including the filters. Will take a closer look at the sprint tomorrow.

#3 @kraftbj
13 years ago

I've been taking my sweet time, doing a bit at a time. I'll finish this up today or tomorrow.

@kraftbj
13 years ago

Attachment 26049-2.patch added

All hooks in /wp-includes/class-http.php documented.

#4 @kraftbj
13 years ago

Keywords has-patch added; needs-patch removed

@kpdesign
13 years ago

Attachment 26049-3.patch added

First pass on 26049-2.patch

#5 @kpdesign
13 years ago

Keywords docs-feedback added

26049-3.patch fixes some formatting issues, and changes some descriptions for language consistency.

#6 @kraftbj
13 years ago

Thanks for updating the patch. I'll review it to learn for the next one.

@DrewAPicture
12 years ago

Attachment 26049.patch added

2nd pass

#7 @DrewAPicture
12 years ago

Keywords docs-feedback removed
Milestone Awaiting Review → 3.9
Owner changed from kpdesign to DrewAPicture

26049.patch improves many descriptions and parameter descriptions from 2604903.patch.

Version 0, edited 12 years ago by DrewAPicture (next)

#8 @jeremyfelt
12 years ago

Component Inline Docs → HTTP
Focuses docs added

@kpdesign
12 years ago

Attachment 26049.1.patch added

Third pass

#9 @kpdesign
12 years ago

26049.1.patch incorporates some minor changes (typos, better docs-specific variables) to 26049.patch .

@DrewAPicture - I think we've got a commit candidate with this one. What do you think?

#10 @DrewAPicture
12 years ago

Resolution → fixed
Status reviewing → closed

Inline documentation for hooks in wp-includes/class-http.php.

Props kraftbj for the initial patches. Props kpdesign, DrewAPicture.
Fixes #26049.

Note: See TracTickets for help on using tickets.

Download in other formats: