Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
06/02/2024 02:07:21 AM (4 months ago)
Author:
espellcaste
Message:

WPCS: Part VII: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], [13887], [13888], [13891], and [13892]

See #9164 and #7228

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/classes/class-bp-media-extractor.php

    r13793 r13893  
    66 * @subpackage Core
    77 * @since 2.3.0
     8 *
     9 * @phpcs:disable Squiz.PHP.CommentedOutCode.Found
    810 */
    911
    … …  
    4042 */
    4143class BP_Media_Extractor {
     44
    4245    /**
    4346     * Media type.
    … …  
    5457    const AUDIO      = 32;
    5558    const VIDEOS     = 64;
    56 
    5759
    5860    /**
    … …  
    132134        $plaintext = $this->strip_markup( $richtext );
    133135
    134 
    135136        // Extract links.
    136137        if ( self::LINKS & $what_to_extract ) {
    … …  
    181182        return apply_filters( 'bp_media_extractor_extract', $media, $richtext, $what_to_extract, $extra_args, $plaintext );
    182183    }
    183 
    184184
    185185    /**
    … …  
    209209     */
    210210    protected function extract_links( $richtext, $plaintext, $extra_args = array() ) {
    211         $data = array( 'has' => array( 'links' => 0 ), 'links' => array() );
     211        $data = array(
     212            'has'   => array( 'links' => 0 ),
     213            'links' => array(),
     214        );
    212215
    213216        // Matches: href="text" and href='text'.
    … …  
    269272     */
    270273    protected function extract_mentions( $richtext, $plaintext, $extra_args = array() ) {
    271         $data     = array( 'has' => array( 'mentions' => 0 ), 'mentions' => array() );
     274        $data     = array(
     275            'has'      => array( 'mentions' => 0 ),
     276            'mentions' => array(),
     277        );
    272278        $mentions = array();
    273279
    … …  
    279285            }
    280286
    281         // If the Activity component is disabled, instead do a basic parse.
    282         } else {
    283             if ( strpos( $plaintext, '@' ) !== false ) {
     287            // If the Activity component is disabled, instead do a basic parse.
     288        } elseif ( strpos( $plaintext, '@' ) !== false ) {
    284289                preg_match_all( '/[@]+([A-Za-z0-9-_\.@]+)\b/', $plaintext, $matches );
    285290
    286                 if ( ! empty( $matches[1] ) ) {
    287                     $mentions = array_unique( array_map( 'strtolower', $matches[1] ) );
    288                 }
     291            if ( ! empty( $matches[1] ) ) {
     292                $mentions = array_unique( array_map( 'strtolower', $matches[1] ) );
    289293            }
    290294        }
    … …  
    343347     */
    344348    protected function extract_images( $richtext, $plaintext, $extra_args = array() ) {
    345         $media = array( 'has' => array( 'images' => 0 ), 'images' => array() );
     349        $media = array(
     350            'has'    => array( 'images' => 0 ),
     351            'images' => array(),
     352        );
    346353
    347354        $featured_image = $this->extract_images_from_featured_images( $richtext, $plaintext, $extra_args );
    348355        $galleries      = $this->extract_images_from_galleries( $richtext, $plaintext, $extra_args );
    349 
    350356
    351357        // `<img src>` tags.
    … …  
    421427        $media['has']['images'] = count( $media['images'] );
    422428
    423 
    424429        /**
    425430         * Filters images extracted from text.
    … …  
    461466     */
    462467    protected function extract_shortcodes( $richtext, $plaintext, $extra_args = array() ) {
    463         $data = array( 'has' => array( 'shortcodes' => 0 ), 'shortcodes' => array() );
     468        $data = array(
     469            'has'        => array( 'shortcodes' => 0 ),
     470            'shortcodes' => array(),
     471        );
    464472
    465473        // Match any registered WordPress shortcodes.
    … …  
    519527     */
    520528    protected function extract_embeds( $richtext, $plaintext, $extra_args = array() ) {
    521         $data   = array( 'has' => array( 'embeds' => 0 ), 'embeds' => array() );
     529        $data = array(
     530            'has'    => array( 'embeds' => 0 ),
     531            'embeds' => array(),
     532        );
    522533
    523534        if ( ! function_exists( '_wp_oembed_get_object' ) ) {
    524             require( ABSPATH . WPINC . '/class-oembed.php' );
    525         }
    526 
     535            require ABSPATH . WPINC . '/class-oembed.php';
     536        }
    527537
    528538        // Matches any links on their own lines. They may be oEmbeds.
    … …  
    599609     */
    600610    protected function extract_audio( $richtext, $plaintext, $extra_args = array() ) {
    601         $data   = array( 'has' => array( 'audio' => 0 ), 'audio' => array() );
     611        $data   = array(
     612            'has'   => array( 'audio' => 0 ),
     613            'audio' => array(),
     614        );
    602615        $audios = $this->extract_shortcodes( $richtext, $plaintext, $extra_args );
    603616        $links  = $this->extract_links( $richtext, $plaintext, $extra_args );
    604617
    605618        $audio_types = wp_get_audio_extensions();
    606 
    607619
    608620        // [audio]
    … …  
    695707     */
    696708    protected function extract_video( $richtext, $plaintext, $extra_args = array() ) {
    697         $data   = array( 'has' => array( 'videos' => 0 ), 'videos' => array() );
     709        $data   = array(
     710            'has'    => array( 'videos' => 0 ),
     711            'videos' => array(),
     712        );
    698713        $videos = $this->extract_shortcodes( $richtext, $plaintext, $extra_args );
    699714
    700715        $video_types = wp_get_video_extensions();
    701716
    702 
    703         // [video]
     717        // [video].
    704718        $videos = wp_list_filter( $videos['shortcodes'], array( 'type' => 'video' ) );
    705719        foreach ( $videos as $video ) {
    … …  
    787801                    $image_size = $extra_args['width'];  // E.g. "thumb", "medium".
    788802                }
    789 
    790803            } else {
    791804                $image_size = 'full';
    … …  
    807820                    $images = wp_parse_id_list( $gallery['ids'] );
    808821
    809                 // Gallery post_parent variant.
     822                    // Gallery post_parent variant.
    810823                } elseif ( isset( $extra_args['post'] ) ) {
    811824                    $images = wp_parse_id_list(
    812                         get_children( array(
    813                             'fields'         => 'ids',
    814                             'order'          => 'ASC',
    815                             'orderby'        => 'menu_order ID',
    816                             'post_mime_type' => 'image',
    817                             'post_parent'    => $extra_args['post']->ID,
    818                             'post_status'    => 'inherit',
    819                             'post_type'      => 'attachment',
    820                         ) )
     825                        get_children(
     826                            array(
     827                                'fields'         => 'ids',
     828                                'order'          => 'ASC',
     829                                'orderby'        => 'menu_order ID',
     830                                'post_mime_type' => 'image',
     831                                'post_parent'    => $extra_args['post']->ID,
     832                                'post_status'    => 'inherit',
     833                                'post_type'      => 'attachment',
     834                            )
     835                        )
    821836                    );
    822837                }
    … …  
    824839                // Extract the data we need from each image in this gallery.
    825840                foreach ( $images as $image_id ) {
    826                     $image  = wp_get_attachment_image_src( $image_id, $image_size );
     841                    $image = wp_get_attachment_image_src( $image_id, $image_size );
    827842
    828843                    $image_url    = isset( $image[0] ) ? $image[0] : '';
    … …  
    831846
    832847                    $data[] = array(
    833                         'url'    => $image_url,
    834                         'width'  => $image_width,
    835                         'height' => $image_height,
     848                        'url'        => $image_url,
     849                        'width'      => $image_width,
     850                        'height'     => $image_height,
    836851
    837852                        'gallery_id' => 1 + $gallery_id,
Note: See TracChangeset for help on using the changeset viewer.