Changeset 5089

Timestamp:

03/23/2007 02:05:29 AM (18 years ago)

Author:

matt

Message:

Ryan slays the dragon. (Some of these return WP_Error objects.)

Location:

trunk

Files:

• app.php (modified) (2 diffs)
• wp-admin/admin-ajax.php (modified) (4 diffs)
• wp-admin/admin-functions.php (modified) (2 diffs)
• wp-admin/custom-header.php (modified) (1 diff)
• wp-admin/import/wordpress.php (modified) (1 diff)
• wp-comments-post.php (modified) (1 diff)
• wp-includes/comment.php (modified) (1 diff)
• wp-login.php (modified) (1 diff)
• wp-mail.php (modified) (1 diff)
• xmlrpc.php (modified) (6 diffs)

trunk/app.php

@@ -431 +431 @@
         log_app('Inserting Post. Data:', print_r($post_data,true));
 
-        $postID = (int) wp_insert_post($post_data);
+        $postID = wp_insert_post($post_data);
 
         if (!$postID) {
@@ -583 +583 @@
 
         // Save the data
-        $postID = (int) wp_insert_attachment($attachment, $file, $post);
+        $postID = wp_insert_attachment($attachment, $file, $post);
 
         if (!$postID) {

trunk/wp-admin/admin-ajax.php

@@ -116 +116 @@
         if ( !$category_nicename = sanitize_title($cat_name) )
             die('0');
-        if ( !$cat_id = (int) category_exists( $cat_name ) )
-            $cat_id = (int) wp_create_category( $cat_name );
+        if ( !$cat_id = category_exists( $cat_name ) )
+            $cat_id = wp_create_category( $cat_name );
         $cat_name = wp_specialchars(stripslashes($cat_name));
         $x->add( array(
@@ -157 +157 @@
     if ( $id < 0 ) {
         $now = current_time('timestamp', 1);
-        if ( $pid = (int) wp_insert_post( array(
+        if ( $pid = wp_insert_post( array(
             'post_title' => sprintf('Draft created on %s at %s', date(get_option('date_format'), $now), date(get_option('time_format'), $now))
         ) ) )
-            $mid = (int) add_meta( $pid );
+            $mid = add_meta( $pid );
         else
             die('0');
-    } else if ( !$mid = (int) add_meta( $id ) ) {
+    } else if ( !$mid = add_meta( $id ) ) {
         die('0');
     }
@@ -205 +205 @@
         die('-1');
     require_once(ABSPATH . WPINC . '/registration.php');
-    if ( !$user_id = (int) add_user() )
+    if ( !$user_id = add_user() )
         die('0');
     elseif ( is_wp_error( $user_id ) ) {
@@ -231 +231 @@
     if($_POST['post_ID'] < 0) {
         $_POST['temp_ID'] = $_POST['post_ID'];
-        $id = (int) wp_write_post();
-        if(is_wp_error($id))
+        $id = wp_write_post();
+        if( is_wp_error($id) )
             die($id->get_error_message());
         else

trunk/wp-admin/admin-functions.php

@@ -563 +563 @@
 
     if ( $update ) {
-        $user_id = (int) wp_update_user( get_object_vars( $user ));
-    } else {
-        $user_id = (int) wp_insert_user( get_object_vars( $user ));
+        $user_id = wp_update_user( get_object_vars( $user ));
+    } else {
+        $user_id = wp_insert_user( get_object_vars( $user ));
         wp_new_user_notification( $user_id );
     }
@@ -1970 +1970 @@
 
     // Save the data
-    $id = (int) wp_insert_attachment( $object, $file );
+    $id = wp_insert_attachment( $object, $file );
 
     return array( 'file' => $file, 'id' => $id );

trunk/wp-admin/custom-header.php

@@ -216 +216 @@
 
         // Save the data
-        $id = (int) wp_insert_attachment($object, $file);
+        $id = wp_insert_attachment($object, $file);
 
         $upload = array('file' => $file, 'id' => $id);

trunk/wp-admin/import/wordpress.php

@@ -202 +202 @@
                 $category_parent = '0';
             else
-                $category_parent = (int) category_exists($parent);
+                $category_parent = category_exists($parent);
 
             $catarr = compact('category_nicename', 'category_parent', 'posts_private', 'links_private', 'posts_private', 'cat_name');

trunk/wp-comments-post.php

@@ -55 +55 @@
 $commentdata = compact('comment_post_ID', 'comment_author', 'comment_author_email', 'comment_author_url', 'comment_content', 'comment_type', 'user_ID');
 
-$comment_id = (int) wp_new_comment( $commentdata );
+$comment_id = wp_new_comment( $commentdata );
 
 $comment = get_comment($comment_id);

trunk/wp-includes/comment.php

@@ -393 +393 @@
     $commentdata['comment_approved'] = wp_allow_comment($commentdata);
 
-    $comment_ID = (int) wp_insert_comment($commentdata);
+    $comment_ID = wp_insert_comment($commentdata);
 
     do_action('comment_post', $comment_ID, $commentdata['comment_approved']);

trunk/wp-login.php

@@ -243 +243 @@
             $user_pass = substr( md5( uniqid( microtime() ) ), 0, 7);
 
-            $user_id = (int) wp_create_user( $user_login, $user_pass, $user_email );
+            $user_id = wp_create_user( $user_login, $user_pass, $user_email );
             if ( !$user_id )
                 $errors['registerfail'] = sprintf(__('<strong>ERROR</strong>: Couldn&#8217;t register you... please contact the <a href="mailto:%s">webmaster</a> !'), get_option('admin_email'));

trunk/wp-mail.php

@@ -137 +137 @@
     $post_data = add_magic_quotes($post_data);
 
-    $post_ID = (int) wp_insert_post($post_data);
+    $post_ID = wp_insert_post($post_data);
 
     if (!$post_ID) {

trunk/xmlrpc.php

@@ -513 +513 @@
         );
 
-        $cat_id = (int) wp_insert_category($new_category);
+        $cat_id = wp_insert_category($new_category);
         if(!$cat_id) {
             return(new IXR_Error(500, "Sorry, the new category failed."));
@@ -803 +803 @@
       $post_data = compact('blog_ID', 'post_author', 'post_date', 'post_date_gmt', 'post_content', 'post_title', 'post_category', 'post_status');
 
-      $post_ID = (int) wp_insert_post($post_data);
+      $post_ID = wp_insert_post($post_data);
 
       if (!$post_ID) {
@@ -1031 +1031 @@
       $postdata = compact('post_author', 'post_date', 'post_date_gmt', 'post_content', 'post_title', 'post_category', 'post_status', 'post_excerpt', 'comment_status', 'ping_status', 'to_ping', 'post_type', 'post_name', 'post_password', 'post_parent', 'menu_order');
 
-      $post_ID = (int) wp_insert_post($postdata);
+      $post_ID = wp_insert_post($postdata);
 
       if (!$post_ID) {
@@ -1448 +1448 @@
 
         // Save the data
-        $id = (int) wp_insert_attachment( $attachment, $upload[ 'file' ], $post_id );
+        $id = wp_insert_attachment( $attachment, $upload[ 'file' ], $post_id );
         wp_update_attachment_metadata( $id, wp_generate_attachment_metadata( $id, $upload['file'] ) );
 
@@ -1838 +1838 @@
         $commentdata = compact('comment_post_ID', 'comment_author', 'comment_author_url', 'comment_content', 'comment_type');
 
-        $comment_ID = (int) wp_new_comment($commentdata);
+        $comment_ID = wp_new_comment($commentdata);
         do_action('pingback_post', $comment_ID);
 
@@ -1856 +1856 @@
         $url = $args;
 
-        $post_ID = (int) url_to_postid($url);
+        $post_ID = url_to_postid($url);
         if (!$post_ID) {
             // We aren't sure that the resource is available and/or pingback enabled