Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like Jetpack_PostImages often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Jetpack_PostImages, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 13 | class Jetpack_PostImages { |
||
| 14 | /** |
||
| 15 | * If a slideshow is embedded within a post, then parse out the images involved and return them |
||
| 16 | */ |
||
| 17 | static function from_slideshow( $post_id, $width = 200, $height = 200 ) { |
||
| 18 | $images = array(); |
||
| 19 | |||
| 20 | $post = get_post( $post_id ); |
||
| 21 | |||
| 22 | if ( ! $post ) { |
||
| 23 | return $images; |
||
| 24 | } |
||
| 25 | |||
| 26 | if ( ! empty( $post->post_password ) ) { |
||
| 27 | return $images; |
||
| 28 | } |
||
| 29 | |||
| 30 | if ( false === has_shortcode( $post->post_content, 'slideshow' ) ) { |
||
| 31 | return $images; // no slideshow - bail |
||
| 32 | } |
||
| 33 | |||
| 34 | $permalink = get_permalink( $post->ID ); |
||
| 35 | |||
| 36 | // Mechanic: Somebody set us up the bomb |
||
| 37 | $old_post = $GLOBALS['post']; |
||
| 38 | $GLOBALS['post'] = $post; |
||
| 39 | $old_shortcodes = $GLOBALS['shortcode_tags']; |
||
| 40 | $GLOBALS['shortcode_tags'] = array( 'slideshow' => $old_shortcodes['slideshow'] ); |
||
| 41 | |||
| 42 | // Find all the slideshows |
||
| 43 | preg_match_all( '/' . get_shortcode_regex() . '/sx', $post->post_content, $slideshow_matches, PREG_SET_ORDER ); |
||
| 44 | |||
| 45 | ob_start(); // The slideshow shortcode handler calls wp_print_scripts and wp_print_styles... not too happy about that |
||
| 46 | |||
| 47 | foreach ( $slideshow_matches as $slideshow_match ) { |
||
|
|
|||
| 48 | $slideshow = do_shortcode_tag( $slideshow_match ); |
||
| 49 | if ( false === $pos = stripos( $slideshow, 'jetpack-slideshow' ) ) // must be something wrong - or we changed the output format in which case none of the following will work |
||
| 50 | continue; |
||
| 51 | $start = strpos( $slideshow, '[', $pos ); |
||
| 52 | $end = strpos( $slideshow, ']', $start ); |
||
| 53 | $post_images = json_decode( wp_specialchars_decode( str_replace( "'", '"', substr( $slideshow, $start, $end - $start + 1 ) ), ENT_QUOTES ) ); // parse via JSON |
||
| 54 | foreach ( $post_images as $post_image ) { |
||
| 55 | if ( !$post_image_id = absint( $post_image->id ) ) |
||
| 56 | continue; |
||
| 57 | |||
| 58 | $meta = wp_get_attachment_metadata( $post_image_id ); |
||
| 59 | |||
| 60 | // Must be larger than 200x200 (or user-specified) |
||
| 61 | if ( !isset( $meta['width'] ) || $meta['width'] < $width ) |
||
| 62 | continue; |
||
| 63 | if ( !isset( $meta['height'] ) || $meta['height'] < $height ) |
||
| 64 | continue; |
||
| 65 | |||
| 66 | $url = wp_get_attachment_url( $post_image_id ); |
||
| 67 | |||
| 68 | $images[] = array( |
||
| 69 | 'type' => 'image', |
||
| 70 | 'from' => 'slideshow', |
||
| 71 | 'src' => $url, |
||
| 72 | 'src_width' => $meta['width'], |
||
| 73 | 'src_height' => $meta['height'], |
||
| 74 | 'href' => $permalink, |
||
| 75 | ); |
||
| 76 | } |
||
| 77 | } |
||
| 78 | ob_end_clean(); |
||
| 79 | |||
| 80 | // Operator: Main screen turn on |
||
| 81 | $GLOBALS['shortcode_tags'] = $old_shortcodes; |
||
| 82 | $GLOBALS['post'] = $old_post; |
||
| 83 | |||
| 84 | return $images; |
||
| 85 | } |
||
| 86 | |||
| 87 | /** |
||
| 88 | * If a gallery is detected, then get all the images from it. |
||
| 89 | */ |
||
| 90 | static function from_gallery( $post_id ) { |
||
| 91 | $images = array(); |
||
| 92 | |||
| 93 | $post = get_post( $post_id ); |
||
| 94 | |||
| 95 | if ( ! $post ) { |
||
| 96 | return $images; |
||
| 97 | } |
||
| 98 | |||
| 99 | if ( ! empty( $post->post_password ) ) { |
||
| 100 | return $images; |
||
| 101 | } |
||
| 102 | |||
| 103 | $permalink = get_permalink( $post->ID ); |
||
| 104 | |||
| 105 | /** |
||
| 106 | * Juggle global post object because the gallery shortcode uses the |
||
| 107 | * global object. |
||
| 108 | * |
||
| 109 | * See core ticket: |
||
| 110 | * https://core.trac.wordpress.org/ticket/39304 |
||
| 111 | */ |
||
| 112 | $juggle_post = $GLOBALS['post']; |
||
| 113 | $GLOBALS['post'] = $post; |
||
| 114 | $galleries = get_post_galleries( $post->ID, false ); |
||
| 115 | $GLOBALS['post'] = $juggle_post; |
||
| 116 | |||
| 117 | foreach ( $galleries as $gallery ) { |
||
| 118 | if ( isset( $gallery['type'] ) && 'slideshow' === $gallery['type'] && ! empty( $gallery['ids'] ) ) { |
||
| 119 | $image_ids = explode( ',', $gallery['ids'] ); |
||
| 120 | $image_size = isset( $gallery['size'] ) ? $gallery['size'] : 'thumbnail'; |
||
| 121 | foreach ( $image_ids as $image_id ) { |
||
| 122 | $image = wp_get_attachment_image_src( $image_id, $image_size ); |
||
| 123 | View Code Duplication | if ( ! empty( $image[0] ) ) { |
|
| 124 | list( $raw_src ) = explode( '?', $image[0] ); // pull off any Query string (?w=250) |
||
| 125 | $raw_src = wp_specialchars_decode( $raw_src ); // rawify it |
||
| 126 | $raw_src = esc_url_raw( $raw_src ); // clean it |
||
| 127 | $images[] = array( |
||
| 128 | 'type' => 'image', |
||
| 129 | 'from' => 'gallery', |
||
| 130 | 'src' => $raw_src, |
||
| 131 | 'href' => $permalink, |
||
| 132 | ); |
||
| 133 | } |
||
| 134 | } |
||
| 135 | View Code Duplication | } elseif ( ! empty( $gallery['src'] ) ) { |
|
| 136 | foreach ( $gallery['src'] as $src ) { |
||
| 137 | list( $raw_src ) = explode( '?', $src ); // pull off any Query string (?w=250) |
||
| 138 | $raw_src = wp_specialchars_decode( $raw_src ); // rawify it |
||
| 139 | $raw_src = esc_url_raw( $raw_src ); // clean it |
||
| 140 | $images[] = array( |
||
| 141 | 'type' => 'image', |
||
| 142 | 'from' => 'gallery', |
||
| 143 | 'src' => $raw_src, |
||
| 144 | 'href' => $permalink, |
||
| 145 | ); |
||
| 146 | } |
||
| 147 | } |
||
| 148 | } |
||
| 149 | |||
| 150 | return $images; |
||
| 151 | } |
||
| 152 | |||
| 153 | /** |
||
| 154 | * Get attachment images for a specified post and return them. Also make sure |
||
| 155 | * their dimensions are at or above a required minimum. |
||
| 156 | */ |
||
| 157 | static function from_attachment( $post_id, $width = 200, $height = 200 ) { |
||
| 158 | $images = array(); |
||
| 159 | |||
| 160 | $post = get_post( $post_id ); |
||
| 161 | |||
| 162 | if ( ! empty( $post->post_password ) ) { |
||
| 163 | return $images; |
||
| 164 | } |
||
| 165 | |||
| 166 | $post_images = get_posts( array( |
||
| 167 | 'post_parent' => $post_id, // Must be children of post |
||
| 168 | 'numberposts' => 5, // No more than 5 |
||
| 169 | 'post_type' => 'attachment', // Must be attachments |
||
| 170 | 'post_mime_type' => 'image', // Must be images |
||
| 171 | ) ); |
||
| 172 | |||
| 173 | if ( ! $post_images ) { |
||
| 174 | return $images; |
||
| 175 | } |
||
| 176 | |||
| 177 | $permalink = get_permalink( $post_id ); |
||
| 178 | |||
| 179 | foreach ( $post_images as $post_image ) { |
||
| 180 | $meta = wp_get_attachment_metadata( $post_image->ID ); |
||
| 181 | // Must be larger than 200x200 |
||
| 182 | if ( !isset( $meta['width'] ) || $meta['width'] < $width ) |
||
| 183 | continue; |
||
| 184 | if ( !isset( $meta['height'] ) || $meta['height'] < $height ) |
||
| 185 | continue; |
||
| 186 | |||
| 187 | $url = wp_get_attachment_url( $post_image->ID ); |
||
| 188 | |||
| 189 | $images[] = array( |
||
| 190 | 'type' => 'image', |
||
| 191 | 'from' => 'attachment', |
||
| 192 | 'src' => $url, |
||
| 193 | 'src_width' => $meta['width'], |
||
| 194 | 'src_height' => $meta['height'], |
||
| 195 | 'href' => $permalink, |
||
| 196 | ); |
||
| 197 | } |
||
| 198 | |||
| 199 | /* |
||
| 200 | * We only want to pass back attached images that were actually inserted. |
||
| 201 | * We can load up all the images found in the HTML source and then |
||
| 202 | * compare URLs to see if an image is attached AND inserted. |
||
| 203 | */ |
||
| 204 | $html_images = self::from_html( $post_id ); |
||
| 205 | $inserted_images = array(); |
||
| 206 | |||
| 207 | foreach( $html_images as $html_image ) { |
||
| 208 | $src = parse_url( $html_image['src'] ); |
||
| 209 | // strip off any query strings from src |
||
| 210 | if( ! empty( $src['scheme'] ) && ! empty( $src['host'] ) ) { |
||
| 211 | $inserted_images[] = $src['scheme'] . '://' . $src['host'] . $src['path']; |
||
| 212 | } elseif( ! empty( $src['host'] ) ) { |
||
| 213 | $inserted_images[] = set_url_scheme( 'http://' . $src['host'] . $src['path'] ); |
||
| 214 | } else { |
||
| 215 | $inserted_images[] = site_url( '/' ) . $src['path']; |
||
| 216 | } |
||
| 217 | } |
||
| 218 | foreach( $images as $i => $image ) { |
||
| 219 | if ( !in_array( $image['src'], $inserted_images ) ) |
||
| 220 | unset( $images[$i] ); |
||
| 221 | } |
||
| 222 | |||
| 223 | return $images; |
||
| 224 | } |
||
| 225 | |||
| 226 | /** |
||
| 227 | * Check if a Featured Image is set for this post, and return it in a similar |
||
| 228 | * format to the other images?_from_*() methods. |
||
| 229 | * @param int $post_id The post ID to check |
||
| 230 | * @return Array containing details of the Featured Image, or empty array if none. |
||
| 231 | */ |
||
| 232 | static function from_thumbnail( $post_id, $width = 200, $height = 200 ) { |
||
| 233 | $images = array(); |
||
| 234 | |||
| 235 | $post = get_post( $post_id ); |
||
| 236 | |||
| 237 | if ( ! empty( $post->post_password ) ) { |
||
| 238 | return $images; |
||
| 239 | } |
||
| 240 | |||
| 241 | if ( ! function_exists( 'get_post_thumbnail_id' ) ) { |
||
| 242 | return $images; |
||
| 243 | } |
||
| 244 | |||
| 245 | $thumb = get_post_thumbnail_id( $post_id ); |
||
| 246 | |||
| 247 | if ( $thumb ) { |
||
| 248 | $meta = wp_get_attachment_metadata( $thumb ); |
||
| 249 | |||
| 250 | // Must be larger than requested minimums |
||
| 251 | if ( !isset( $meta['width'] ) || $meta['width'] < $width ) |
||
| 252 | return $images; |
||
| 253 | if ( !isset( $meta['height'] ) || $meta['height'] < $height ) |
||
| 254 | return $images; |
||
| 255 | |||
| 256 | $too_big = ( ( ! empty( $meta['width'] ) && $meta['width'] > 1200 ) || ( ! empty( $meta['height'] ) && $meta['height'] > 1200 ) ); |
||
| 257 | |||
| 258 | if ( |
||
| 259 | $too_big && |
||
| 260 | ( |
||
| 261 | ( method_exists( 'Jetpack', 'is_module_active' ) && Jetpack::is_module_active( 'photon' ) ) || |
||
| 262 | ( defined( 'WPCOM' ) && IS_WPCOM ) |
||
| 263 | ) |
||
| 264 | ) { |
||
| 265 | $img_src = wp_get_attachment_image_src( $thumb, array( 1200, 1200 ) ); |
||
| 266 | } else { |
||
| 267 | $img_src = wp_get_attachment_image_src( $thumb, 'full' ); |
||
| 268 | } |
||
| 269 | |||
| 270 | $url = $img_src[0]; |
||
| 271 | |||
| 272 | $images = array( array( // Other methods below all return an array of arrays |
||
| 273 | 'type' => 'image', |
||
| 274 | 'from' => 'thumbnail', |
||
| 275 | 'src' => $url, |
||
| 276 | 'src_width' => $img_src[1], |
||
| 277 | 'src_height' => $img_src[2], |
||
| 278 | 'href' => get_permalink( $thumb ), |
||
| 279 | ) ); |
||
| 280 | } |
||
| 281 | |||
| 282 | if ( empty( $images ) && ( defined( 'IS_WPCOM' ) && IS_WPCOM ) ) { |
||
| 283 | $meta_thumbnail = get_post_meta( $post_id, '_jetpack_post_thumbnail', true ); |
||
| 284 | if ( ! empty( $meta_thumbnail ) ) { |
||
| 285 | if ( ! isset( $meta_thumbnail['width'] ) || $meta_thumbnail['width'] < $width ) { |
||
| 286 | return $images; |
||
| 287 | } |
||
| 288 | |||
| 289 | if ( ! isset( $meta_thumbnail['height'] ) || $meta_thumbnail['height'] < $height ) { |
||
| 290 | return $images; |
||
| 291 | } |
||
| 292 | |||
| 293 | $images = array( array( // Other methods below all return an array of arrays |
||
| 294 | 'type' => 'image', |
||
| 295 | 'from' => 'thumbnail', |
||
| 296 | 'src' => $meta_thumbnail['URL'], |
||
| 297 | 'src_width' => $meta_thumbnail['width'], |
||
| 298 | 'src_height' => $meta_thumbnail['height'], |
||
| 299 | 'href' => $meta_thumbnail['URL'], |
||
| 300 | ) ); |
||
| 301 | } |
||
| 302 | } |
||
| 303 | |||
| 304 | return $images; |
||
| 305 | } |
||
| 306 | |||
| 307 | /** |
||
| 308 | * Very raw -- just parse the HTML and pull out any/all img tags and return their src |
||
| 309 | * @param mixed $html_or_id The HTML string to parse for images, or a post id |
||
| 310 | * @return Array containing images |
||
| 311 | */ |
||
| 312 | static function from_html( $html_or_id ) { |
||
| 313 | $images = array(); |
||
| 314 | |||
| 315 | if ( is_numeric( $html_or_id ) ) { |
||
| 316 | $post = get_post( $html_or_id ); |
||
| 317 | |||
| 318 | if ( empty( $post ) || ! empty( $post->post_password ) ) { |
||
| 319 | return $images; |
||
| 320 | } |
||
| 321 | |||
| 322 | $html = $post->post_content; // DO NOT apply the_content filters here, it will cause loops |
||
| 323 | } else { |
||
| 324 | $html = $html_or_id; |
||
| 325 | } |
||
| 326 | |||
| 327 | if ( ! $html ) { |
||
| 328 | return $images; |
||
| 329 | } |
||
| 330 | |||
| 331 | preg_match_all( '!<img.*src=[\'"]([^"]+)[\'"].*/?>!iUs', $html, $matches ); |
||
| 332 | if ( !empty( $matches[1] ) ) { |
||
| 333 | foreach ( $matches[1] as $match ) { |
||
| 334 | if ( stristr( $match, '/smilies/' ) ) |
||
| 335 | continue; |
||
| 336 | |||
| 337 | $images[] = array( |
||
| 338 | 'type' => 'image', |
||
| 339 | 'from' => 'html', |
||
| 340 | 'src' => html_entity_decode( $match ), |
||
| 341 | 'href' => '', // No link to apply to these. Might potentially parse for that as well, but not for now |
||
| 342 | ); |
||
| 343 | } |
||
| 344 | } |
||
| 345 | |||
| 346 | return $images; |
||
| 347 | } |
||
| 348 | |||
| 349 | /** |
||
| 350 | * @param int $post_id The post ID to check |
||
| 351 | * @param int $size |
||
| 352 | * @return Array containing details of the image, or empty array if none. |
||
| 353 | */ |
||
| 354 | static function from_blavatar( $post_id, $size = 96 ) { |
||
| 355 | |||
| 356 | $permalink = get_permalink( $post_id ); |
||
| 357 | |||
| 358 | if ( function_exists( 'blavatar_domain' ) && function_exists( 'blavatar_exists' ) && function_exists( 'blavatar_url' ) ) { |
||
| 359 | $domain = blavatar_domain( $permalink ); |
||
| 360 | |||
| 361 | if ( ! blavatar_exists( $domain ) ) { |
||
| 362 | return array(); |
||
| 363 | } |
||
| 364 | |||
| 365 | $url = blavatar_url( $domain, 'img', $size ); |
||
| 366 | } elseif ( function_exists( 'has_site_icon' ) && has_site_icon() ) { |
||
| 367 | $url = get_site_icon_url( $size ); |
||
| 368 | } else { |
||
| 369 | return array(); |
||
| 370 | } |
||
| 371 | |||
| 372 | return array( array( |
||
| 373 | 'type' => 'image', |
||
| 374 | 'from' => 'blavatar', |
||
| 375 | 'src' => $url, |
||
| 376 | 'src_width' => $size, |
||
| 377 | 'src_height' => $size, |
||
| 378 | 'href' => $permalink, |
||
| 379 | ) ); |
||
| 380 | } |
||
| 381 | |||
| 382 | /** |
||
| 383 | * Gets a post image from the author avatar. |
||
| 384 | * |
||
| 385 | * @param int $post_id The post ID to check. |
||
| 386 | * @param int $size The size of the avatar to get. |
||
| 387 | * @param string $default The default image to use. |
||
| 388 | * @return Array containing details of the image, or empty array if none. |
||
| 389 | */ |
||
| 390 | static function from_gravatar( $post_id, $size = 96, $default = false ) { |
||
| 391 | $post = get_post( $post_id ); |
||
| 392 | $permalink = get_permalink( $post_id ); |
||
| 393 | |||
| 394 | if ( function_exists( 'wpcom_get_avatar_url' ) ) { |
||
| 395 | $url = wpcom_get_avatar_url( $post->post_author, $size, $default, true ); |
||
| 396 | if ( $url && is_array( $url ) ) { |
||
| 397 | $url = $url[0]; |
||
| 398 | } |
||
| 399 | } else { |
||
| 400 | $url = get_avatar_url( $post->post_author, array( |
||
| 401 | 'size' => $size, |
||
| 402 | 'default' => $default, |
||
| 403 | ) ); |
||
| 404 | } |
||
| 405 | |||
| 406 | return array( |
||
| 407 | array( |
||
| 408 | 'type' => 'image', |
||
| 409 | 'from' => 'gravatar', |
||
| 410 | 'src' => $url, |
||
| 411 | 'src_width' => $size, |
||
| 412 | 'src_height' => $size, |
||
| 413 | 'href' => $permalink, |
||
| 414 | ), |
||
| 415 | ); |
||
| 416 | } |
||
| 417 | |||
| 418 | /** |
||
| 419 | * Run through the different methods that we have available to try to find a single good |
||
| 420 | * display image for this post. |
||
| 421 | * @param int $post_id |
||
| 422 | * @param array $args Other arguments (currently width and height required for images where possible to determine) |
||
| 423 | * @return Array containing details of the best image to be used |
||
| 424 | */ |
||
| 425 | static function get_image( $post_id, $args = array() ) { |
||
| 459 | |||
| 460 | /** |
||
| 461 | * Get an array containing a collection of possible images for this post, stopping once we hit a method |
||
| 462 | * that returns something useful. |
||
| 463 | * @param int $post_id |
||
| 464 | * @param array $args Optional args, see defaults list for details |
||
| 465 | * @return Array containing images that would be good for representing this post |
||
| 466 | */ |
||
| 467 | static function get_images( $post_id, $args = array() ) { |
||
| 537 | |||
| 538 | /** |
||
| 539 | * Takes an image URL and pixel dimensions then returns a URL for the |
||
| 540 | * resized and croped image. |
||
| 541 | * |
||
| 542 | * @param string $src |
||
| 543 | * @param int $dimension |
||
| 544 | * @return string Transformed image URL |
||
| 545 | */ |
||
| 546 | static function fit_image_url( $src, $width, $height ) { |
||
| 583 | } |
||
| 584 |
There are different options of fixing this problem.
If you want to be on the safe side, you can add an additional type-check:
If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:
Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.