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 |