Opened 5 months ago
Closed 4 months ago
#63692 closed defect (bug) (fixed)
Bundled Theme: Add missing PHPDoc Block for the action.
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.9 | Priority: | normal |
| Severity: | minor | Version: | |
| Component: | Bundled Theme | Keywords: | has-patch changes-requested |
| Focuses: | docs | Cc: |
Description
Hello Team,
I reviewed the theme files and noticed the use of a PHPDoc block for the twentynineteen_scripts & twentyseventeen_scripts action hook in the Twenty Nineteen & Twenty Seventeen themes.
I think that it should be appropriate, as it helps with documentation and clarity.
Thanks,
Attachments (1)
Change History (34)
#1
@
5 months ago
- Keywords changes-requested added; needs-testing removed
- Severity changed from normal to minor
- Version set to trunk
There is a difference between the function comment. Enqueues vs Enqueue: should make it consistent.
This ticket was mentioned in PR #9294 on WordPress/wordpress-develop by @vladimiraus.
5 months ago
#2
Theme consistency in php doc.
#4
@
5 months ago
- Keywords dev-feedback 2nd-opinion reporter-feedback removed
- Milestone changed from Awaiting Review to 6.9
- Version trunk deleted
Thanks for the PR!
It looks good to me.
Move to 6.9
#5
@
5 months ago
- Summary changed from Twenty Nineteen & Twenty Seventeen: Add PHPDoc Block for the twentynineteen_scripts & twentyseventeen_scripts action. to Bundled Theme: Add missing PHPDoc Block for the action.
#7
@
5 months ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 60495:
#8
follow-up:
↓ 9
@
5 months ago
@SergeyBiryukov could you please commit changes for src/wp-content/themes/twentyfifteen/functions.php. The PR https://github.com/WordPress/wordpress-develop/pull/9294 have that changes.
#9
in reply to:
↑ 8
@
5 months ago
Replying to mukesh27:
could you please commit changes for
src/wp-content/themes/twentyfifteen/functions.php
I left that out for now, as those older themes are not super consistent with using third-person singular verbs in function descriptions, and there are at least 8 other instances of "Enqueue scripts" in various themes. So I think fixing just that one instance would not really add any consistency, see also twentyfifteen_block_editor_styles() for example.
Happy to reopen if someone is interested in updating all of those DocBlocks to use third-person singular verbs :)
This ticket was mentioned in PR #9305 on WordPress/wordpress-develop by @vladimiraus.
5 months ago
#11
Third-person singular verbs update for themes.
I have reviewed above mentioned issue and resolved it. Here, I have added its patch.