Conditions | 1 |
Paths | 1 |
Total Lines | 78 |
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 |
||
165 | public function render_banner() { |
||
166 | $jetpack_logo = new Jetpack_Logo(); |
||
167 | $powering_up_logo = plugins_url( 'images/jetpack-powering-up.svg', JETPACK__PLUGIN_FILE ); |
||
168 | |||
169 | ?> |
||
170 | <div id="jp-wizard-banner" class="jp-wizard-banner"> |
||
171 | <div class="jp-wizard-banner-grid"> |
||
172 | <div class="jp-wizard-banner-grid-a"> |
||
173 | <div class="jp-emblem"> |
||
174 | <?php |
||
175 | echo $jetpack_logo->get_jp_emblem_larger(); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped |
||
176 | ?> |
||
177 | </div> |
||
178 | <h2 class="jp-wizard-banner-wizard-header"> |
||
179 | <?php esc_html_e( 'Set up Jetpack for better site security, performance, and more.', 'jetpack' ); ?> |
||
180 | </h2> |
||
181 | <p class="jp-wizard-banner-wizard-paragraph"> |
||
182 | <?php esc_html_e( 'Jetpack is a cloud-powered tool built by Automattic.', 'jetpack' ); ?> |
||
183 | </p> |
||
184 | <p class="jp-wizard-banner-wizard-paragraph"> |
||
185 | <?php esc_html_e( 'Answer a few questions and we’ll help you secure, speed up, customize, and grow your WordPress website.', 'jetpack' ); ?> |
||
186 | </p> |
||
187 | <div class="jp-wizard-banner-wizard-intro-question"> |
||
188 | <h2> |
||
189 | <?php |
||
190 | printf( |
||
191 | /* translators: %s is the site name */ |
||
192 | esc_html__( 'What will %s be used for?', 'jetpack' ), |
||
193 | esc_html( get_bloginfo( 'name' ) ) |
||
194 | ); |
||
195 | ?> |
||
196 | </h2> |
||
197 | <div class="jp-wizard-banner-wizard-answer-buttons"> |
||
198 | <a |
||
199 | id="jp-wizard-banner-personal-button" |
||
200 | class="button button-primary jp-wizard-banner-wizard-button" |
||
201 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/income?use=personal' ) ); ?>" |
||
202 | > |
||
203 | <?php esc_html_e( 'Personal Use', 'jetpack' ); ?> |
||
204 | </a> |
||
205 | <a |
||
206 | id="jp-wizard-banner-business-button" |
||
207 | class="button button-primary jp-wizard-banner-wizard-button" |
||
208 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/income?use=business' ) ); ?>" |
||
209 | > |
||
210 | <?php esc_html_e( 'Business Use', 'jetpack' ); ?> |
||
211 | </a> |
||
212 | </div> |
||
213 | <a |
||
214 | class="jp-wizard-banner-wizard-skip-link" |
||
215 | href="<?php echo esc_url( Jetpack::admin_url( 'page=jetpack#/setup/features' ) ); ?>" |
||
216 | > |
||
217 | <?php esc_html_e( 'Skip to recommended features', 'jetpack' ); ?> |
||
218 | </a> |
||
219 | </div> |
||
220 | </div> |
||
221 | |||
222 | |||
223 | <div class="jp-wizard-banner-grid-b"> |
||
224 | <img |
||
225 | class="powering-up-img" |
||
226 | width="200px" |
||
227 | height="200px" |
||
228 | src="<?php echo esc_url( $powering_up_logo ); ?>" |
||
229 | alt="<?php esc_attr_e( 'A jetpack site powering up', 'jetpack' ); ?>" |
||
230 | /> |
||
231 | </div> |
||
232 | |||
233 | <span |
||
234 | class="notice-dismiss" |
||
235 | title="<?php esc_attr_e( 'Dismiss this notice', 'jetpack' ); ?>"> |
||
236 | </span> |
||
237 | |||
238 | </div> |
||
239 | |||
240 | </div> |
||
241 | <?php |
||
242 | } |
||
243 | } |
||
244 |