Opened 8 years ago
Closed 8 years ago
#39371 closed defect (bug) (fixed)
Fix DocBlocks for duplicate hooks
Reported by: | keesiemeijer | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch needs-handbook |
Focuses: | docs | Cc: |
Description
DocBlocks should start with /**
to be recognized as a DocBlock by the phpdoc-parser. Duplicate undocumented hooks are imported if normal comments (/* */) are used.
These hooks don't have a correct DocBlock.
load-widgets.php
widgets.php
sidebar_admin_setup
comment_email
session_token_manager
rest_insert_attachment
rest_insert_comment
rest_insert_{$this->post_type}
rest_insert_{$this->taxonomy}
rest_insert_user
Attachments (1)
Change History (9)
#2
@
8 years ago
- Keywords needs-handbook added
We should add something to the handbook here documenting this:
#3
@
8 years ago
I just found out the hooks are in fact, not imported when a DocBlock isn't found by the parser. But we should adhere to the DocBlock syntax /**
where possible.
There is a note for multi-line comments in the handbook documenting this.
Note: See
TracTickets for help on using
tickets.
Fix DocBlocks for duplicate hooks