Wordpress 3.5.2
[autoinstalls/wordpress.git] / wp-includes / pluggable.php
index a36bd1ed2a17c96ea16a1530b1cf652f8709c05a..83afefab805df6f7dc921aeb48b630169025be27 100644 (file)
@@ -27,12 +27,12 @@ if ( !function_exists('wp_set_current_user') ) :
 function wp_set_current_user($id, $name = '') {
        global $current_user;
 
-       if ( isset($current_user) && ($id == $current_user->ID) )
+       if ( isset( $current_user ) && ( $current_user instanceof WP_User ) && ( $id == $current_user->ID ) )
                return $current_user;
 
-       $current_user = new WP_User($id, $name);
+       $current_user = new WP_User( $id, $name );
 
-       setup_userdata($current_user->ID);
+       setup_userdata( $current_user->ID );
 
        do_action('set_current_user');
 
@@ -74,20 +74,37 @@ if ( !function_exists('get_currentuserinfo') ) :
 function get_currentuserinfo() {
        global $current_user;
 
-       if ( defined('XMLRPC_REQUEST') && XMLRPC_REQUEST )
+       if ( ! empty( $current_user ) ) {
+               if ( $current_user instanceof WP_User )
+                       return;
+
+               // Upgrade stdClass to WP_User
+               if ( is_object( $current_user ) && isset( $current_user->ID ) ) {
+                       $cur_id = $current_user->ID;
+                       $current_user = null;
+                       wp_set_current_user( $cur_id );
+                       return;
+               }
+
+               // $current_user has a junk value. Force to WP_User with ID 0.
+               $current_user = null;
+               wp_set_current_user( 0 );
                return false;
+       }
 
-       if ( ! empty($current_user) )
-               return;
+       if ( defined('XMLRPC_REQUEST') && XMLRPC_REQUEST ) {
+               wp_set_current_user( 0 );
+               return false;
+       }
 
        if ( ! $user = wp_validate_auth_cookie() ) {
-                if ( is_admin() || empty($_COOKIE[LOGGED_IN_COOKIE]) || !$user = wp_validate_auth_cookie($_COOKIE[LOGGED_IN_COOKIE], 'logged_in') ) {
-                       wp_set_current_user(0);
+                if ( is_blog_admin() || is_network_admin() || empty( $_COOKIE[LOGGED_IN_COOKIE] ) || !$user = wp_validate_auth_cookie( $_COOKIE[LOGGED_IN_COOKIE], 'logged_in' ) ) {
+                       wp_set_current_user( 0 );
                        return false;
                 }
        }
 
-       wp_set_current_user($user);
+       wp_set_current_user( $user );
 }
 endif;
 
@@ -98,27 +115,31 @@ if ( !function_exists('get_userdata') ) :
  * @since 0.71
  *
  * @param int $user_id User ID
- * @return bool|object False on failure, User DB row object
+ * @return bool|object False on failure, WP_User object on success
  */
 function get_userdata( $user_id ) {
-       global $wpdb;
-
-       if ( ! is_numeric( $user_id ) )
-               return false;
-
-       $user_id = absint( $user_id );
-       if ( ! $user_id )
-               return false;
-
-       $user = wp_cache_get( $user_id, 'users' );
+       return get_user_by( 'id', $user_id );
+}
+endif;
 
-       if ( $user )
-               return $user;
+if ( !function_exists('get_user_by') ) :
+/**
+ * Retrieve user info by a given field
+ *
+ * @since 2.8.0
+ *
+ * @param string $field The field to retrieve the user with. id | slug | email | login
+ * @param int|string $value A value for $field. A user ID, slug, email address, or login name.
+ * @return bool|object False on failure, WP_User object on success
+ */
+function get_user_by( $field, $value ) {
+       $userdata = WP_User::get_data_by( $field, $value );
 
-       if ( ! $user = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->users WHERE ID = %d LIMIT 1", $user_id ) ) )
+       if ( !$userdata )
                return false;
 
-       _fill_user( $user );
+       $user = new WP_User;
+       $user->init( $userdata );
 
        return $user;
 }
@@ -130,103 +151,26 @@ if ( !function_exists('cache_users') ) :
  *
  * @since 3.0.0
  *
- * @param array $users User ID numbers list
+ * @param array $user_ids User ID numbers list
  */
-function cache_users( $users ) {
+function cache_users( $user_ids ) {
        global $wpdb;
 
-       $clean = array();
-       foreach($users as $id) {
-               $id = (int) $id;
-               if (wp_cache_get($id, 'users')) {
-                       // seems to be cached already
-               } else {
-                       $clean[] = $id;
-               }
-       }
+       $clean = _get_non_cached_ids( $user_ids, 'users' );
 
-       if ( 0 == count($clean) )
+       if ( empty( $clean ) )
                return;
 
-       $list = implode(',', $clean);
-
-       $results = $wpdb->get_results("SELECT * FROM $wpdb->users WHERE ID IN ($list)");
-
-       _fill_many_users($results);
-}
-endif;
+       $list = implode( ',', $clean );
 
-if ( !function_exists('get_user_by') ) :
-/**
- * Retrieve user info by a given field
- *
- * @since 2.8.0
- *
- * @param string $field The field to retrieve the user with.  id | slug | email | login
- * @param int|string $value A value for $field.  A user ID, slug, email address, or login name.
- * @return bool|object False on failure, User DB row object
- */
-function get_user_by($field, $value) {
-       global $wpdb;
+       $users = $wpdb->get_results( "SELECT * FROM $wpdb->users WHERE ID IN ($list)" );
 
-       switch ($field) {
-               case 'id':
-                       return get_userdata($value);
-                       break;
-               case 'slug':
-                       $user_id = wp_cache_get($value, 'userslugs');
-                       $field = 'user_nicename';
-                       break;
-               case 'email':
-                       $user_id = wp_cache_get($value, 'useremail');
-                       $field = 'user_email';
-                       break;
-               case 'login':
-                       $value = sanitize_user( $value );
-                       $user_id = wp_cache_get($value, 'userlogins');
-                       $field = 'user_login';
-                       break;
-               default:
-                       return false;
+       $ids = array();
+       foreach ( $users as $user ) {
+               update_user_caches( $user );
+               $ids[] = $user->ID;
        }
-
-        if ( false !== $user_id )
-               return get_userdata($user_id);
-
-       if ( !$user = $wpdb->get_row( $wpdb->prepare("SELECT * FROM $wpdb->users WHERE $field = %s", $value) ) )
-               return false;
-
-       _fill_user($user);
-
-       return $user;
-}
-endif;
-
-if ( !function_exists('get_userdatabylogin') ) :
-/**
- * Retrieve user info by login name.
- *
- * @since 0.71
- *
- * @param string $user_login User's username
- * @return bool|object False on failure, User DB row object
- */
-function get_userdatabylogin($user_login) {
-       return get_user_by('login', $user_login);
-}
-endif;
-
-if ( !function_exists('get_user_by_email') ) :
-/**
- * Retrieve user info by email.
- *
- * @since 2.5
- *
- * @param string $email User's email address
- * @return bool|object False on failure, User DB row object
- */
-function get_user_by_email($email) {
-       return get_user_by('email', $email);
+       update_meta_cache( 'user', $ids );
 }
 endif;
 
@@ -259,7 +203,6 @@ if ( !function_exists( 'wp_mail' ) ) :
  * @uses do_action_ref_array() Calls 'phpmailer_init' hook on the reference to
  *             phpmailer object.
  * @uses PHPMailer
- * @
  *
  * @param string|array $to Array or comma-separated list of email addresses to send message.
  * @param string $subject Email subject
@@ -281,7 +224,7 @@ function wp_mail( $to, $subject, $message, $headers = '', $attachments = array()
        if ( !is_object( $phpmailer ) || !is_a( $phpmailer, 'PHPMailer' ) ) {
                require_once ABSPATH . WPINC . '/class-phpmailer.php';
                require_once ABSPATH . WPINC . '/class-smtp.php';
-               $phpmailer = new PHPMailer();
+               $phpmailer = new PHPMailer( true );
        }
 
        // Headers
@@ -296,6 +239,8 @@ function wp_mail( $to, $subject, $message, $headers = '', $attachments = array()
                        $tempheaders = $headers;
                }
                $headers = array();
+               $cc = array();
+               $bcc = array();
 
                // If it's actually got contents
                if ( !empty( $tempheaders ) ) {
@@ -400,7 +345,19 @@ function wp_mail( $to, $subject, $message, $headers = '', $attachments = array()
                $to = explode( ',', $to );
 
        foreach ( (array) $to as $recipient ) {
-               $phpmailer->AddAddress( trim( $recipient ) );
+               try {
+                       // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
+                       $recipient_name = '';
+                       if( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
+                               if ( count( $matches ) == 3 ) {
+                                       $recipient_name = $matches[1];
+                                       $recipient = $matches[2];
+                               }
+                       }
+                       $phpmailer->AddAddress( $recipient, $recipient_name);
+               } catch ( phpmailerException $e ) {
+                       continue;
+               }
        }
 
        // Set mail's subject and body
@@ -410,13 +367,37 @@ function wp_mail( $to, $subject, $message, $headers = '', $attachments = array()
        // Add any CC and BCC recipients
        if ( !empty( $cc ) ) {
                foreach ( (array) $cc as $recipient ) {
-                       $phpmailer->AddCc( trim($recipient) );
+                       try {
+                               // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
+                               $recipient_name = '';
+                               if( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
+                                       if ( count( $matches ) == 3 ) {
+                                               $recipient_name = $matches[1];
+                                               $recipient = $matches[2];
+                                       }
+                               }
+                               $phpmailer->AddCc( $recipient, $recipient_name );
+                       } catch ( phpmailerException $e ) {
+                               continue;
+                       }
                }
        }
 
        if ( !empty( $bcc ) ) {
                foreach ( (array) $bcc as $recipient) {
-                       $phpmailer->AddBcc( trim($recipient) );
+                       try {
+                               // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
+                               $recipient_name = '';
+                               if( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
+                                       if ( count( $matches ) == 3 ) {
+                                               $recipient_name = $matches[1];
+                                               $recipient = $matches[2];
+                                       }
+                               }
+                               $phpmailer->AddBcc( $recipient, $recipient_name );
+                       } catch ( phpmailerException $e ) {
+                               continue;
+                       }
                }
        }
 
@@ -455,16 +436,24 @@ function wp_mail( $to, $subject, $message, $headers = '', $attachments = array()
 
        if ( !empty( $attachments ) ) {
                foreach ( $attachments as $attachment ) {
-                       $phpmailer->AddAttachment($attachment);
+                       try {
+                               $phpmailer->AddAttachment($attachment);
+                       } catch ( phpmailerException $e ) {
+                               continue;
+                       }
                }
        }
 
        do_action_ref_array( 'phpmailer_init', array( &$phpmailer ) );
 
        // Send!
-       $result = @$phpmailer->Send();
+       try {
+               $phpmailer->Send();
+       } catch ( phpmailerException $e ) {
+               return false;
+       }
 
-       return $result;
+       return true;
 }
 endif;
 
@@ -540,7 +529,7 @@ function wp_validate_auth_cookie($cookie = '', $scheme = '') {
 
        // Allow a grace period for POST and AJAX requests
        if ( defined('DOING_AJAX') || 'POST' == $_SERVER['REQUEST_METHOD'] )
-               $expired += 3600;
+               $expired += HOUR_IN_SECONDS;
 
        // Quick check to see if an honest cookie has expired
        if ( $expired < time() ) {
@@ -548,7 +537,7 @@ function wp_validate_auth_cookie($cookie = '', $scheme = '') {
                return false;
        }
 
-       $user = get_userdatabylogin($username);
+       $user = get_user_by('login', $username);
        if ( ! $user ) {
                do_action('auth_cookie_bad_username', $cookie_elements);
                return false;
@@ -671,6 +660,9 @@ function wp_set_auth_cookie($user_id, $remember = false, $secure = '') {
        if ( '' === $secure )
                $secure = is_ssl();
 
+       $secure = apply_filters('secure_auth_cookie', $secure, $user_id);
+       $secure_logged_in_cookie = apply_filters('secure_logged_in_cookie', false, $user_id, $secure);
+
        if ( $secure ) {
                $auth_cookie_name = SECURE_AUTH_COOKIE;
                $scheme = 'secure_auth';
@@ -685,23 +677,11 @@ function wp_set_auth_cookie($user_id, $remember = false, $secure = '') {
        do_action('set_auth_cookie', $auth_cookie, $expire, $expiration, $user_id, $scheme);
        do_action('set_logged_in_cookie', $logged_in_cookie, $expire, $expiration, $user_id, 'logged_in');
 
-       // Set httponly if the php version is >= 5.2.0
-       if ( version_compare(phpversion(), '5.2.0', 'ge') ) {
-               setcookie($auth_cookie_name, $auth_cookie, $expire, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN, $secure, true);
-               setcookie($auth_cookie_name, $auth_cookie, $expire, ADMIN_COOKIE_PATH, COOKIE_DOMAIN, $secure, true);
-               setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, COOKIEPATH, COOKIE_DOMAIN, false, true);
-               if ( COOKIEPATH != SITECOOKIEPATH )
-                       setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, SITECOOKIEPATH, COOKIE_DOMAIN, false, true);
-       } else {
-               $cookie_domain = COOKIE_DOMAIN;
-               if ( !empty($cookie_domain) )
-                       $cookie_domain .= '; HttpOnly';
-               setcookie($auth_cookie_name, $auth_cookie, $expire, PLUGINS_COOKIE_PATH, $cookie_domain, $secure);
-               setcookie($auth_cookie_name, $auth_cookie, $expire, ADMIN_COOKIE_PATH, $cookie_domain, $secure);
-               setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, COOKIEPATH, $cookie_domain);
-               if ( COOKIEPATH != SITECOOKIEPATH )
-                       setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, SITECOOKIEPATH, $cookie_domain);
-       }
+       setcookie($auth_cookie_name, $auth_cookie, $expire, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN, $secure, true);
+       setcookie($auth_cookie_name, $auth_cookie, $expire, ADMIN_COOKIE_PATH, COOKIE_DOMAIN, $secure, true);
+       setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, COOKIEPATH, COOKIE_DOMAIN, $secure_logged_in_cookie, true);
+       if ( COOKIEPATH != SITECOOKIEPATH )
+               setcookie(LOGGED_IN_COOKIE, $logged_in_cookie, $expire, SITECOOKIEPATH, COOKIE_DOMAIN, $secure_logged_in_cookie, true);
 }
 endif;
 
@@ -714,24 +694,24 @@ if ( !function_exists('wp_clear_auth_cookie') ) :
 function wp_clear_auth_cookie() {
        do_action('clear_auth_cookie');
 
-       setcookie(AUTH_COOKIE, ' ', time() - 31536000, ADMIN_COOKIE_PATH, COOKIE_DOMAIN);
-       setcookie(SECURE_AUTH_COOKIE, ' ', time() - 31536000, ADMIN_COOKIE_PATH, COOKIE_DOMAIN);
-       setcookie(AUTH_COOKIE, ' ', time() - 31536000, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN);
-       setcookie(SECURE_AUTH_COOKIE, ' ', time() - 31536000, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN);
-       setcookie(LOGGED_IN_COOKIE, ' ', time() - 31536000, COOKIEPATH, COOKIE_DOMAIN);
-       setcookie(LOGGED_IN_COOKIE, ' ', time() - 31536000, SITECOOKIEPATH, COOKIE_DOMAIN);
+       setcookie( AUTH_COOKIE,        ' ', time() - YEAR_IN_SECONDS, ADMIN_COOKIE_PATH,   COOKIE_DOMAIN );
+       setcookie( SECURE_AUTH_COOKIE, ' ', time() - YEAR_IN_SECONDS, ADMIN_COOKIE_PATH,   COOKIE_DOMAIN );
+       setcookie( AUTH_COOKIE,        ' ', time() - YEAR_IN_SECONDS, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN );
+       setcookie( SECURE_AUTH_COOKIE, ' ', time() - YEAR_IN_SECONDS, PLUGINS_COOKIE_PATH, COOKIE_DOMAIN );
+       setcookie( LOGGED_IN_COOKIE,   ' ', time() - YEAR_IN_SECONDS, COOKIEPATH,          COOKIE_DOMAIN );
+       setcookie( LOGGED_IN_COOKIE,   ' ', time() - YEAR_IN_SECONDS, SITECOOKIEPATH,      COOKIE_DOMAIN );
 
        // Old cookies
-       setcookie(AUTH_COOKIE, ' ', time() - 31536000, COOKIEPATH, COOKIE_DOMAIN);
-       setcookie(AUTH_COOKIE, ' ', time() - 31536000, SITECOOKIEPATH, COOKIE_DOMAIN);
-       setcookie(SECURE_AUTH_COOKIE, ' ', time() - 31536000, COOKIEPATH, COOKIE_DOMAIN);
-       setcookie(SECURE_AUTH_COOKIE, ' ', time() - 31536000, SITECOOKIEPATH, COOKIE_DOMAIN);
+       setcookie( AUTH_COOKIE,        ' ', time() - YEAR_IN_SECONDS, COOKIEPATH,     COOKIE_DOMAIN );
+       setcookie( AUTH_COOKIE,        ' ', time() - YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN );
+       setcookie( SECURE_AUTH_COOKIE, ' ', time() - YEAR_IN_SECONDS, COOKIEPATH,     COOKIE_DOMAIN );
+       setcookie( SECURE_AUTH_COOKIE, ' ', time() - YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN );
 
        // Even older cookies
-       setcookie(USER_COOKIE, ' ', time() - 31536000, COOKIEPATH, COOKIE_DOMAIN);
-       setcookie(PASS_COOKIE, ' ', time() - 31536000, COOKIEPATH, COOKIE_DOMAIN);
-       setcookie(USER_COOKIE, ' ', time() - 31536000, SITECOOKIEPATH, COOKIE_DOMAIN);
-       setcookie(PASS_COOKIE, ' ', time() - 31536000, SITECOOKIEPATH, COOKIE_DOMAIN);
+       setcookie( USER_COOKIE, ' ', time() - YEAR_IN_SECONDS, COOKIEPATH,     COOKIE_DOMAIN );
+       setcookie( PASS_COOKIE, ' ', time() - YEAR_IN_SECONDS, COOKIEPATH,     COOKIE_DOMAIN );
+       setcookie( USER_COOKIE, ' ', time() - YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN );
+       setcookie( PASS_COOKIE, ' ', time() - YEAR_IN_SECONDS, SITECOOKIEPATH, COOKIE_DOMAIN );
 }
 endif;
 
@@ -746,7 +726,7 @@ if ( !function_exists('is_user_logged_in') ) :
 function is_user_logged_in() {
        $user = wp_get_current_user();
 
-       if ( $user->id == 0 )
+       if ( ! $user->exists() )
                return false;
 
        return true;
@@ -764,27 +744,34 @@ function auth_redirect() {
 
        $secure = ( is_ssl() || force_ssl_admin() );
 
+       $secure = apply_filters('secure_auth_redirect', $secure);
+
        // If https is required and request is http, redirect
        if ( $secure && !is_ssl() && false !== strpos($_SERVER['REQUEST_URI'], 'wp-admin') ) {
-               if ( 0 === strpos($_SERVER['REQUEST_URI'], 'http') ) {
-                       wp_redirect(preg_replace('|^http://|', 'https://', $_SERVER['REQUEST_URI']));
+               if ( 0 === strpos( $_SERVER['REQUEST_URI'], 'http' ) ) {
+                       wp_redirect( set_url_scheme( $_SERVER['REQUEST_URI'], 'https' ) );
                        exit();
                } else {
-                       wp_redirect('https://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI']);
+                       wp_redirect( 'https://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'] );
                        exit();
                }
        }
 
-       if ( $user_id = wp_validate_auth_cookie( '', apply_filters( 'auth_redirect_scheme', '' ) ) ) {
+       if ( is_user_admin() )
+               $scheme = 'logged_in';
+       else
+               $scheme = apply_filters( 'auth_redirect_scheme', '' );
+
+       if ( $user_id = wp_validate_auth_cookie( '',  $scheme) ) {
                do_action('auth_redirect', $user_id);
 
                // If the user wants ssl but the session is not ssl, redirect.
                if ( !$secure && get_user_option('use_ssl', $user_id) && false !== strpos($_SERVER['REQUEST_URI'], 'wp-admin') ) {
-                       if ( 0 === strpos($_SERVER['REQUEST_URI'], 'http') ) {
-                               wp_redirect(preg_replace('|^http://|', 'https://', $_SERVER['REQUEST_URI']));
+                       if ( 0 === strpos( $_SERVER['REQUEST_URI'], 'http' ) ) {
+                               wp_redirect( set_url_scheme( $_SERVER['REQUEST_URI'], 'https' ) );
                                exit();
                        } else {
-                               wp_redirect('https://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI']);
+                               wp_redirect( 'https://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'] );
                                exit();
                        }
                }
@@ -795,12 +782,7 @@ function auth_redirect() {
        // The cookie is no good so force login
        nocache_headers();
 
-       if ( is_ssl() )
-               $proto = 'https://';
-       else
-               $proto = 'http://';
-
-       $redirect = ( strpos($_SERVER['REQUEST_URI'], '/options.php') && wp_get_referer() ) ? wp_get_referer() : $proto . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
+       $redirect = ( strpos( $_SERVER['REQUEST_URI'], '/options.php' ) && wp_get_referer() ) ? wp_get_referer() : set_url_scheme( 'http://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'] );
 
        $login_url = wp_login_url($redirect, true);
 
@@ -822,6 +804,9 @@ if ( !function_exists('check_admin_referer') ) :
  * @param string $query_arg where to look for nonce in $_REQUEST (since 2.5)
  */
 function check_admin_referer($action = -1, $query_arg = '_wpnonce') {
+       if ( -1 == $action )
+               _doing_it_wrong( __FUNCTION__, __( 'You should specify a nonce action to be verified by using the first parameter.' ), '3.2' );
+
        $adminurl = strtolower(admin_url());
        $referer = strtolower(wp_get_referer());
        $result = isset($_REQUEST[$query_arg]) ? wp_verify_nonce($_REQUEST[$query_arg], $action) : false;
@@ -850,8 +835,12 @@ function check_ajax_referer( $action = -1, $query_arg = false, $die = true ) {
 
        $result = wp_verify_nonce( $nonce, $action );
 
-       if ( $die && false == $result )
-               die('-1');
+       if ( $die && false == $result ) {
+               if ( defined( 'DOING_AJAX' ) && DOING_AJAX )
+                       wp_die( -1 );
+               else
+                       die( '-1' );
+       }
 
        do_action('check_ajax_referer', $action, $result);
 
@@ -861,9 +850,8 @@ endif;
 
 if ( !function_exists('wp_redirect') ) :
 /**
- * Redirects to another page, with a workaround for the IIS Set-Cookie bug.
+ * Redirects to another page.
  *
- * @link http://support.microsoft.com/kb/q176113/
  * @since 1.5.1
  * @uses apply_filters() Calls 'wp_redirect' hook on $location and $status.
  *
@@ -882,13 +870,10 @@ function wp_redirect($location, $status = 302) {
 
        $location = wp_sanitize_redirect($location);
 
-       if ( $is_IIS ) {
-               header("Refresh: 0;url=$location");
-       } else {
-               if ( php_sapi_name() != 'cgi-fcgi' )
-                       status_header($status); // This causes problems on IIS and some FastCGI setups
-               header("Location: $location", true, $status);
-       }
+       if ( !$is_IIS && php_sapi_name() != 'cgi-fcgi' )
+               status_header($status); // This causes problems on IIS and some FastCGI setups
+
+       header("Location: $location", true, $status);
 }
 endif;
 
@@ -954,7 +939,7 @@ if ( !function_exists('wp_validate_redirect') ) :
  *             WordPress host string and $location host string.
  *
  * @param string $location The redirect to validate
- * @param string $default The value to return is $location is not allowed
+ * @param string $default The value to return if $location is not allowed
  * @return string redirect-sanitized URL
  **/
 function wp_validate_redirect($location, $default = '') {
@@ -1000,14 +985,22 @@ if ( ! function_exists('wp_notify_postauthor') ) :
  * @param string $comment_type Optional. The comment type either 'comment' (default), 'trackback', or 'pingback'
  * @return bool False if user email does not exist. True on completion.
  */
-function wp_notify_postauthor($comment_id, $comment_type='') {
-       $comment = get_comment($comment_id);
-       $post    = get_post($comment->comment_post_ID);
-       $user    = get_userdata( $post->post_author );
+function wp_notify_postauthor( $comment_id, $comment_type = '' ) {
+       $comment = get_comment( $comment_id );
+       $post    = get_post( $comment->comment_post_ID );
+       $author  = get_userdata( $post->post_author );
 
-       if ( $comment->user_id == $post->post_author ) return false; // The author moderated a comment on his own post
+       // The comment was left by the author
+       if ( $comment->user_id == $post->post_author )
+               return false;
 
-       if ('' == $user->user_email) return false; // If there's no email to send the comment to
+       // The author moderated a comment on his own post
+       if ( $post->post_author == get_current_user_id() )
+               return false;
+
+       // If there's no email to send the comment to
+       if ( '' == $author->user_email )
+               return false;
 
        $comment_author_domain = @gethostbyaddr($comment->comment_author_IP);
 
@@ -1023,7 +1016,7 @@ function wp_notify_postauthor($comment_id, $comment_type='') {
                $notify_message .= sprintf( __('Author : %1$s (IP: %2$s , %3$s)'), $comment->comment_author, $comment->comment_author_IP, $comment_author_domain ) . "\r\n";
                $notify_message .= sprintf( __('E-mail : %s'), $comment->comment_author_email ) . "\r\n";
                $notify_message .= sprintf( __('URL    : %s'), $comment->comment_author_url ) . "\r\n";
-               $notify_message .= sprintf( __('Whois  : http://ws.arin.net/cgi-bin/whois.pl?queryinput=%s'), $comment->comment_author_IP ) . "\r\n";
+               $notify_message .= sprintf( __('Whois  : http://whois.arin.net/rest/ip/%s'), $comment->comment_author_IP ) . "\r\n";
                $notify_message .= __('Comment: ') . "\r\n" . $comment->comment_content . "\r\n\r\n";
                $notify_message .= __('You can see all comments on this post here: ') . "\r\n";
                /* translators: 1: blog name, 2: post title */
@@ -1048,6 +1041,7 @@ function wp_notify_postauthor($comment_id, $comment_type='') {
                $subject = sprintf( __('[%1$s] Pingback: "%2$s"'), $blogname, $post->post_title );
        }
        $notify_message .= get_permalink($comment->comment_post_ID) . "#comments\r\n\r\n";
+       $notify_message .= sprintf( __('Permalink: %s'), get_permalink( $comment->comment_post_ID ) . '#comment-' . $comment_id ) . "\r\n";
        if ( EMPTY_TRASH_DAYS )
                $notify_message .= sprintf( __('Trash it: %s'), admin_url("comment.php?action=trash&c=$comment_id") ) . "\r\n";
        else
@@ -1076,7 +1070,7 @@ function wp_notify_postauthor($comment_id, $comment_type='') {
        $subject = apply_filters('comment_notification_subject', $subject, $comment_id);
        $message_headers = apply_filters('comment_notification_headers', $message_headers, $comment_id);
 
-       @wp_mail($user->user_email, $subject, $notify_message, $message_headers);
+       @wp_mail( $author->user_email, $subject, $notify_message, $message_headers );
 
        return true;
 }
@@ -1095,11 +1089,16 @@ if ( !function_exists('wp_notify_moderator') ) :
 function wp_notify_moderator($comment_id) {
        global $wpdb;
 
-       if( get_option( "moderation_notify" ) == 0 )
+       if ( 0 == get_option( 'moderation_notify' ) )
                return true;
 
-       $comment = $wpdb->get_row($wpdb->prepare("SELECT * FROM $wpdb->comments WHERE comment_ID=%d LIMIT 1", $comment_id));
-       $post = $wpdb->get_row($wpdb->prepare("SELECT * FROM $wpdb->posts WHERE ID=%d LIMIT 1", $comment->comment_post_ID));
+       $comment = get_comment($comment_id);
+       $post = get_post($comment->comment_post_ID);
+       $user = get_userdata( $post->post_author );
+       // Send to the administration and to the post author if the author can modify the comment.
+       $email_to = array( get_option('admin_email') );
+       if ( user_can($user->ID, 'edit_comment', $comment_id) && !empty($user->user_email) && ( get_option('admin_email') != $user->user_email) )
+               $email_to[] = $user->user_email;
 
        $comment_author_domain = @gethostbyaddr($comment->comment_author_IP);
        $comments_waiting = $wpdb->get_var("SELECT count(comment_ID) FROM $wpdb->comments WHERE comment_approved = '0'");
@@ -1130,7 +1129,7 @@ function wp_notify_moderator($comment_id) {
                        $notify_message .= sprintf( __('Author : %1$s (IP: %2$s , %3$s)'), $comment->comment_author, $comment->comment_author_IP, $comment_author_domain ) . "\r\n";
                        $notify_message .= sprintf( __('E-mail : %s'), $comment->comment_author_email ) . "\r\n";
                        $notify_message .= sprintf( __('URL    : %s'), $comment->comment_author_url ) . "\r\n";
-                       $notify_message .= sprintf( __('Whois  : http://ws.arin.net/cgi-bin/whois.pl?queryinput=%s'), $comment->comment_author_IP ) . "\r\n";
+                       $notify_message .= sprintf( __('Whois  : http://whois.arin.net/rest/ip/%s'), $comment->comment_author_IP ) . "\r\n";
                        $notify_message .= __('Comment: ') . "\r\n" . $comment->comment_content . "\r\n\r\n";
                        break;
        }
@@ -1147,14 +1146,14 @@ function wp_notify_moderator($comment_id) {
        $notify_message .= admin_url("edit-comments.php?comment_status=moderated") . "\r\n";
 
        $subject = sprintf( __('[%1$s] Please moderate: "%2$s"'), $blogname, $post->post_title );
-       $admin_email = get_option('admin_email');
        $message_headers = '';
 
        $notify_message = apply_filters('comment_moderation_text', $notify_message, $comment_id);
        $subject = apply_filters('comment_moderation_subject', $subject, $comment_id);
        $message_headers = apply_filters('comment_moderation_headers', $message_headers);
 
-       @wp_mail($admin_email, $subject, $notify_message, $message_headers);
+       foreach ( $email_to as $email )
+               @wp_mail($email, $subject, $notify_message, $message_headers);
 
        return true;
 }
@@ -1191,7 +1190,7 @@ if ( !function_exists('wp_new_user_notification') ) :
  * @param string $plaintext_pass Optional. The user's plaintext password
  */
 function wp_new_user_notification($user_id, $plaintext_pass = '') {
-       $user = new WP_User($user_id);
+       $user = get_userdata( $user_id );
 
        $user_login = stripslashes($user->user_login);
        $user_email = stripslashes($user->user_email);
@@ -1230,7 +1229,7 @@ if ( !function_exists('wp_nonce_tick') ) :
  * @return int
  */
 function wp_nonce_tick() {
-       $nonce_life = apply_filters('nonce_life', 86400);
+       $nonce_life = apply_filters( 'nonce_life', DAY_IN_SECONDS );
 
        return ceil(time() / ( $nonce_life / 2 ));
 }
@@ -1251,15 +1250,17 @@ if ( !function_exists('wp_verify_nonce') ) :
  */
 function wp_verify_nonce($nonce, $action = -1) {
        $user = wp_get_current_user();
-       $uid = (int) $user->id;
+       $uid = (int) $user->ID;
+       if ( ! $uid )
+               $uid = apply_filters( 'nonce_user_logged_out', $uid, $action );
 
        $i = wp_nonce_tick();
 
        // Nonce generated 0-12 hours ago
-       if ( substr(wp_hash($i . $action . $uid, 'nonce'), -12, 10) == $nonce )
+       if ( substr(wp_hash($i . $action . $uid, 'nonce'), -12, 10) === $nonce )
                return 1;
        // Nonce generated 12-24 hours ago
-       if ( substr(wp_hash(($i - 1) . $action . $uid, 'nonce'), -12, 10) == $nonce )
+       if ( substr(wp_hash(($i - 1) . $action . $uid, 'nonce'), -12, 10) === $nonce )
                return 2;
        // Invalid nonce
        return false;
@@ -1277,7 +1278,9 @@ if ( !function_exists('wp_create_nonce') ) :
  */
 function wp_create_nonce($action = -1) {
        $user = wp_get_current_user();
-       $uid = (int) $user->id;
+       $uid = (int) $user->ID;
+       if ( ! $uid )
+               $uid = apply_filters( 'nonce_user_logged_out', $uid, $action );
 
        $i = wp_nonce_tick();
 
@@ -1287,108 +1290,89 @@ endif;
 
 if ( !function_exists('wp_salt') ) :
 /**
- * Get salt to add to hashes to help prevent attacks.
- *
- * The secret key is located in two places: the database in case the secret key
- * isn't defined in the second place, which is in the wp-config.php file. If you
- * are going to set the secret key, then you must do so in the wp-config.php
- * file.
+ * Get salt to add to hashes.
  *
- * The secret key in the database is randomly generated and will be appended to
- * the secret key that is in wp-config.php file in some instances. It is
- * important to have the secret key defined or changed in wp-config.php.
+ * Salts are created using secret keys. Secret keys are located in two places:
+ * in the database and in the wp-config.php file. The secret key in the database
+ * is randomly generated and will be appended to the secret keys in wp-config.php.
  *
- * If you have installed WordPress 2.5 or later, then you will have the
- * SECRET_KEY defined in the wp-config.php already. You will want to change the
- * value in it because hackers will know what it is. If you have upgraded to
- * WordPress 2.5 or later version from a version before WordPress 2.5, then you
- * should add the constant to your wp-config.php file.
- *
- * Below is an example of how the SECRET_KEY constant is defined with a value.
- * You must not copy the below example and paste into your wp-config.php. If you
- * need an example, then you can have a
- * {@link https://api.wordpress.org/secret-key/1.1/ secret key created} for you.
+ * The secret keys in wp-config.php should be updated to strong, random keys to maximize
+ * security. Below is an example of how the secret key constants are defined.
+ * Do not paste this example directly into wp-config.php. Instead, have a
+ * {@link https://api.wordpress.org/secret-key/1.1/salt/ secret key created} just
+ * for you.
  *
  * <code>
- * define('SECRET_KEY', 'mAry1HadA15|\/|b17w55w1t3asSn09w');
+ * define('AUTH_KEY',         ' Xakm<o xQy rw4EMsLKM-?!T+,PFF})H4lzcW57AF0U@N@< >M%G4Yt>f`z]MON');
+ * define('SECURE_AUTH_KEY',  'LzJ}op]mr|6+![P}Ak:uNdJCJZd>(Hx.-Mh#Tz)pCIU#uGEnfFz|f ;;eU%/U^O~');
+ * define('LOGGED_IN_KEY',    '|i|Ux`9<p-h$aFf(qnT:sDO:D1P^wZ$$/Ra@miTJi9G;ddp_<q}6H1)o|a +&JCM');
+ * define('NONCE_KEY',        '%:R{[P|,s.KuMltH5}cI;/k<Gx~j!f0I)m_sIyu+&NJZ)-iO>z7X>QYR0Z_XnZ@|');
+ * define('AUTH_SALT',        'eZyT)-Naw]F8CwA*VaW#q*|.)g@o}||wf~@C-YSt}(dh_r6EbI#A,y|nU2{B#JBW');
+ * define('SECURE_AUTH_SALT', '!=oLUTXh,QW=H `}`L|9/^4-3 STz},T(w}W<I`.JjPi)<Bmf1v,HpGe}T1:Xt7n');
+ * define('LOGGED_IN_SALT',   '+XSqHc;@Q*K_b|Z?NC[3H!!EONbh.n<+=uKR:>*c(u`g~EJBf#8u#R{mUEZrozmm');
+ * define('NONCE_SALT',       'h`GXHhD>SLWVfg1(1(N{;.V!MoE(SfbA_ksP@&`+AycHcAV$+?@3q+rxV{%^VyKT');
  * </code>
  *
  * Salting passwords helps against tools which has stored hashed values of
- * common dictionary strings. The added values makes it harder to crack if given
- * salt string is not weak.
+ * common dictionary strings. The added values makes it harder to crack.
  *
  * @since 2.5
- * @link https://api.wordpress.org/secret-key/1.1/ Create a Secret Key for wp-config.php
  *
- * @param string $scheme Authentication scheme
+ * @link https://api.wordpress.org/secret-key/1.1/salt/ Create secrets for wp-config.php
+ *
+ * @param string $scheme Authentication scheme (auth, secure_auth, logged_in, nonce)
  * @return string Salt value
  */
-function wp_salt($scheme = 'auth') {
-       global $wp_default_secret_key;
-       $secret_key = '';
-       if ( defined('SECRET_KEY') && ('' != SECRET_KEY) && ( $wp_default_secret_key != SECRET_KEY) )
-               $secret_key = SECRET_KEY;
-
-       if ( 'auth' == $scheme ) {
-               if ( defined('AUTH_KEY') && ('' != AUTH_KEY) && ( $wp_default_secret_key != AUTH_KEY) )
-                       $secret_key = AUTH_KEY;
-
-               if ( defined('AUTH_SALT') && ('' != AUTH_SALT) && ( $wp_default_secret_key != AUTH_SALT) ) {
-                       $salt = AUTH_SALT;
-               } elseif ( defined('SECRET_SALT') && ('' != SECRET_SALT) && ( $wp_default_secret_key != SECRET_SALT) ) {
-                       $salt = SECRET_SALT;
-               } else {
-                       $salt = get_site_option('auth_salt');
-                       if ( empty($salt) ) {
-                               $salt = wp_generate_password( 64, true, true );
-                               update_site_option('auth_salt', $salt);
-                       }
-               }
-       } elseif ( 'secure_auth' == $scheme ) {
-               if ( defined('SECURE_AUTH_KEY') && ('' != SECURE_AUTH_KEY) && ( $wp_default_secret_key != SECURE_AUTH_KEY) )
-                       $secret_key = SECURE_AUTH_KEY;
-
-               if ( defined('SECURE_AUTH_SALT') && ('' != SECURE_AUTH_SALT) && ( $wp_default_secret_key != SECURE_AUTH_SALT) ) {
-                       $salt = SECURE_AUTH_SALT;
-               } else {
-                       $salt = get_site_option('secure_auth_salt');
-                       if ( empty($salt) ) {
-                               $salt = wp_generate_password( 64, true, true );
-                               update_site_option('secure_auth_salt', $salt);
+function wp_salt( $scheme = 'auth' ) {
+       static $cached_salts = array();
+       if ( isset( $cached_salts[ $scheme ] ) )
+               return apply_filters( 'salt', $cached_salts[ $scheme ], $scheme );
+
+       static $duplicated_keys;
+       if ( null === $duplicated_keys ) {
+               $duplicated_keys = array( 'put your unique phrase here' => true );
+               foreach ( array( 'AUTH', 'SECURE_AUTH', 'LOGGED_IN', 'NONCE', 'SECRET' ) as $first ) {
+                       foreach ( array( 'KEY', 'SALT' ) as $second ) {
+                               if ( ! defined( "{$first}_{$second}" ) )
+                                       continue;
+                               $value = constant( "{$first}_{$second}" );
+                               $duplicated_keys[ $value ] = isset( $duplicated_keys[ $value ] );
                        }
                }
-       } elseif ( 'logged_in' == $scheme ) {
-               if ( defined('LOGGED_IN_KEY') && ('' != LOGGED_IN_KEY) && ( $wp_default_secret_key != LOGGED_IN_KEY) )
-                       $secret_key = LOGGED_IN_KEY;
+       }
 
-               if ( defined('LOGGED_IN_SALT') && ('' != LOGGED_IN_SALT) && ( $wp_default_secret_key != LOGGED_IN_SALT) ) {
-                       $salt = LOGGED_IN_SALT;
-               } else {
-                       $salt = get_site_option('logged_in_salt');
-                       if ( empty($salt) ) {
-                               $salt = wp_generate_password( 64, true, true );
-                               update_site_option('logged_in_salt', $salt);
+       $key = $salt = '';
+       if ( defined( 'SECRET_KEY' ) && SECRET_KEY && empty( $duplicated_keys[ SECRET_KEY ] ) )
+               $key = SECRET_KEY;
+       if ( 'auth' == $scheme && defined( 'SECRET_SALT' ) && SECRET_SALT && empty( $duplicated_keys[ SECRET_SALT ] ) )
+               $salt = SECRET_SALT;
+
+       if ( in_array( $scheme, array( 'auth', 'secure_auth', 'logged_in', 'nonce' ) ) ) {
+               foreach ( array( 'key', 'salt' ) as $type ) {
+                       $const = strtoupper( "{$scheme}_{$type}" );
+                       if ( defined( $const ) && constant( $const ) && empty( $duplicated_keys[ constant( $const ) ] ) ) {
+                               $$type = constant( $const );
+                       } elseif ( ! $$type ) {
+                               $$type = get_site_option( "{$scheme}_{$type}" );
+                               if ( ! $$type ) {
+                                       $$type = wp_generate_password( 64, true, true );
+                                       update_site_option( "{$scheme}_{$type}", $$type );
+                               }
                        }
                }
-       } elseif ( 'nonce' == $scheme ) {
-               if ( defined('NONCE_KEY') && ('' != NONCE_KEY) && ( $wp_default_secret_key != NONCE_KEY) )
-                       $secret_key = NONCE_KEY;
-
-               if ( defined('NONCE_SALT') && ('' != NONCE_SALT) && ( $wp_default_secret_key != NONCE_SALT) ) {
-                       $salt = NONCE_SALT;
-               } else {
-                       $salt = get_site_option('nonce_salt');
-                       if ( empty($salt) ) {
-                               $salt = wp_generate_password( 64, true, true );
-                               update_site_option('nonce_salt', $salt);
+       } else {
+               if ( ! $key ) {
+                       $key = get_site_option( 'secret_key' );
+                       if ( ! $key ) {
+                               $key = wp_generate_password( 64, true, true );
+                               update_site_option( 'secret_key', $key );
                        }
                }
-       } else {
-               // ensure each auth scheme has its own unique salt
-               $salt = hash_hmac('md5', $scheme, $secret_key);
+               $salt = hash_hmac( 'md5', $scheme, $key );
        }
 
-       return apply_filters('salt', $secret_key . $salt, $scheme);
+       $cached_salts[ $scheme ] = $key . $salt;
+       return apply_filters( 'salt', $cached_salts[ $scheme ], $scheme );
 }
 endif;
 
@@ -1429,7 +1413,7 @@ function wp_hash_password($password) {
        if ( empty($wp_hasher) ) {
                require_once( ABSPATH . 'wp-includes/class-phpass.php');
                // By default, use the portable hash from phpass
-               $wp_hasher = new PasswordHash(8, TRUE);
+               $wp_hasher = new PasswordHash(8, true);
        }
 
        return $wp_hasher->HashPassword($password);
@@ -1442,7 +1426,7 @@ if ( !function_exists('wp_check_password') ) :
  *
  * Maintains compatibility between old version and the new cookie authentication
  * protocol using PHPass library. The $hash parameter is the encrypted password
- * and the function compares the plain text password when encypted similarly
+ * and the function compares the plain text password when encrypted similarly
  * against the already encrypted password to see if they match.
  *
  * For integration with other applications, this function can be overwritten to
@@ -1477,7 +1461,7 @@ function wp_check_password($password, $hash, $user_id = '') {
        if ( empty($wp_hasher) ) {
                require_once( ABSPATH . 'wp-includes/class-phpass.php');
                // By default, use the portable hash from phpass
-               $wp_hasher = new PasswordHash(8, TRUE);
+               $wp_hasher = new PasswordHash(8, true);
        }
 
        $check = $wp_hasher->CheckPassword($password, $hash);
@@ -1516,13 +1500,13 @@ function wp_generate_password( $length = 12, $special_chars = true, $extra_speci
 endif;
 
 if ( !function_exists('wp_rand') ) :
- /**
+/**
  * Generates a random number
  *
  * @since 2.6.2
  *
- * @param int $min Lower limit for the generated number (optional, default is 0)
- * @param int $max Upper limit for the generated number (optional, default is 4294967295)
+ * @param int $min Lower limit for the generated number
+ * @param int $max Upper limit for the generated number
  * @return int A random number between min and max
  */
 function wp_rand( $min = 0, $max = 0 ) {
@@ -1551,10 +1535,12 @@ function wp_rand( $min = 0, $max = 0 ) {
 
        $value = abs(hexdec($value));
 
+       // Some misconfigured 32bit environments (Entropy PHP, for example) truncate integers larger than PHP_INT_MAX to PHP_INT_MAX rather than overflowing them to floats.
+       $max_random_number = 3000000000 === 2147483647 ? (float) "4294967295" : 4294967295; // 4294967295 = 0xffffffff
+
        // Reduce the value to be within the min - max range
-       // 4294967295 = 0xffffffff = max random number
        if ( $max != 0 )
-               $value = $min + (($max - $min + 1) * ($value / (4294967295 + 1)));
+               $value = $min + ( $max - $min + 1 ) * $value / ( $max_random_number + 1 );
 
        return abs(intval($value));
 }
@@ -1592,7 +1578,7 @@ if ( !function_exists( 'get_avatar' ) ) :
  * @param int|string|object $id_or_email A user ID,  email address, or comment object
  * @param int $size Size of the avatar image
  * @param string $default URL to a default image to use if no avatar is available
- * @param string $alt Alternate text to use in image tag. Defaults to blank
+ * @param string $alt Alternative text to use in image tag. Defaults to blank
  * @return string <img> tag for the user's avatar
 */
 function get_avatar( $id_or_email, $size = '96', $default = '', $alt = false ) {
@@ -1640,13 +1626,13 @@ function get_avatar( $id_or_email, $size = '96', $default = '', $alt = false ) {
        }
 
        if ( !empty($email) )
-               $email_hash = md5( strtolower( $email ) );
+               $email_hash = md5( strtolower( trim( $email ) ) );
 
        if ( is_ssl() ) {
                $host = 'https://secure.gravatar.com';
        } else {
                if ( !empty($email) )
-                       $host = sprintf( "http://%d.gravatar.com", ( hexdec( $email_hash{0} ) % 2 ) );
+                       $host = sprintf( "http://%d.gravatar.com", ( hexdec( $email_hash[0] ) % 2 ) );
                else
                        $host = 'http://0.gravatar.com';
        }
@@ -1654,11 +1640,11 @@ function get_avatar( $id_or_email, $size = '96', $default = '', $alt = false ) {
        if ( 'mystery' == $default )
                $default = "$host/avatar/ad516503a11cd5ca435acc9bb6523536?s={$size}"; // ad516503a11cd5ca435acc9bb6523536 == md5('unknown@gravatar.com')
        elseif ( 'blank' == $default )
-               $default = includes_url('images/blank.gif');
+               $default = $email ? 'blank' : includes_url( 'images/blank.gif' );
        elseif ( !empty($email) && 'gravatar_default' == $default )
                $default = '';
        elseif ( 'gravatar_default' == $default )
-               $default = "$host/avatar/s={$size}";
+               $default = "$host/avatar/?s={$size}";
        elseif ( empty($email) )
                $default = "$host/avatar/?d=$default&amp;s={$size}";
        elseif ( strpos($default, 'http://') === 0 )
@@ -1720,8 +1706,8 @@ function wp_text_diff( $left_string, $right_string, $args = null ) {
        $left_string  = normalize_whitespace($left_string);
        $right_string = normalize_whitespace($right_string);
 
-       $left_lines  = split("\n", $left_string);
-       $right_lines = split("\n", $right_string);
+       $left_lines  = explode("\n", $left_string);
+       $right_lines = explode("\n", $right_string);
 
        $text_diff = new Text_Diff($left_lines, $right_lines);
        $renderer  = new WP_Text_Diff_Renderer_Table();