WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/13/2008 07:17:40 AM (18 years ago)
Author:
markjaquith
Message:

Code cleanup for wp-admin/includes/bookmark.php

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/includes/bookmark.php

    r9053 r9659  
    7777 * @return unknown
    7878 */
    79 function wp_delete_link($link_id) {
     79function wp_delete_link( $link_id ) {
    8080        global $wpdb;
    8181
    82         do_action('delete_link', $link_id);
    83 
    84         wp_delete_object_term_relationships($link_id, 'link_category');
    85 
    86         $wpdb->query( $wpdb->prepare("DELETE FROM $wpdb->links WHERE link_id = %d", $link_id) );
    87 
    88         do_action('deleted_link', $link_id);
    89 
    90         clean_bookmark_cache($link_id);
     82        do_action( 'delete_link', $link_id );
     83
     84        wp_delete_object_term_relationships( $link_id, 'link_category' );
     85
     86        $wpdb->query( $wpdb->prepare( "DELETE FROM $wpdb->links WHERE link_id = %d", $link_id ) );
     87
     88        do_action( 'deleted_link', $link_id );
     89
     90        clean_bookmark_cache( $link_id );
    9191
    9292        return true;
    … …  
    101101 * @return unknown
    102102 */
    103 function wp_get_link_cats($link_id = 0) {
    104 
    105         $cats = wp_get_object_terms($link_id, 'link_category', 'fields=ids');
    106 
    107         return array_unique($cats);
     103function wp_get_link_cats( $link_id = 0 ) {
     104
     105        $cats = wp_get_object_terms( $link_id, 'link_category', 'fields=ids' );
     106
     107        return array_unique( $cats );
    108108}
    109109
    … …  
    128128 * @return unknown
    129129 */
    130 function wp_insert_link($linkdata, $wp_error = false) {
     130function wp_insert_link( $linkdata, $wp_error = false ) {
    131131        global $wpdb, $current_user;
    132132
    133         $defaults = array('link_id' => 0, 'link_name' => '', 'link_url' => '', 'link_rating' => 0 );
    134 
    135         $linkdata = wp_parse_args($linkdata, $defaults);
    136         $linkdata = sanitize_bookmark($linkdata, 'db');
    137 
    138         extract(stripslashes_deep($linkdata), EXTR_SKIP);
     133        $defaults = array( 'link_id' => 0, 'link_name' => '', 'link_url' => '', 'link_rating' => 0 );
     134
     135        $linkdata = wp_parse_args( $linkdata, $defaults );
     136        $linkdata = sanitize_bookmark( $linkdata, 'db' );
     137
     138        extract( stripslashes_deep( $linkdata ), EXTR_SKIP );
    139139
    140140        $update = false;
    141141
    142         if ( !empty($link_id) )
     142        if ( !empty( $link_id ) )
    143143                $update = true;
    144144
    … …  
    149149                return 0;
    150150
    151         if ( empty($link_rating) )
     151        if ( empty( $link_rating ) )
    152152                $link_rating = 0;
    153153
    154         if ( empty($link_image) )
     154        if ( empty( $link_image ) )
    155155                $link_image = '';
    156156
    157         if ( empty($link_target) )
     157        if ( empty( $link_target ) )
    158158                $link_target = '';
    159159
    160         if ( empty($link_visible) )
     160        if ( empty( $link_visible ) )
    161161                $link_visible = 'Y';
    162162
    163         if ( empty($link_owner) )
     163        if ( empty( $link_owner ) )
    164164                $link_owner = $current_user->id;
    165165
    166         if ( empty($link_notes) )
     166        if ( empty( $link_notes ) )
    167167                $link_notes = '';
    168168
    169         if ( empty($link_description) )
     169        if ( empty( $link_description ) )
    170170                $link_description = '';
    171171
    172         if ( empty($link_rss) )
     172        if ( empty( $link_rss ) )
    173173                $link_rss = '';
    174174
    175         if ( empty($link_rel) )
     175        if ( empty( $link_rel ) )
    176176                $link_rel = '';
    177177
    178178        // Make sure we set a valid category
    179         if (0 == count($link_category) || !is_array($link_category)) {
    180                 $link_category = array(get_option('default_link_category'));
     179        if ( 0 == count( $link_category ) || !is_array( $link_category ) ) {
     180                $link_category = array( get_option( 'default_link_category' ) );
    181181        }
    182182
    183183        if ( $update ) {
    184                 if ( false === $wpdb->query( $wpdb->prepare("UPDATE $wpdb->links SET link_url = %s,
     184                if ( false === $wpdb->query( $wpdb->prepare( "UPDATE $wpdb->links SET link_url = %s,
    185185                        link_name = %s, link_image = %s, link_target = %s,
    186186                        link_visible = %s, link_description = %s, link_rating = %s,
    187187                        link_rel = %s, link_notes = %s, link_rss = %s
    188                         WHERE link_id = %s", $link_url, $link_name, $link_image, $link_target, $link_visible, $link_description, $link_rating, $link_rel, $link_notes, $link_rss, $link_id) ) ) {
     188                        WHERE link_id = %s", $link_url, $link_name, $link_image, $link_target, $link_visible, $link_description, $link_rating, $link_rel, $link_notes, $link_rss, $link_id ) ) ) {
    189189                        if ( $wp_error )
    190                                 return new WP_Error('db_update_error', __('Could not update link in the database'), $wpdb->last_error);
     190                                return new WP_Error( 'db_update_error', __( 'Could not update link in the database' ), $wpdb->last_error );
    191191                        else
    192192                                return 0;
    193193                }
    194194        } else {
    195                 if ( false === $wpdb->query( $wpdb->prepare("INSERT INTO $wpdb->links (link_url, link_name, link_image, link_target, link_description, link_visible, link_owner, link_rating, link_rel, link_notes, link_rss) VALUES(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)",
    196                 $link_url,$link_name, $link_image, $link_target, $link_description, $link_visible, $link_owner, $link_rating, $link_rel, $link_notes, $link_rss) ) ) {
     195                if ( false === $wpdb->query( $wpdb->prepare( "INSERT INTO $wpdb->links (link_url, link_name, link_image, link_target, link_description, link_visible, link_owner, link_rating, link_rel, link_notes, link_rss) VALUES(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)",
     196                $link_url,$link_name, $link_image, $link_target, $link_description, $link_visible, $link_owner, $link_rating, $link_rel, $link_notes, $link_rss ) ) ) {
    197197                        if ( $wp_error )
    198                                 return new WP_Error('db_insert_error', __('Could not insert link into the database'), $wpdb->last_error);
     198                                return new WP_Error( 'db_insert_error', __( 'Could not insert link into the database' ), $wpdb->last_error );
    199199                        else
    200200                                return 0;
    … …  
    203203        }
    204204
    205         wp_set_link_cats($link_id, $link_category);
     205        wp_set_link_cats( $link_id, $link_category );
    206206
    207207        if ( $update )
    208                 do_action('edit_link', $link_id);
    209         else
    210                 do_action('add_link', $link_id);
    211 
    212         clean_bookmark_cache($link_id);
     208                do_action( 'edit_link', $link_id );
     209        else
     210                do_action( 'add_link', $link_id );
     211
     212        clean_bookmark_cache( $link_id );
    213213
    214214        return $link_id;
    … …  
    223223 * @param unknown_type $link_categories
    224224 */
    225 function wp_set_link_cats($link_id = 0, $link_categories = array()) {
     225function wp_set_link_cats( $link_id = 0, $link_categories = array() ) {
    226226        // If $link_categories isn't already an array, make it one:
    227         if (!is_array($link_categories) || 0 == count($link_categories))
    228                 $link_categories = array(get_option('default_link_category'));
    229 
    230         $link_categories = array_map('intval', $link_categories);
    231         $link_categories = array_unique($link_categories);
    232 
    233         wp_set_object_terms($link_id, $link_categories, 'link_category');
    234 
    235         clean_bookmark_cache($link_id);
     227        if ( !is_array( $link_categories ) || 0 == count( $link_categories ) )
     228                $link_categories = array( get_option( 'default_link_category' ) );
     229
     230        $link_categories = array_map( 'intval', $link_categories );
     231        $link_categories = array_unique( $link_categories );
     232
     233        wp_set_object_terms( $link_id, $link_categories, 'link_category' );
     234
     235        clean_bookmark_cache( $link_id );
    236236}       // wp_set_link_cats()
    237237
    … …  
    244244 * @return unknown
    245245 */
    246 function wp_update_link($linkdata) {
     246function wp_update_link( $linkdata ) {
    247247        $link_id = (int) $linkdata['link_id'];
    248248
    249         $link = get_link($link_id, ARRAY_A);
     249        $link = get_link( $link_id, ARRAY_A );
    250250
    251251        // Escape data pulled from DB.
    252         $link = add_magic_quotes($link);
     252        $link = add_magic_quotes( $link );
    253253
    254254        // Passed link category list overwrites existing category list if not empty.
    255         if ( isset($linkdata['link_category']) && is_array($linkdata['link_category'])
    256                          && 0 != count($linkdata['link_category']) )
     255        if ( isset( $linkdata['link_category'] ) && is_array( $linkdata['link_category'] )
     256                         && 0 != count( $linkdata['link_category'] ) )
    257257                $link_cats = $linkdata['link_category'];
    258258        else
    … …  
    260260
    261261        // Merge old and new fields with new fields overwriting old ones.
    262         $linkdata = array_merge($link, $linkdata);
     262        $linkdata = array_merge( $link, $linkdata );
    263263        $linkdata['link_category'] = $link_cats;
    264264
    265         return wp_insert_link($linkdata);
     265        return wp_insert_link( $linkdata );
    266266}
    267267
Note: See TracChangeset for help on using the changeset viewer.