mirror of
https://github.com/gosticks/wordpress-develop.git
synced 2026-06-28 14:20:15 +00:00
Coding Standards: Reorder conditionals in is_random_header_image().
This aims to slightly improve performance by checking the faster `empty()` language construct first and potentially avoiding an unnecessary function call. Additionally, this better matches a similar conditional a few lines below. Follow-up to [17757], [17770]. Props Cybr. See #59650. git-svn-id: https://develop.svn.wordpress.org/trunk@57139 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
@@ -1437,14 +1437,16 @@ function is_random_header_image( $type = 'any' ) {
|
||||
if ( 'any' === $type ) {
|
||||
if ( 'random-default-image' === $header_image_mod
|
||||
|| 'random-uploaded-image' === $header_image_mod
|
||||
|| ( '' !== get_random_header_image() && empty( $header_image_mod ) )
|
||||
|| ( empty( $header_image_mod ) && '' !== get_random_header_image() )
|
||||
) {
|
||||
return true;
|
||||
}
|
||||
} else {
|
||||
if ( "random-$type-image" === $header_image_mod ) {
|
||||
return true;
|
||||
} elseif ( 'default' === $type && empty( $header_image_mod ) && '' !== get_random_header_image() ) {
|
||||
} elseif ( 'default' === $type
|
||||
&& empty( $header_image_mod ) && '' !== get_random_header_image()
|
||||
) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user