Opened 9 years ago
Closed 9 years ago
#36912 closed defect (bug) (fixed)
Improve documentation for install_plugin_install_status()
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.7 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Plugins | Keywords: | good-first-bug has-patch |
| Focuses: | docs | Cc: |
Description
The helper function install_plugin_install_status() is currently lacking some proper documentation. Besides not stating what exactly can be passed to the function, the return type is currently declared as type instead of array.
Attachments (6)
Change History (22)
#1
@
9 years ago
- Keywords has-patch added; needs-patch removed
- Owner set to clarionwpdeveloper
- Status changed from new to assigned
This ticket was mentioned in Slack in #core by mynamvenu24. View the logs.
9 years ago
#3
@
9 years ago
- Milestone changed from Awaiting Review to Future Release
The patch needs a little bit more work. It needs to explain what exactly $loop does and what kind of array is being returned.
#5
@
9 years ago
@clarionwpdeveloper thanks for the updated patch. We should use the hash-notation style to document the return array. See the return notation for get_plugin_data() for a hint on how to do that.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
9 years ago
#10
@
9 years ago
- Owner changed from clarionwpdeveloper to DrewAPicture
- Status changed from assigned to reviewing
#12
@
9 years ago
@sudar Thanks, this is getting along nicely.
Note that optional parameters should be documented according to the standards, see https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#docblock-formatting. Also, Date about the plugin should probably be something like Data about the plugin retrieved from the API.
#13
@
9 years ago
@swissspidy Thanks for reviewing it.
I am attaching the modified patch after implementing the changes you suggested.
Assigning to mark the good-first-bug as "claimed".
See plugin-install.php.patch