WordPress.org

Make WordPress Core

Changes between Initial Version and Version 2 of Ticket #26206


Ignore:
Timestamp:
11/24/2013 04:49:03 PM (8 years ago)
Author:
kpdesign
Comment:

@richard2222: Thanks for the patch. :)

Your changes are to functional docs, rather than documenting the two hooks in the file (wp_cache_themes_persistently and allowed_themes).

I've changed the subject of this ticket to reflect that, and this patch will be reviewed in that context.

Please read the PHP Documentation Standard section on documenting Hooks (Actions and Filters) to familiarize yourself with how the hook doc blocks and duplicate hooks should be formatted.

wp_cache_themes_persistently is being documented in the patch on #25588, so it would be considered a duplicate hook in this file.

The allowed_themes hook would need to be fully documented per the standard.

If you would like to also document the hooks, create a new patch containing just the hook doc changes, then open a new ticket for that patch please. :)

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #26206

    • Property Version changed from trunk to
    • Property Type changed from enhancement to defect (bug)
    • Property Summary changed from Hook Docs: wp-includes/class-wp-theme.php to PHPDoc Updates for wp-includes/class-wp-theme.php
  • Ticket #26206 – Description

    initial v2  
    1 Attached patch for functions
    2     _name_sort
    3   _name_sort_i18n
    4 in wp-includes/class-wp-theme.php
     1Attached patch for functional doc changes in wp-includes/class-wp-theme.php.