| Conditions | 26 | 
| Paths | 10457 | 
| Total Lines | 169 | 
| 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  | 
            ||
| 238 | 	function get_messages( $message_path, $query ) { | 
            ||
| 239 | // custom filters go here  | 
            ||
| 240 | add_filter( 'jitm_woocommerce_services_msg', array( 'Jetpack_JITM', 'jitm_woocommerce_services_msg' ) );  | 
            ||
| 241 | add_filter( 'jitm_jetpack_woo_services_install', array( 'Jetpack_JITM', 'jitm_jetpack_woo_services_install' ) );  | 
            ||
| 242 | add_filter( 'jitm_jetpack_woo_services_activate', array(  | 
            ||
| 243 | 'Jetpack_JITM',  | 
            ||
| 244 | 'jitm_jetpack_woo_services_activate'  | 
            ||
| 245 | ) );  | 
            ||
| 246 | |||
| 247 | $user = wp_get_current_user();  | 
            ||
| 248 | |||
| 249 | // unauthenticated or invalid requests just bail  | 
            ||
| 250 | 		if ( ! $user ) { | 
            ||
| 251 | return array();  | 
            ||
| 252 | }  | 
            ||
| 253 | |||
| 254 | require_once( JETPACK__PLUGIN_DIR . 'class.jetpack-client.php' );  | 
            ||
| 255 | |||
| 256 | $site_id = Jetpack_Options::get_option( 'id' );  | 
            ||
| 257 | |||
| 258 | // build our jitm request  | 
            ||
| 259 | $path = add_query_arg( array(  | 
            ||
| 260 | 'external_user_id' => urlencode_deep( $user->ID ),  | 
            ||
| 261 | 'query_string' => urlencode_deep( $query ),  | 
            ||
| 262 | 'mobile_browser' => jetpack_is_mobile( 'smart' ) ? 1 : 0,  | 
            ||
| 263 | '_locale' => get_user_locale(),  | 
            ||
| 264 | ), sprintf( '/sites/%d/jitm/%s', $site_id, $message_path ) );  | 
            ||
| 265 | |||
| 266 | // attempt to get from cache  | 
            ||
| 267 | $envelopes = get_transient( 'jetpack_jitm_' . substr( md5( $path ), 0, 31 ) );  | 
            ||
| 268 | |||
| 269 | // if something is in the cache and it was put in the cache after the last sync we care about, use it  | 
            ||
| 270 | $use_cache = false;  | 
            ||
| 271 | |||
| 272 | /** This filter is documented in class.jetpack.php */  | 
            ||
| 273 | 		if ( apply_filters( 'jetpack_just_in_time_msg_cache', false ) ) { | 
            ||
| 274 | $use_cache = true;  | 
            ||
| 275 | }  | 
            ||
| 276 | |||
| 277 | 		if ( $use_cache ) { | 
            ||
| 278 | $last_sync = (int) get_transient( 'jetpack_last_plugin_sync' );  | 
            ||
| 279 | $from_cache = $envelopes && $last_sync > 0 && $last_sync < $envelopes['last_response_time'];  | 
            ||
| 280 | 		} else { | 
            ||
| 281 | $from_cache = false;  | 
            ||
| 282 | }  | 
            ||
| 283 | |||
| 284 | // otherwise, ask again  | 
            ||
| 285 | 		if ( ! $from_cache ) { | 
            ||
| 286 | $wpcom_response = Jetpack_Client::wpcom_json_api_request_as_blog(  | 
            ||
| 287 | $path,  | 
            ||
| 288 | '2',  | 
            ||
| 289 | array(  | 
            ||
| 290 | 'user_id' => $user->ID,  | 
            ||
| 291 | 'user_roles' => implode( ',', $user->roles ),  | 
            ||
| 292 | ),  | 
            ||
| 293 | null,  | 
            ||
| 294 | 'wpcom'  | 
            ||
| 295 | );  | 
            ||
| 296 | |||
| 297 | // silently fail...might be helpful to track it?  | 
            ||
| 298 | 			if ( is_wp_error( $wpcom_response ) ) { | 
            ||
| 299 | return array();  | 
            ||
| 300 | }  | 
            ||
| 301 | |||
| 302 | $envelopes = json_decode( $wpcom_response['body'] );  | 
            ||
| 303 | |||
| 304 | 			if ( ! is_array( $envelopes ) ) { | 
            ||
| 305 | return array();  | 
            ||
| 306 | }  | 
            ||
| 307 | |||
| 308 | $expiration = isset( $envelopes[0] ) ? $envelopes[0]->ttl : 300;  | 
            ||
| 309 | |||
| 310 | // do not cache if expiration is 0 or we're not using the cache  | 
            ||
| 311 | 			if ( 0 != $expiration && $use_cache ) { | 
            ||
| 312 | $envelopes['last_response_time'] = time();  | 
            ||
| 313 | |||
| 314 | set_transient( 'jetpack_jitm_' . substr( md5( $path ), 0, 31 ), $envelopes, $expiration );  | 
            ||
| 315 | }  | 
            ||
| 316 | }  | 
            ||
| 317 | |||
| 318 | $hidden_jitms = Jetpack_Options::get_option( 'hide_jitm' );  | 
            ||
| 319 | unset( $envelopes['last_response_time'] );  | 
            ||
| 320 | |||
| 321 | /**  | 
            ||
| 322 | * Allow adding your own custom JITMs after a set of JITMs has been received.  | 
            ||
| 323 | *  | 
            ||
| 324 | * @since 6.9.0  | 
            ||
| 325 | *  | 
            ||
| 326 | * @param array $envelopes array of existing JITMs.  | 
            ||
| 327 | */  | 
            ||
| 328 | $envelopes = apply_filters( 'jetpack_jitm_received_envelopes', $envelopes );  | 
            ||
| 329 | |||
| 330 | 		foreach ( $envelopes as $idx => &$envelope ) { | 
            ||
| 331 | |||
| 332 | $dismissed_feature = isset( $hidden_jitms[ $envelope->feature_class ] ) && is_array( $hidden_jitms[ $envelope->feature_class ] ) ? $hidden_jitms[ $envelope->feature_class ] : null;  | 
            ||
| 333 | |||
| 334 | // if the this feature class has been dismissed and the request has not passed the ttl, skip it as it's been dismissed  | 
            ||
| 335 | 			if ( is_array( $dismissed_feature ) && ( time() - $dismissed_feature['last_dismissal'] < $envelope->expires || $dismissed_feature['number'] >= $envelope->max_dismissal ) ) { | 
            ||
| 336 | unset( $envelopes[ $idx ] );  | 
            ||
| 337 | continue;  | 
            ||
| 338 | }  | 
            ||
| 339 | |||
| 340 | JetpackTracking::record_user_event( 'jitm_view_client', array(  | 
            ||
| 341 | 'jitm_id' => $envelope->id,  | 
            ||
| 342 | ) );  | 
            ||
| 343 | |||
| 344 | $normalized_site_url = Jetpack::build_raw_urls( get_home_url() );  | 
            ||
| 345 | |||
| 346 | $url_params = array(  | 
            ||
| 347 | 'source' => "jitm-$envelope->id",  | 
            ||
| 348 | 'site' => $normalized_site_url,  | 
            ||
| 349 | 'u' => $user->ID,  | 
            ||
| 350 | );  | 
            ||
| 351 | |||
| 352 | 			if ( ! class_exists( 'Jetpack_Affiliate' ) ) { | 
            ||
| 353 | require_once JETPACK__PLUGIN_DIR . 'class.jetpack-affiliate.php';  | 
            ||
| 354 | }  | 
            ||
| 355 | // Get affiliate code and add it to the array of URL parameters  | 
            ||
| 356 | 			if ( '' !== ( $aff = Jetpack_Affiliate::init()->get_affiliate_code() ) ) { | 
            ||
| 357 | $url_params['aff'] = $aff;  | 
            ||
| 358 | }  | 
            ||
| 359 | |||
| 360 | $envelope->url = add_query_arg( $url_params, 'https://jetpack.com/redirect/' );  | 
            ||
| 361 | |||
| 362 | $envelope->jitm_stats_url = Jetpack::build_stats_url( array( 'x_jetpack-jitm' => $envelope->id ) );  | 
            ||
| 363 | |||
| 364 | 			if ( $envelope->CTA->hook ) { | 
            ||
| 365 | $envelope->url = apply_filters( 'jitm_' . $envelope->CTA->hook, $envelope->url );  | 
            ||
| 366 | unset( $envelope->CTA->hook );  | 
            ||
| 367 | }  | 
            ||
| 368 | |||
| 369 | 			if ( isset( $envelope->content->hook ) ) { | 
            ||
| 370 | $envelope->content = apply_filters( 'jitm_' . $envelope->content->hook, $envelope->content );  | 
            ||
| 371 | unset( $envelope->content->hook );  | 
            ||
| 372 | }  | 
            ||
| 373 | |||
| 374 | // no point in showing an empty message  | 
            ||
| 375 | 			if ( empty( $envelope->content->message ) ) { | 
            ||
| 376 | unset( $envelopes[ $idx ] );  | 
            ||
| 377 | continue;  | 
            ||
| 378 | }  | 
            ||
| 379 | |||
| 380 | 			switch ( $envelope->content->icon ) { | 
            ||
| 381 | case 'jetpack':  | 
            ||
| 382 | $envelope->content->icon = '<div class="jp-emblem">' . Jetpack::get_jp_emblem() . '</div>';  | 
            ||
| 383 | break;  | 
            ||
| 384 | case 'woocommerce':  | 
            ||
| 385 | $envelope->content->icon = '<div class="jp-emblem"><svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1" id="Layer_1" x="0px" y="0px" viewBox="0 0 168 100" xml:space="preserve" enable-background="new 0 0 168 100" width="50" height="30"><style type="text/css">  | 
            ||
| 386 | 					.st0{clip-path:url(#SVGID_2_);enable-background:new    ;} | 
            ||
| 387 | 					.st1{clip-path:url(#SVGID_4_);} | 
            ||
| 388 | 					.st2{clip-path:url(#SVGID_6_);} | 
            ||
| 389 | 					.st3{clip-path:url(#SVGID_8_);fill:#8F567F;} | 
            ||
| 390 | 					.st4{clip-path:url(#SVGID_10_);fill:#FFFFFE;} | 
            ||
| 391 | 					.st5{clip-path:url(#SVGID_12_);fill:#FFFFFE;} | 
            ||
| 392 | 					.st6{clip-path:url(#SVGID_14_);fill:#FFFFFE;} | 
            ||
| 393 | </style><g><defs><polygon id="SVGID_1_" points="83.8 100 0 100 0 0.3 83.8 0.3 167.6 0.3 167.6 100 "/></defs><clipPath id="SVGID_2_"><use xlink:href="#SVGID_1_" overflow="visible"/></clipPath><g class="st0"><g><defs><rect id="SVGID_3_" width="168" height="100"/></defs><clipPath id="SVGID_4_"><use xlink:href="#SVGID_3_" overflow="visible"/></clipPath><g class="st1"><defs><path id="SVGID_5_" d="M15.6 0.3H152c8.6 0 15.6 7 15.6 15.6v52c0 8.6-7 15.6-15.6 15.6h-48.9l6.7 16.4L80.2 83.6H15.6C7 83.6 0 76.6 0 67.9v-52C0 7.3 7 0.3 15.6 0.3"/></defs><clipPath id="SVGID_6_"><use xlink:href="#SVGID_5_" overflow="visible"/></clipPath><g class="st2"><defs><rect id="SVGID_7_" width="168" height="100"/></defs><clipPath id="SVGID_8_"><use xlink:href="#SVGID_7_" overflow="visible"/></clipPath><rect x="-10" y="-9.7" class="st3" width="187.6" height="119.7"/></g></g></g></g></g><g><defs><path id="SVGID_9_" d="M8.4 14.5c1-1.3 2.4-2 4.3-2.1 3.5-0.2 5.5 1.4 6 4.9 2.1 14.3 4.4 26.4 6.9 36.4l15-28.6c1.4-2.6 3.1-3.9 5.2-4.1 3-0.2 4.9 1.7 5.6 5.7 1.7 9.1 3.9 16.9 6.5 23.4 1.8-17.4 4.8-30 9-37.7 1-1.9 2.5-2.9 4.5-3 1.6-0.1 3 0.3 4.3 1.4 1.3 1 2 2.3 2.1 3.9 0.1 1.2-0.1 2.3-0.7 3.3 -2.7 5-4.9 13.2-6.6 24.7 -1.7 11.1-2.3 19.8-1.9 26.1 0.1 1.7-0.1 3.2-0.8 4.5 -0.8 1.5-2 2.4-3.7 2.5 -1.8 0.1-3.6-0.7-5.4-2.5C52.4 66.7 47.4 57 43.7 44.1c-4.4 8.8-7.7 15.3-9.9 19.7 -4 7.7-7.5 11.7-10.3 11.9 -1.9 0.1-3.5-1.4-4.8-4.7 -3.5-9-7.3-26.3-11.3-52C7.1 17.3 7.5 15.8 8.4 14.5"/></defs><clipPath id="SVGID_10_"><use xlink:href="#SVGID_9_" overflow="visible"/></clipPath><rect x="-2.7" y="-0.6" class="st4" width="90.6" height="86.4"/></g><g><defs><path id="SVGID_11_" d="M155.6 25.2c-2.5-4.3-6.1-6.9-11-7.9 -1.3-0.3-2.5-0.4-3.7-0.4 -6.6 0-11.9 3.4-16.1 10.2 -3.6 5.8-5.3 12.3-5.3 19.3 0 5.3 1.1 9.8 3.3 13.6 2.5 4.3 6.1 6.9 11 7.9 1.3 0.3 2.5 0.4 3.7 0.4 6.6 0 12-3.4 16.1-10.2 3.6-5.9 5.3-12.4 5.3-19.4C159 33.4 157.9 28.9 155.6 25.2zM147 44.2c-0.9 4.5-2.7 7.9-5.2 10.1 -2 1.8-3.9 2.5-5.5 2.2 -1.7-0.3-3-1.8-4-4.4 -0.8-2.1-1.2-4.2-1.2-6.2 0-1.7 0.2-3.4 0.5-5 0.6-2.8 1.8-5.5 3.6-8.1 2.3-3.3 4.7-4.8 7.1-4.2 1.7 0.3 3 1.8 4 4.4 0.8 2.1 1.2 4.2 1.2 6.2C147.5 40.9 147.3 42.6 147 44.2z"/></defs><clipPath id="SVGID_12_"><use xlink:href="#SVGID_11_" overflow="visible"/></clipPath><rect x="109.6" y="6.9" class="st5" width="59.4" height="71.4"/></g><g><defs><path id="SVGID_13_" d="M112.7 25.2c-2.5-4.3-6.1-6.9-11-7.9 -1.3-0.3-2.5-0.4-3.7-0.4 -6.6 0-11.9 3.4-16.1 10.2 -3.5 5.8-5.3 12.3-5.3 19.3 0 5.3 1.1 9.8 3.3 13.6 2.5 4.3 6.1 6.9 11 7.9 1.3 0.3 2.5 0.4 3.7 0.4 6.6 0 12-3.4 16.1-10.2 3.5-5.9 5.3-12.4 5.3-19.4C116 33.4 114.9 28.9 112.7 25.2zM104.1 44.2c-0.9 4.5-2.7 7.9-5.2 10.1 -2 1.8-3.9 2.5-5.5 2.2 -1.7-0.3-3-1.8-4-4.4 -0.8-2.1-1.2-4.2-1.2-6.2 0-1.7 0.2-3.4 0.5-5 0.6-2.8 1.8-5.5 3.6-8.1 2.3-3.3 4.7-4.8 7.1-4.2 1.7 0.3 3 1.8 4 4.4 0.8 2.1 1.2 4.2 1.2 6.2C104.6 40.9 104.4 42.6 104.1 44.2z"/></defs><clipPath id="SVGID_14_"><use xlink:href="#SVGID_13_" overflow="visible"/></clipPath><rect x="66.7" y="6.9" class="st6" width="59.4" height="71.4"/></g></svg></div>';  | 
            ||
| 394 | break;  | 
            ||
| 395 | default:  | 
            ||
| 396 | $envelope->content->icon = '';  | 
            ||
| 397 | break;  | 
            ||
| 398 | }  | 
            ||
| 399 | |||
| 400 | $jetpack = Jetpack::init();  | 
            ||
| 401 | $jetpack->stat( 'jitm', $envelope->id . '-viewed-' . JETPACK__VERSION );  | 
            ||
| 402 | $jetpack->do_stats( 'server_side' );  | 
            ||
| 403 | }  | 
            ||
| 404 | |||
| 405 | return $envelopes;  | 
            ||
| 406 | }  | 
            ||
| 407 | }  | 
            ||
| 410 |