Skip to content

fix: Patch for PHPCS fixing in bundled themes #1395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

fix: Patch for PHPCS fixing in bundled themes #1395

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/wp-content/themes/twentyeleven/functions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ function twentyeleven_header_style() {
$text_color = get_header_textcolor();

// If no custom options for text are set, let's bail.
if ( HEADER_TEXTCOLOR == $text_color ) {
if ( HEADER_TEXTCOLOR === $text_color ) {
return;
}

Expand Down Expand Up @@ -386,7 +386,7 @@ function twentyeleven_admin_header_style() {
}
<?php
// If the user has set a custom color for the text, use that.
if ( get_header_textcolor() != HEADER_TEXTCOLOR ) :
if ( get_header_textcolor() !== HEADER_TEXTCOLOR ) :
?>
#site-title a,
#site-description {
Expand Down Expand Up @@ -724,7 +724,7 @@ function twentyeleven_comment( $comment, $args, $depth ) {
<?php
$avatar_size = 68;

if ( '0' != $comment->comment_parent ) {
if ( '0' !== $comment->comment_parent ) {
$avatar_size = 39;
}

Expand Down Expand Up @@ -756,7 +756,7 @@ function twentyeleven_comment( $comment, $args, $depth ) {
}
?>

<?php if ( '0' == $comment->comment_approved ) : ?>
<?php if ( '0' === $comment->comment_approved ) : ?>
<em class="comment-awaiting-moderation"><?php echo $moderation_note; ?></em>
<br />
<?php endif; ?>
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyeleven/image.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
)
);
foreach ( $attachments as $k => $attachment ) {
if ( $attachment->ID == $post->ID ) {
if ( $attachment->ID === $post->ID ) {
break;
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyeleven/inc/block-patterns.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
* @since Twenty Eleven 1.0
*/

/**
/**
* Register Block Pattern Category.
*/
if ( function_exists( 'register_block_pattern_category' ) ) {
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyeleven/inc/theme-options.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -447,7 +447,7 @@ function twentyeleven_print_link_color_style() {
$default_options = twentyeleven_get_default_theme_options();

// Don't do anything if the current link color is the default.
if ( $default_options['link_color'] == $link_color ) {
if ( $default_options['link_color'] === $link_color ) {
return;
}
?>
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyeleven/showcase.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@
while ( $featured->have_posts() ) :
$featured->the_post();
$counter_slider++;
if ( 1 == $counter_slider ) {
if ( 1 === $counter_slider ) {
$class = ' class="active"';
} else {
$class = '';
Expand Down
4 changes: 2 additions & 2 deletions src/wp-content/themes/twentyfifteen/inc/custom-header.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,11 @@ function twentyfifteen_custom_header_setup() {
function twentyfifteen_hex2rgb( $color ) {
$color = trim( $color, '#' );

if ( strlen( $color ) == 3 ) {
if ( strlen( $color ) === 3 ) {
$r = hexdec( substr( $color, 0, 1 ) . substr( $color, 0, 1 ) );
$g = hexdec( substr( $color, 1, 1 ) . substr( $color, 1, 1 ) );
$b = hexdec( substr( $color, 2, 1 ) . substr( $color, 2, 1 ) );
} elseif ( strlen( $color ) == 6 ) {
} elseif ( strlen( $color ) === 6 ) {
$r = hexdec( substr( $color, 0, 2 ) );
$g = hexdec( substr( $color, 2, 2 ) );
$b = hexdec( substr( $color, 4, 2 ) );
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyfourteen/functions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -461,7 +461,7 @@ function twentyfourteen_the_attached_image() {
// If there is more than 1 attachment in a gallery...
if ( count( $attachment_ids ) > 1 ) {
foreach ( $attachment_ids as $idx => $attachment_id ) {
if ( $attachment_id == $post->ID ) {
if ( $attachment_id === $post->ID ) {
$next_id = $attachment_ids[ ( $idx + 1 ) % count( $attachment_ids ) ];
break;
}
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyfourteen/inc/block-patterns.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
* @since Twenty Fourteen 3.2
*/

/**
/**
* Register Block Pattern Category.
*/
if ( function_exists( 'register_block_pattern_category' ) ) {
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyfourteen/inc/custom-header.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ function twentyfourteen_header_style() {
}
<?php
// If the user has set a custom color for the text, use that.
elseif ( get_theme_support( 'custom-header', 'default-text-color' ) != $text_color ) :
elseif ( get_theme_support( 'custom-header', 'default-text-color' ) !== $text_color ) :
?>
.site-title a {
color: #<?php echo esc_attr( $text_color ); ?>;
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyfourteen/inc/featured-content.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ public static function pre_get_posts( $query ) {
public static function delete_post_tag( $tag_id ) {
$settings = self::get_setting();

if ( empty( $settings['tag-id'] ) || $tag_id != $settings['tag-id'] ) {
if ( empty( $settings['tag-id'] ) || $tag_id !== $settings['tag-id'] ) {
return;
}

Expand Down
4 changes: 2 additions & 2 deletions src/wp-content/themes/twentynineteen/classes/class-twentynineteen-walker-comment.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ protected function html5_comment( $comment, $depth, $args ) {
$comment_author_url = get_comment_author_url( $comment );
$comment_author = get_comment_author( $comment );
$avatar = get_avatar( $comment, $args['avatar_size'] );
if ( 0 != $args['avatar_size'] ) {
if ( 0 !== $args['avatar_size'] ) {
if ( empty( $comment_author_url ) ) {
echo $avatar;
} else {
Expand Down Expand Up @@ -99,7 +99,7 @@ protected function html5_comment( $comment, $depth, $args ) {
}
?>

<?php if ( '0' == $comment->comment_approved ) : ?>
<?php if ( '0' === $comment->comment_approved ) : ?>
<p class="comment-awaiting-moderation"><?php echo $moderation_note; ?></p>
<?php endif; ?>

Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentynineteen/comments.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
_e( 'Leave a comment', 'twentynineteen' );
}
} else {
if ( '1' == $discussion->responses ) {
if ( '1' === $discussion->responses ) {
/* translators: %s: Post title. */
printf( _x( 'One reply on &ldquo;%s&rdquo;', 'comments title', 'twentynineteen' ), get_the_title() );
} else {
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyten/functions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -439,7 +439,7 @@ function twentyten_comment( $comment, $args, $depth ) {
}
?>

<?php if ( '0' == $comment->comment_approved ) : ?>
<?php if ( '0' === $comment->comment_approved ) : ?>
<em class="comment-awaiting-moderation"><?php echo $moderation_note; ?></em>
<br />
<?php endif; ?>
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentyten/loop-attachment.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@
)
);
foreach ( $attachments as $k => $attachment ) {
if ( $attachment->ID == $post->ID ) {
if ( $attachment->ID === $post->ID ) {
break;
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentythirteen/functions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -594,7 +594,7 @@ function twentythirteen_the_attached_image() {
// If there is more than 1 attachment in a gallery...
if ( count( $attachment_ids ) > 1 ) {
foreach ( $attachment_ids as $idx => $attachment_id ) {
if ( $attachment_id == $post->ID ) {
if ( $attachment_id === $post->ID ) {
$next_id = $attachment_ids[ ( $idx + 1 ) % count( $attachment_ids ) ];
break;
}
Expand Down
3 changes: 2 additions & 1 deletion src/wp-content/themes/twentythirteen/image.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@
/* translators: 1: Date, 2: Date, 3, Parent permalink, 4, Post title, 5: Post title. */
$published_text = __( '<span class="attachment-meta">Published on <time class="entry-date" datetime="%1$s">%2$s</time> in <a href="%3$s" title="Go to %4$s" rel="gallery">%5$s</a></span>', 'twentythirteen' );
$post_title = get_the_title( $post->post_parent );
if ( empty( $post_title ) || 0 == $post->post_parent ) {

if ( empty( $post_title ) || 0 === $post->post_parent ) {
$published_text = '<span class="attachment-meta"><time class="entry-date" datetime="%1$s">%2$s</time></span>';
}

Expand Down
4 changes: 2 additions & 2 deletions src/wp-content/themes/twentythirteen/inc/custom-header.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ function twentythirteen_header_style() {
$text_color = get_header_textcolor();

// If no custom options for text are set, let's bail.
if ( empty( $header_image ) && get_theme_support( 'custom-header', 'default-text-color' ) == $text_color ) {
if ( empty( $header_image ) && get_theme_support( 'custom-header', 'default-text-color' ) === $text_color ) {
return;
}

Expand Down Expand Up @@ -136,7 +136,7 @@ function twentythirteen_header_style() {
endif;

// If the user has set a custom color for the text, use that.
elseif ( get_theme_support( 'custom-header', 'default-text-color' ) != $text_color ) :
elseif ( get_theme_support( 'custom-header', 'default-text-color' ) !== $text_color ) :
?>
.site-title,
.site-description {
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentytwelve/functions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -460,7 +460,7 @@ function twentytwelve_comment( $comment, $args, $depth ) {
}
?>

<?php if ( '0' == $comment->comment_approved ) : ?>
<?php if ( '0' === $comment->comment_approved ) : ?>
<p class="comment-awaiting-moderation"><?php echo $moderation_note; ?></p>
<?php endif; ?>

Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentytwelve/image.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
)
);
foreach ( $attachments as $k => $attachment ) :
if ( $attachment->ID == $post->ID ) {
if ( $attachment->ID === $post->ID ) {
break;
}
endforeach;
Expand Down
2 changes: 1 addition & 1 deletion src/wp-content/themes/twentytwelve/inc/custom-header.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ function twentytwelve_header_style() {
$text_color = get_header_textcolor();

// If no custom options for text are set, let's bail.
if ( get_theme_support( 'custom-header', 'default-text-color' ) == $text_color ) {
if ( get_theme_support( 'custom-header', 'default-text-color' ) === $text_color ) {
return;
}

Expand Down