Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#25469 closed defect (bug) (fixed)

Hook Docs: wp-settings.php

Reported by: miyauchi Owned by: DrewAPicture
Priority: normal Milestone: 3.7
Component: Inline Docs Version:
Severity: normal Keywords: needs-patch
Cc: Focuses:

Description

Attached patch has done for the all hooks in wp-settings.php

Attachments (3)

wp-settings.diff (1.8 KB ) - added by miyauchi 13 years ago.
patch for wp-settings.php
wp-settings.2.diff (1.8 KB ) - added by miyauchi 13 years ago.
some fix
25469.diff (1.9 KB ) - added by DrewAPicture 13 years ago.
Final.

Download all attachments as: .zip

Change History (8)

@miyauchi
13 years ago

patch for wp-settings.php

#1 @DrewAPicture
13 years ago

  • Owner set to DrewAPicture
  • Status newreviewing

#2 follow-up: @DrewAPicture
13 years ago

  • Keywords needs-patch added; has-patch removed
  • Type enhancementdefect (bug)

Hi, thanks for the patch. Some notes on wp-settings.diff:

plugins_loaded hook:

  • "Fires once plugins are loaded."

sanitize_comment_cookies hook:

  • Pretty vague. Maybe "Fires when comment cookies are sanitized."

setup_theme hook:

  • Should be a complete sentence. Maybe "Fires before the theme is loaded."

after_setup_theme hook:

  • "Fires after the theme is loaded."

init hook:

  • "Runs" should be "Fires"

@miyauchi
13 years ago

some fix

#3 in reply to: ↑ 2 @miyauchi
13 years ago

Hi, I was fixed! Thanks. :)

@DrewAPicture
13 years ago

Final.

#4 @DrewAPicture
13 years ago

  • Milestone Awaiting Review3.7

#5 @DrewAPicture
13 years ago

  • Resolutionfixed
  • Status reviewingclosed

In 25721:

Inline documentation for hooks in wp-settings.php.

Props miyauchi.
Fixes #25469

Note: See TracTickets for help on using tickets.