Opened 6 years ago
Closed 4 years ago
#44253 closed defect (bug) (fixed)
@return missing in doc comment of get_admin_page_parent function
Reported by: | subrataemfluence | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | dev-feedback |
Focuses: | docs | Cc: |
Description
@return is missing in the Doc comment of get_admin_page_parent function.
Also I have a question. At the end of the function body we are returning hard coded empty string i.e. return ''
;
Why don't we return the $parent_file
instead? Since we are already returning the value from rest of the if-else
blocks and foreach
loop the last final statement either will return an empty
string or a value assigned to $parent_file
variable.
<?php if( empty( $parent_file ) ) { $parent_file = ''; } return $parent_file;
Attachments (3)
Change History (9)
#2
@
6 years ago
Given PHPs type coercion, this really could just return $parent_file;
without any check for empty, the caller should be responsible for checking for empty().
Version 0, edited 6 years ago
by
(next)
#3
@
6 years ago
@kennithnichol I understand the concern! Thank you for explaining and correcting me.
Note: See
TracTickets for help on using
tickets.
Since
get_admin_page_title
function is also missing the @return in Doc comment I am uploading a revised patch which combines both the functions.