Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#25467 closed enhancement (fixed)

Hook Docs: wp-includes/update.php

Reported by: ShinichiN Owned by: kpdesign
Priority: normal Milestone: 3.7
Component: Inline Docs Version:
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

Adding inline documentation for all hooks in wp-includes/update.php

Attachments (5)

25467.diff​ (1.6 KB ) - added by ShinichiN 13 years ago.
A inline doc patch for wp-includes/update.php
25467-2.diff​ (1.6 KB ) - added by ShinichiN 13 years ago.
25467-3.diff​ (2.6 KB ) - added by ShinichiN 13 years ago.
Bug fix.
25467-4.diff​ (2.6 KB ) - added by ShinichiN 13 years ago.
Converted tab to space to make it more readable.
25467-5.diff​ (2.6 KB ) - added by ShinichiN 13 years ago.
I got rid of the noise tabs and spaces.

Download all attachments as: .zip

Change History (11)

@ShinichiN
13 years ago

A inline doc patch for wp-includes/update.php

@ShinichiN
13 years ago

@ShinichiN
13 years ago

Bug fix.

@ShinichiN
13 years ago

Converted tab to space to make it more readable.

#1 @DrewAPicture
13 years ago

  • Owner set to kpdesign
  • Status new → reviewing

#2 @kpdesign
13 years ago

@ShinichiN: Thanks for the patch. :)

I noticed a couple small things that should be fixed.

  • The @param array for $updates_counts_title: The short description and both @type lines are using tabs for indentation. They should be indented using spaces instead.
  • The line after the short description in the same docblock has an extra space and a tab after the *. The closing */ also has an extra tab after it.

It would be helpful to set your text editor to make whitespace characters visible.

Could you fix these things and resubmit the patch? :)

#3 @ShinichiN
13 years ago

Thank you for the review. I fix it now.

@ShinichiN
13 years ago

I got rid of the noise tabs and spaces.

#4 @kpdesign
13 years ago

  • Keywords commit added
  • Status reviewing → accepted

@ShinichiN: Thanks for fixing those - patch looks good. :)

Recommend commit.

#5 @DrewAPicture
13 years ago

  • Resolution → fixed
  • Status accepted → closed

In 25699:

Inline documentation for hooks in wp-includes/update.php.

Props ShinichiN.
Fixes #25467.

#6 @DrewAPicture
13 years ago

  • Milestone Awaiting Review → 3.7
Note: See TracTickets for help on using tickets.