Opened 9 years ago
Closed 8 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.
8 years ago
#10
@
8 years ago
- Owner changed from clarionwpdeveloper to DrewAPicture
- Status changed from assigned to reviewing
#12
@
8 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
@
8 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