WordPress.org

Make WordPress Core

Changeset 28406


Ignore:
Timestamp:
05/15/2014 02:27:55 AM (7 years ago)
Author:
wonderboymusic
Message:

Eliminate use of extract() in wp_insert_link().

See #22400.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/bookmark.php

    r25706 r28406  
    147147    $defaults = array( 'link_id' => 0, 'link_name' => '', 'link_url' => '', 'link_rating' => 0 );
    148148
    149     $linkdata = wp_parse_args( $linkdata, $defaults );
    150     $linkdata = sanitize_bookmark( $linkdata, 'db' );
     149    $args = wp_parse_args( $linkdata, $defaults );
     150    $r = sanitize_bookmark( $args, 'db' );
    151151
    152152    extract( wp_unslash( $linkdata ), EXTR_SKIP );
    153153
    154154    $update = false;
    155 
    156     if ( !empty( $link_id ) )
     155    if ( ! empty( $r['link_id'] ) ) {
    157156        $update = true;
    158 
    159     if ( trim( $link_name ) == '' ) {
    160         if ( trim( $link_url ) != '' ) {
    161             $link_name = $link_url;
     157    }
     158
     159    if ( trim( $r['link_name'] ) == '' ) {
     160        if ( trim( $r['link_url'] ) != '' ) {
     161            $r['link_name'] = $r['link_url'];
    162162        } else {
    163163            return 0;
     
    165165    }
    166166
    167     if ( trim( $link_url ) == '' )
     167    if ( trim( $r['link_url'] ) == '' ) {
    168168        return 0;
    169 
    170     if ( empty( $link_rating ) )
    171         $link_rating = 0;
    172 
    173     if ( empty( $link_image ) )
    174         $link_image = '';
    175 
    176     if ( empty( $link_target ) )
    177         $link_target = '';
    178 
    179     if ( empty( $link_visible ) )
    180         $link_visible = 'Y';
    181 
    182     if ( empty( $link_owner ) )
    183         $link_owner = get_current_user_id();
    184 
    185     if ( empty( $link_notes ) )
    186         $link_notes = '';
    187 
    188     if ( empty( $link_description ) )
    189         $link_description = '';
    190 
    191     if ( empty( $link_rss ) )
    192         $link_rss = '';
    193 
    194     if ( empty( $link_rel ) )
    195         $link_rel = '';
     169    }
     170    if ( empty( $r['link_rating'] ) ) {
     171        $r['link_rating'] = 0;
     172    }
     173    if ( empty( $r['link_image'] ) ) {
     174        $r['link_image'] = '';
     175    }
     176    if ( empty( $r['link_target'] ) ) {
     177        $r['link_target'] = '';
     178    }
     179    if ( empty( $r['link_visible'] ) ) {
     180        $r['link_visible'] = 'Y';
     181    }
     182    if ( empty( $r['link_owner'] ) ) {
     183        $r['link_owner'] = get_current_user_id();
     184    }
     185    if ( empty( $r['link_notes'] ) ) {
     186        $r['link_notes'] = '';
     187    }
     188    if ( empty( $r['link_description'] ) ) {
     189        $r['link_description'] = '';
     190    }
     191    if ( empty( $r['link_rss'] ) ) {
     192        $r['link_rss'] = '';
     193    }
     194    if ( empty( $r['link_rel'] ) ) {
     195        $r['link_rel'] = '';
     196    }
    196197
    197198    // Make sure we set a valid category
    198     if ( ! isset( $link_category ) || 0 == count( $link_category ) || !is_array( $link_category ) ) {
     199    if ( ! isset( $r['link_category'] ) || ! is_array( $r['link_category'] ) || 0 == count( $r['link_category'] ) ) {
    199200        $link_category = array( get_option( 'default_link_category' ) );
    200201    }
    201202
    202203    if ( $update ) {
    203         if ( false === $wpdb->update( $wpdb->links, compact('link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_rating', 'link_rel', 'link_notes', 'link_rss'), compact('link_id') ) ) {
    204             if ( $wp_error )
     204        if ( false === $wpdb->update( $wpdb->links, compact( 'link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_rating', 'link_rel', 'link_notes', 'link_rss' ), compact( 'link_id' ) ) ) {
     205            if ( $wp_error ) {
    205206                return new WP_Error( 'db_update_error', __( 'Could not update link in the database' ), $wpdb->last_error );
    206             else
     207            } else {
    207208                return 0;
     209            }
    208210        }
    209211    } else {
    210         if ( false === $wpdb->insert( $wpdb->links, compact('link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_owner', 'link_rating', 'link_rel', 'link_notes', 'link_rss') ) ) {
    211             if ( $wp_error )
     212        if ( false === $wpdb->insert( $wpdb->links, compact( 'link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_owner', 'link_rating', 'link_rel', 'link_notes', 'link_rss' ) ) ) {
     213            if ( $wp_error ) {
    212214                return new WP_Error( 'db_insert_error', __( 'Could not insert link into the database' ), $wpdb->last_error );
    213             else
     215            } else {
    214216                return 0;
     217            }
    215218        }
    216219        $link_id = (int) $wpdb->insert_id;
Note: See TracChangeset for help on using the changeset viewer.