Opened 6 years ago
Closed 4 years ago
#46011 closed defect (bug) (duplicate)
Argument types mismatch
Reported by: | subrataemfluence | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | has-patch |
Focuses: | docs, coding-standards | Cc: |
Description
A lot of areas of wp-inclues/comment-template.php
arguments types described in function Doc-blocks are reported mismatched with that of actual arguments in function declaration.
I searched Trac repo and unable to find a ticket specific to this.
Attachments (2)
Change History (6)
#1
follow-ups:
↓ 2
↓ 3
@
5 years ago
- Version 5.0.3 deleted
Clearing the version since this isn't specific to 5.0.3 and these docblocks apply to functions added at different times.
Also just a suggestion to each of you @mukesh27 and @subrataemfluence. When you make your patches, you should generate them from the root, so it would include src/
in the path of the file(s) being edited.
#2
in reply to:
↑ 1
@
5 years ago
Thank you @earnjam . I will follow this in my future patches.
Replying to earnjam:
Clearing the version since this isn't specific to 5.0.3 and these docblocks apply to functions added at different times.
Also just a suggestion to each of you @mukesh27 and @subrataemfluence. When you make your patches, you should generate them from the root, so it would include
src/
in the path of the file(s) being edited.
#3
in reply to:
↑ 1
@
5 years ago
@earnjam Thanks for your guidance. Will follow it in future patches.
Replying to earnjam:
Clearing the version since this isn't specific to 5.0.3 and these docblocks apply to functions added at different times.
Also just a suggestion to each of you @mukesh27 and @subrataemfluence. When you make your patches, you should generate them from the root, so it would include
src/
in the path of the file(s) being edited.
Updated Patch. Change some docs and change false placement as per other docs.