![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 2 years ago
Closed 2 years ago
#61124 closed enhancement (fixed)
Missing doc comment text
| Reported by: | yagniksangani | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.6 |
| Component: | Upgrade/Install | Version: | |
| Severity: | normal | Keywords: | has-patch 2nd-opinion |
| Cc: | Focuses: | docs |
Description
There is missing doc comment text in the file "\wp-admin\includes\class-bulk-plugin-upgrader-skin.php" for function "public function bulk_footer() {" at line number 42.
Attachments (2)
Change History (7)
#1
@
2 years ago
- Keywords 2nd-opinion added
Thanks for the ticket,
I'm not sure these docblock description suggestions provide enough value as it, but if we want to add these then please note that (at least) the following similar files should also be updated:
- class-bulk-upgrader-skin.php
- class-bulk-theme-upgrader-skin.php
#2
@
2 years ago
- Keywords 2nd-opinion removed
Found some more files where missing comments,
\wp-admin\includes\class-bulk-theme-upgrader-skin.php
\wp-admin\includes\class-bulk-upgrader-skin.php
\wp-admin\includes\class-ftp-pure.php
\wp-admin\includes\class-ftp-sockets.php
\wp-admin\includes\class-ftp.php
I think there are many more files where missing doc comments.
#3
@
2 years ago
- Keywords 2nd-opinion added
Re-adding 2nd-opinion to gather feedback from other developers since I'm unsure these changes are worth the effort :)
Added missing doc comments