WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#34197 closed enhancement (fixed)

Improve the parameter names and documentation for add_rewrite_rule()

Reported by: johnbillion Owned by: johnbillion
Priority: normal Milestone: 4.4
Component: Rewrite Rules Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

[34708] has highlighted the fact that the parameter names and docblock descriptions for add_rewrite_rule(), WP_Rewrite::add_rule(), and WP_Rewrite::add_external_rule() make no sense.

Attachments (1)

34197.diff​ (5.0 KB ) - added by johnbillion 11 years ago.

Download all attachments as: .zip

Change History (3)

@johnbillion
11 years ago

#1 @johnbillion
11 years ago

  • Keywords has-patch added

34197.diff​ changes the name of the $redirect parameter to $query, which is more appropriate, and updates the documentation accordingly.

#2 @johnbillion
11 years ago

  • Owner set to johnbillion
  • Resolution → fixed
  • Status new → closed

In 34946:

Improve the parameter names and inline documentation for add_rewrite_rule(), WP_Rewrite::add_rule(), and WP_Rewrite::add_external_rule().

Fixes #34197

Note: See TracTickets for help on using tickets.