WordPress.org
Get WordPress

Make WordPress Core

Changeset 6125


Ignore:
Timestamp:
09/18/2007 04:32:22 PM (17 years ago)
Author:
ryan
Message:

Add checks for WP_Error. Props filosofo. see #4809

Location:
trunk
Files:
23 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/admin-ajax.php

    r5923 r6125  
    205205        if ( $pid = wp_insert_post( array(
    206206            'post_title' => sprintf('Draft created on %s at %s', date(get_option('date_format'), $now), date(get_option('time_format'), $now))
    207         ) ) )
     207        ) ) ) {
     208            if ( is_wp_error( $pid ) )
     209                return $pid;
    208210            $mid = add_meta( $pid );
     211        }
    209212        else
    210213            die('0');

  • trunk/wp-admin/import/blogger.php

    r6043 r6125  
    381381                        $AtomParser = new AtomParser();
    382382                        $AtomParser->parse( $entry );
    383                         $this->import_post($AtomParser->entry);
     383                        $result = $this->import_post($AtomParser->entry);
     384                        if ( is_wp_error( $result ) )
     385                            return $result;
    384386                        unset($AtomParser);
    385387                    }
    … …  
    519521
    520522            $post_id = wp_insert_post($post);
     523            if ( is_wp_error( $post_id ) )
     524                return $post_id;
    521525
    522526            wp_create_categories( array_map( 'addslashes', $entry->categories ), $post_id );
    … …  
    532536        }
    533537        $this->save_vars();
     538        return;
    534539    }
    535540
    … …  
    768773            $blog = is_array($_REQUEST['blog']) ? array_shift( array_keys( $_REQUEST['blog'] ) ) : $_REQUEST['blog'];
    769774            $blog = (int) $blog;
    770             $this->import_blog( $blog );
     775            $result = $this->import_blog( $blog );
     776            if ( is_wp_error( $result ) )
     777                echo $result->get_error_message();
    771778        } elseif ( isset($_GET['token']) )
    772779            $this->auth();

  • trunk/wp-admin/import/blogware.php

    r5087 r6125  
    9292                $postdata = compact('post_author', 'post_date', 'post_content', 'post_title', 'post_status');
    9393                $post_id = wp_insert_post($postdata);
     94                if ( is_wp_error( $post_id ) ) {
     95                    return $post_id;
     96                }
    9497                if (!$post_id) {
    9598                    _e("Couldn't get post ID");
    … …  
    156159
    157160        $this->file = $file['file'];
    158         $this->import_posts();
     161        $result = $this->import_posts();
     162        if ( is_wp_error( $result ) )
     163            return $result;
    159164        wp_import_cleanup($file['id']);
    160165
    … …  
    177182                break;
    178183            case 1 :
    179                 $this->import();
     184                $result = $this->import();
     185                if ( is_wp_error( $result ) )
     186                    $result->get_error_message();
    180187                break;
    181188        }

  • trunk/wp-admin/import/dotclear.php

    r6026 r6125  
    365365                            'comment_count'     => $post_nb_comment + $post_nb_trackback)
    366366                            );
     367                    if ( is_wp_error( $ret_id ) )
     368                        return $ret_id;
    367369                }
    368370                else
    … …  
    383385                            'comment_count'     => $post_nb_comment + $post_nb_trackback)
    384386                            );
     387                    if ( is_wp_error( $ret_id ) )
     388                        return $ret_id;
    385389                }
    386390                $dcposts2wpposts[$post_id] = $ret_id;
    … …  
    563567        // Post Import
    564568        $posts = $this->get_dc_posts();
    565         $this->posts2wp($posts);
     569        $result = $this->posts2wp($posts);
     570        if ( is_wp_error( $result ) )
     571            return $result;
    566572
    567573        echo '<form action="admin.php?import=dotclear&amp;step=4" method="post">';
    … …  
    711717                break;
    712718            case 3 :
    713                 $this->import_posts();
     719                $result = $this->import_posts();
     720                if ( is_wp_error( $result ) )
     721                    echo $result->get_error_message();
    714722                break;
    715723            case 4 :

  • trunk/wp-admin/import/greymatter.php

    r5404 r6125  
    234234                $postdata = compact('post_author', 'post_date', 'post_date_gmt', 'post_content', 'post_title', 'post_excerpt', 'post_status', 'comment_status', 'ping_status', 'post_modified', 'post_modified_gmt');
    235235                $post_ID = wp_insert_post($postdata);
     236                if ( is_wp_error( $post_ID ) )
     237                    return $post_ID;
    236238            }
    237239
    … …  
    288290<?php
    289291    $this->footer();
     292    return;
    290293    }
    291294
    … …  
    302305            case 1:
    303306                check_admin_referer('import-greymatter');
    304                 $this->import();
     307                $result = $this->import();
     308                if ( is_wp_error( $result ) )
     309                    echo $result->get_error_message();
    305310                break;
    306311        }

  • trunk/wp-admin/import/livejournal.php

    r5404 r6125  
    7272                $postdata = compact('post_author', 'post_date', 'post_content', 'post_title', 'post_status');
    7373                $post_id = wp_insert_post($postdata);
     74                if ( is_wp_error( $post_id ) )
     75                    return $post_id;
    7476                if (!$post_id) {
    7577                    _e("Couldn't get post ID");
    … …  
    133135
    134136        $this->file = $file['file'];
    135         $this->import_posts();
     137        $result = $this->import_posts();
     138        if ( is_wp_error( $result ) )
     139            return $result;
    136140        wp_import_cleanup($file['id']);
    137141
    … …  
    155159            case 1 :
    156160                check_admin_referer('import-upload');
    157                 $this->import();
     161                $result = $this->import();
     162                if ( is_wp_error( $result ) )
     163                    echo $result->get_error_message();
    158164                break;
    159165        }

  • trunk/wp-admin/import/mt.php

    r6101 r6125  
    216216            $post->post_author = $this->checkauthor($post->post_author); //just so that if a post already exists, new users are not created by checkauthor
    217217            $post_id = wp_insert_post($post);
     218            if ( is_wp_error( $post_id ) )
     219                return $post_id;
    218220
    219221            // Add categories.
    … …  
    292294                // Finishing a post.
    293295                $context = '';
    294                 $this->save_post($post, $comments, $pings);
     296                $result = $this->save_post($post, $comments, $pings);
     297                if ( is_wp_error( $result ) )
     298                    return $result;
    295299                $post = new StdClass;
    296300                $comment = new StdClass();
    … …  
    416420            $this->file = get_attached_file($this->id);
    417421        $this->get_authors_from_post();
    418         $this->process_posts();
     422        $result = $this->process_posts();
     423        if ( is_wp_error( $result ) )
     424            return $result;
    419425    }
    420426
    … …  
    435441            case 2:
    436442                check_admin_referer('import-mt');
    437                 $this->import();
     443                $result = $this->import();
     444                if ( is_wp_error( $result ) )
     445                    echo $result->get_error_message();
    438446                break;
    439447        }

  • trunk/wp-admin/import/rss.php

    r5404 r6125  
    111111            } else {
    112112                $post_id = wp_insert_post($post);
     113                if ( is_wp_error( $post_id ) )
     114                    return $post_id;
    113115                if (!$post_id) {
    114116                    _e("Couldn't get post ID");
    … …  
    136138        $this->file = $file['file'];
    137139        $this->get_posts();
    138         $this->import_posts();
     140        $result = $this->import_posts();
     141        if ( is_wp_error( $result ) )
     142            return $result;
    139143        wp_import_cleanup($file['id']);
    140144
    … …  
    158162            case 1 :
    159163                check_admin_referer('import-upload');
    160                 $this->import();
     164                $result = $this->import();
     165                if ( is_wp_error( $result ) )
     166                    echo $result->get_error_message();
    161167                break;
    162168        }

  • trunk/wp-admin/import/textpattern.php

    r6026 r6125  
    306306                        'comment_count'     => $comments_count)
    307307                        );
     308                    if ( is_wp_error( $ret_id ) )
     309                        return $ret_id;
    308310                }
    309311                else
    … …  
    322324                        'comment_count'     => $comments_count)
    323325                        );
     326                    if ( is_wp_error( $ret_id ) )
     327                        return $ret_id;
    324328                }
    325329                $txpposts2wpposts[$ID] = $ret_id;
    … …  
    499503        // Post Import
    500504        $posts = $this->get_txp_posts();
    501         $this->posts2wp($posts);
     505        $result = $this->posts2wp($posts);
     506        if ( is_wp_error( $result ) )
     507            return $result;
    502508
    503509        echo '<form action="admin.php?import=textpattern&amp;step=4" method="post">';
    … …  
    639645                break;
    640646            case 3 :
    641                 $this->import_posts();
     647                $result = $this->import_posts();
     648                if ( is_wp_error( $result ) )
     649                    echo $result->get_error_message();
    642650                break;
    643651            case 4 :

  • trunk/wp-admin/import/wordpress.php

    r6026 r6125  
    251251        echo '<ol>';
    252252
    253         foreach ($this->posts as $post)
    254             $this->process_post($post);
     253        foreach ($this->posts as $post) {
     254            $result = $this->process_post($post);
     255            if ( is_wp_error( $result ) )
     256                return $result;
     257        }
    255258
    256259        echo '</ol>';
    … …  
    304307            $post_parent = (int) $post_parent;
    305308            if ($parent = $this->posts_processed[$post_parent]) {
    306                 if (!$parent[1]) $this->process_post($parent[0]); // If not yet, process the parent first.
     309                if (!$parent[1]) {
     310                    $result = $this->process_post($parent[0]); // If not yet, process the parent first.
     311                    if ( is_wp_error( $result ) )
     312                        return $result;
     313                }
    307314                $post_parent = $parent[1]; // New ID of the parent;
    308315            }
    … …  
    315322            $postdata = compact('post_author', 'post_date', 'post_date_gmt', 'post_content', 'post_title', 'post_excerpt', 'post_status', 'post_name', 'comment_status', 'ping_status', 'post_modified', 'post_modified_gmt', 'guid', 'post_parent', 'menu_order', 'post_type');
    316323            $comment_post_ID = $post_id = wp_insert_post($postdata);
     324            if ( is_wp_error( $post_id ) )
     325                return $post_id;
    317326
    318327            // Memorize old and new ID.
    … …  
    383392        $this->get_entries();
    384393        $this->process_categories();
    385         $this->process_posts();
     394        $result = $this->process_posts();
     395        if ( is_wp_error( $result ) )
     396            return $result;
    386397    }
    387398
    … …  
    403414            case 2:
    404415                check_admin_referer('import-wordpress');
    405                 $this->import();
     416                $result = $this->import();
     417                if ( is_wp_error( $result ) )
     418                    echo $result->get_error_message();
    406419                break;
    407420        }

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

    r5566 r6125  
    99function register_importer( $id, $name, $description, $callback ) {
    1010    global $wp_importers;
    11 
     11    if ( is_wp_error( $callback ) )
     12        return $callback;
    1213    $wp_importers[$id] = array ( $name, $description, $callback );
    1314}

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

    r6026 r6125  
    285285    // Create the post.
    286286    $post_ID = wp_insert_post( $_POST );
     287    if ( is_wp_error( $post_ID ) )
     288        return $post_ID;
    287289
    288290    if ( empty($post_ID) )

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

    r6053 r6125  
    993993                        // Add the column list to the index create string
    994994                        $index_string .= ' ('.$index_columns.')';
    995 
     995                        error_log("Index string: $index_string", 0);
    996996                        if(!(($aindex = array_search($index_string, $indices)) === false)) {
    997997                            unset($indices[$aindex]);

  • trunk/wp-admin/link-manager.php

    r5906 r6125  
    160160                    foreach ($link->link_category as $category) {
    161161                        $cat = get_term($category, 'link_category', OBJECT, 'display');
     162                        if ( is_wp_error( $cat ) )
     163                            echo $cat->get_error_message();
    162164                        $cat_name = $cat->name;
    163165                        if ( $cat_id != $category )

  • trunk/wp-app.php

    r6032 r6125  
    263263
    264264        $postID = wp_insert_post($post_data);
     265        if ( is_wp_error( $postID ) )
     266            $this->internal_error($postID->get_error_message());
    265267
    266268        if (!$postID) {

  • trunk/wp-includes/category-template.php

    r6091 r6125  
    1313
    1414        $category = get_category($cat_id);
     15        if ( is_wp_error( $category ) )
     16            return $category;
    1517        if ( $category->parent == $id ) {
    1618            $chain .= $before.$category->term_id.$after;
    … …  
    3032    } else {
    3133        $category = &get_category($category_id);
     34        if ( is_wp_error( $category ) )
     35            return $category;
    3236        $category_nicename = $category->slug;
    3337
    … …  
    4448    $chain = '';
    4549    $parent = &get_category($id);
     50    if ( is_wp_error( $parent ) )
     51        return $parent;
    4652
    4753    if ( $nicename )
    … …  
    99105    $cat_ID = (int) $cat_ID;
    100106    $category = &get_category($cat_ID);
     107    if ( is_wp_error( $category ) )
     108        return $category;
    101109    return $category->name;
    102110}
    … …  
    314322
    315323    $return = wp_generate_tag_cloud( $tags, $args ); // Here's where those top tags get sorted according to $args
    316     echo apply_filters( 'wp_tag_cloud', $return, $args );
     324    if ( is_wp_error( $return ) )
     325        return false;
     326    else
     327        echo apply_filters( 'wp_tag_cloud', $return, $args );
    317328}
    318329
    … …  
    335346        $counts[$tag->name] = $tag->count;
    336347        $tag_links[$tag->name] = get_tag_link( $tag->term_id );
     348        if ( is_wp_error( $tag_links[$tag->name] ) )
     349            return $tag_links[$tag->name];
    337350        $tag_ids[$tag->name] = $tag->term_id;
    338351    }
    … …  
    411424
    412425    $tag = &get_term($tag_id, 'post_tag');
     426    if ( is_wp_error( $tag ) )
     427        return $tag;
    413428    $slug = $tag->slug;
    414429
    … …  
    451466
    452467    $tag_list = $before;
    453     foreach ( $tags as $tag )
    454         $tag_links[] = '<a href="' . get_tag_link($tag->term_id) . '" rel="tag">' . $tag->name . '</a>';
     468    foreach ( $tags as $tag ) {
     469        $link = get_tag_link($tag->term_id);
     470        if ( is_wp_error( $link ) )
     471            return $link;
     472        $tag_links[] = '<a href="' . $link . '" rel="tag">' . $tag->name . '</a>';
     473    }
    455474
    456475    $tag_links = join( $sep, $tag_links );
    … …  
    464483
    465484function the_tags( $before = 'Tags: ', $sep = ', ', $after = '' ) {
    466     echo get_the_tag_list($before, $sep, $after);
     485    $return = get_the_tag_list($before, $sep, $after);
     486    if ( is_wp_error( $return ) )
     487        return false;
     488    else
     489        echo $return;
    467490}
    468491

  • trunk/wp-includes/category.php

    r6071 r6125  
    3131function &get_category($category, $output = OBJECT, $filter = 'raw') {
    3232    $category = get_term($category, 'category', $output, $filter);
     33    if ( is_wp_error( $category ) )
     34        return $category;
    3335
    3436    _make_cat_compat($category);
    … …  
    5961        while ( ($curcategory->parent != 0) && ($curcategory->parent != $curcategory->term_id) ) {
    6062            $curcategory = get_term($curcategory->parent, 'category');
     63            if ( is_wp_error( $curcategory ) )
     64                return $curcategory;
    6165            $path = '/' . $curcategory->slug . $path;
    6266        }

  • trunk/wp-includes/feed.php

    r6114 r6125  
    1313function get_wp_title_rss($sep = '&#187;') {
    1414    $title = wp_title($sep, false);
     15    if ( is_wp_error( $title ) )
     16        return $title->get_error_message();
    1517    $title = apply_filters('get_wp_title_rss', $title);
    1618    return $title;

  • trunk/wp-includes/general-template.php

    r6119 r6125  
    196196    if ( !empty($tag) ) {
    197197        $tag = get_term($tag, 'post_tag', OBJECT, 'display');
     198        if ( is_wp_error( $tag ) )
     199            return $tag;
    198200        if ( ! empty($tag->name) )
    199201            $title = apply_filters('single_tag_title', $tag->name);

  • trunk/wp-includes/query.php

    r6074 r6125  
    941941        if ( !empty($q['category__not_in']) ) {
    942942            $ids = get_objects_in_term($q['category__not_in'], 'category');
     943            if ( is_wp_error( $ids ) )
     944                return $ids;
    943945            if ( is_array($ids) && count($ids > 0) ) {
    944946                $out_posts = "'" . implode("', '", $ids) . "'";
    … …  
    14161418            $tag_id = $this->get('tag_id');
    14171419            $tag = &get_term($tag_id, 'post_tag');
     1420            if ( is_wp_error( $tag ) )
     1421                return $tag;
    14181422            $this->queried_object = &$tag;
    14191423            $this->queried_object_id = (int) $tag_id;

  • trunk/wp-includes/taxonomy.php

    r6121 r6125  
    392392    $term = (int) $term;
    393393    $term = get_term( $term, $taxonomy );
    394 
    395394    if ( is_wp_error($term) )
    396395        return $term;
    … …  
    802801    if ( is_taxonomy_hierarchical($taxonomy) ) {
    803802        $term_obj = get_term($term, $taxonomy);
     803        if ( is_wp_error( $term_obj ) )
     804            return $term_obj;
    804805        $parent = $term_obj->parent;
    805806
    … …  
    12531254        if ( !is_object($term) ) {
    12541255            $term = get_term($term, $taxonomy);
     1256            if ( is_wp_error( $term ) )
     1257                return $term;
    12551258            $use_id = true;
    12561259        }

  • trunk/wp-mail.php

    r6056 r6125  
    149149
    150150    $post_ID = wp_insert_post($post_data);
     151    if ( is_wp_error( $post_ID ) )
     152        echo "\n" . $post_ID->get_error_message();
    151153
    152154    if (!$post_ID) {

  • trunk/xmlrpc.php

    r6026 r6125  
    801801
    802802      $post_ID = wp_insert_post($post_data);
     803      if ( is_wp_error( $post_ID ) )
     804        return new IXR_Error(500, $post_ID->get_error_message());
    803805
    804806      if (!$post_ID) {
    … …  
    10891091
    10901092      $post_ID = wp_insert_post($postdata);
     1093      if ( is_wp_error( $post_ID ) )
     1094        return new IXR_Error(500, $post_ID->get_error_message());
    10911095
    10921096      if (!$post_ID) {
Note: See TracChangeset for help on using the changeset viewer.