| Conditions | 1 |
| Paths | 1 |
| Total Lines | 76 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 141 | public function render_banner() { |
||
| 142 | $jetpack_logo = new Jetpack_Logo(); |
||
| 143 | $powering_up_logo = plugins_url( 'images/jetpack-powering-up.svg', JETPACK__PLUGIN_FILE ); |
||
| 144 | |||
| 145 | ?> |
||
| 146 | <div id="jp-wizard-banner" class="jp-wizard-banner"> |
||
| 147 | <div class="jp-wizard-banner-grid"> |
||
| 148 | <div class="jp-wizard-banner-grid-a"> |
||
| 149 | <div class="jp-emblem"> |
||
| 150 | <?php |
||
| 151 | echo $jetpack_logo->get_jp_emblem_larger(); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped |
||
| 152 | ?> |
||
| 153 | </div> |
||
| 154 | <h2 class="jp-wizard-banner-wizard-header"> |
||
| 155 | <?php esc_html_e( 'Set up Jetpack for better site security, performance, and more', 'jetpack' ); ?> |
||
| 156 | </h2> |
||
| 157 | <p class="jp-wizard-banner-wizard-paragraph"> |
||
| 158 | <?php esc_html_e( 'Jetpack is a cloud-powered tool built by Automattic.', 'jetpack' ); ?> |
||
| 159 | </p> |
||
| 160 | <p class="jp-wizard-banner-wizard-paragraph"> |
||
| 161 | <?php esc_html_e( 'Answer a few questions and we’ll help you secure, speed up, customize, and grow your WordPress website.', 'jetpack' ); ?> |
||
| 162 | </p> |
||
| 163 | <div class="jp-wizard-banner-wizard-intro-question"> |
||
| 164 | <h2> |
||
| 165 | <?php |
||
| 166 | printf( |
||
| 167 | /* translators: %s is the site name */ |
||
| 168 | esc_html__( 'What will %s be used for?', 'jetpack' ), |
||
| 169 | esc_html( get_bloginfo( 'name' ) ) |
||
| 170 | ); |
||
| 171 | ?> |
||
| 172 | </h2> |
||
| 173 | <div class="jp-wizard-banner-wizard-answer-buttons"> |
||
| 174 | <a |
||
| 175 | class="button button-primary jp-wizard-banner-wizard-button" |
||
| 176 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/income?use=personal' ) ); ?>" |
||
| 177 | > |
||
| 178 | <?php esc_html_e( 'Personal Use', 'jetpack' ); ?> |
||
| 179 | </a> |
||
| 180 | <a |
||
| 181 | class="button button-primary jp-wizard-banner-wizard-button" |
||
| 182 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/income?use=business' ) ); ?>" |
||
| 183 | > |
||
| 184 | <?php esc_html_e( 'Business Use', 'jetpack' ); ?> |
||
| 185 | </a> |
||
| 186 | </div> |
||
| 187 | <a |
||
| 188 | class="jp-wizard-banner-wizard-skip-link" |
||
| 189 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/features' ) ); ?>" |
||
| 190 | > |
||
| 191 | <?php esc_html_e( 'Skip to recommended features', 'jetpack' ); ?> |
||
| 192 | </a> |
||
| 193 | </div> |
||
| 194 | </div> |
||
| 195 | |||
| 196 | |||
| 197 | <div class="jp-wizard-banner-grid-b"> |
||
| 198 | <img |
||
| 199 | class="powering-up-img" |
||
| 200 | width="200px" |
||
| 201 | height="200px" |
||
| 202 | src="<?php echo esc_url( $powering_up_logo ); ?>" |
||
| 203 | alt="<?php esc_attr_e( 'A jetpack site powering up', 'jetpack' ); ?>" |
||
| 204 | /> |
||
| 205 | </div> |
||
| 206 | |||
| 207 | <span |
||
| 208 | class="notice-dismiss wizard-banner-dismiss" |
||
| 209 | title="<?php esc_attr_e( 'Dismiss this notice', 'jetpack' ); ?>"> |
||
| 210 | </span> |
||
| 211 | |||
| 212 | </div> |
||
| 213 | |||
| 214 | </div> |
||
| 215 | <?php |
||
| 216 | } |
||
| 217 | } |
||
| 218 |