Twenty Seventeen: Unused and duplicate variables in content-front-page.php

[dayun123]

<?php if ( has_post_thumbnail() ) :
        $thumbnail = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), 'twentyseventeen-featured-image' );

        $post_thumbnail_id = get_post_thumbnail_id( $post->ID );

        $thumbnail_attributes = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), 'twentyseventeen-featured-image' );

        // Calculate aspect ratio: h / w * 100%.
        $ratio = $thumbnail_attributes[2] / $thumbnail_attributes[1] * 100;
        ?>

        <div class="panel-image" style="background-image: url(<?php echo esc_url( $thumbnail[0] ); ?>);">
            <div class="panel-image-prop" style="padding-top: <?php echo esc_attr( $ratio ); ?>%"></div>
        </div><!-- .panel-image -->

<?php endif; ?>

I'm still a novice programmer, so maybe I'm missing something obvious, but I don't understand why the following two things happen:

1. Why is the $post_thumbnail_id variable created but never used? Why not create it right away instead of constantly calling get_post_thumbnail_id( $post->ID )?
2. $thumbnail and $thumbnail_attributes are the same, right? So why create both?

Wouldn't this make more sense:

<?php if ( has_post_thumbnail() ) :

        $post_thumbnail_id = get_post_thumbnail_id( $post->ID );

        $thumbnail = wp_get_attachment_image_src( $post_thumbnail_id, 'twentyseventeen-featured-image' );

        // Calculate aspect ratio: h / w * 100%.
        $ratio = $thumbnail[2] / $thumbnail[1] * 100;
        ?>

        <div class="panel-image" style="background-image: url(<?php echo esc_url( $thumbnail[0] ); ?>);">
            <div class="panel-image-prop" style="padding-top: <?php echo esc_attr( $ratio ); ?>%"></div>
        </div><!-- .panel-image -->

<?php endif; ?>

[dingo_bastard]

The odd thing is that the content-front-page-panels.php uses

<?php if ( has_post_thumbnail() ) :
        $thumbnail = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), 'twentyseventeen-featured-image' );

        // Calculate aspect ratio: h / w * 100%.
        $ratio = $thumbnail[2] / $thumbnail[1] * 100;
        ?>

        <div class="panel-image" style="background-image: url(<?php echo esc_url( $thumbnail[0] ); ?>);">
                <div class="panel-image-prop" style="padding-top: <?php echo esc_attr( $ratio ); ?>%"></div>
        </div><!-- .panel-image -->

<?php endif; ?>

Maybe somebody missed that.

[dingo_bastard]

Replaced post thumbnail code in content-front-page.php with the one from content-front-page-panels.php, and escaped $twentyseventeencounter in content-front-page-panels.php

[jbpaul17]

Punting to 4.8.1 per today's bug scrub.

[mariusvetrici]

Remove changes from content-front-page-panels.php and only keep changes for content-front-page.php

[mariusvetrici]

The old patch contained changes for 2 files. Only 1 of the files (content-front-page-panels.php) is related to the current track issue. Therefore, the patch only contains the changes for this ticket.

Tested it with @grapplerulrich and @ocean90

[ocean90]

In 40904:

Twenty Seventeen: Remove/merge unused variables in content-front-page.php.

#wceu2017

Props dingo_bastard, mariusvetrici.
Fixes #39771.

[pento]

Moving this to 4.9, it doesn't need to to be backported to the 4.8 branch.