Hook Docs (17): wp-includes/rewrite.php

[mboynes]

Attached patch has docs for the following hooks in wp-includes/rewrite.php:

• url_to_postid
• post_rewrite_rules
• date_rewrite_rules
• root_rewrite_rules
• comments_rewrite_rules
• search_rewrite_rules
• author_rewrite_rules
• page_rewrite_rules
• $permastructname . '_rewrite_rules'
• tag_rewrite_rules
• generate_rewrite_rules
• rewrite_rules_array
• mod_rewrite_rules
• rewrite_rules
• iis7_url_rewrite_rules
• permalink_structure_changed

[mboynes]

Adding inline docs for hooks in wp-includes/rewrite.php

[DrewAPicture]

Hi, thanks for the patch. I should have a more thorough review for you later today, but it looks like most of the long descriptions will need a once-over, as well as reverting some of the whitespacing changes that aren't directly related the hooks we're documenting here.

[DrewAPicture]

2nd pass

[DrewAPicture]

26095.diff​ cleans up a bunch of descriptions, reverts unnecessary code changes and other changes.

The language could all use a once-over, with particular attention to the mod_rewrite_rules, rewrite_rules and iis7_url_rewrite_rules filters. Not 100 percent on the wording of those descriptions.

[DrewAPicture]

final pass

[DrewAPicture]

In 27341:

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

Props mboynes for the initial patch.
Fixes #26095.

[DrewAPicture]

In 27346:

Fix a typo in the author_rewrite_rules hook doc.

Props kpdesign.
See #26095.