WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#25510 closed defect (bug) (fixed)

Hooks Docs: wp-includes/class-feed.php

Reported by: swissspidy Owned by: johnbillion
Priority: normal Milestone: 3.7
Component: Inline Docs Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

There's only 1 hook, so this will be a quick one.

Attachments (3)

25510.diff​ (1.1 KB ) - added by swissspidy 13 years ago.
25510.2.diff​ (1.1 KB ) - added by swissspidy 13 years ago.
25510.3.diff​ (805 bytes ) - added by swissspidy 13 years ago.

Download all attachments as: .zip

Change History (11)

@swissspidy
13 years ago

#1 @swissspidy
13 years ago

  • Keywords has-patch added; needs-patch removed

#2 @johnbillion
13 years ago

  • Keywords needs-refresh added; has-patch removed

I think we should clarify that the lifetime parameter is the cache duration in seconds. The inline doc in 25510.diff​ suggests it might be in hours.

@swissspidy
13 years ago

#3 @swissspidy
13 years ago

  • Keywords has-patch added; needs-refresh removed

You're right. I just updated the doc block to make it more clear.

#4 @johnbillion
13 years ago

Good stuff!

Your latest patch is a patch of your first patch though. Can you provide a unified patch of both changes? Also you might want to remove those Git headers from your patch. See ​http://scribu.net/wordpress/svn-patches-from-git.html

@swissspidy
13 years ago

#5 @swissspidy
13 years ago

All good things go by three...

#6 @DrewAPicture
13 years ago

  • Owner set to johnbillion
  • Status new → reviewing

#7 @DrewAPicture
13 years ago

  • Milestone Awaiting Review → 3.7

#8 @DrewAPicture
13 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 25733:

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

Props swissspidy.
Fixes #25510.

Note: See TracTickets for help on using tickets.