* Avoid `Only variables should be passed by reference` notice by passing a var to `array_pop()` in `tests/user.php`

* Suppress a notice by silencing the passing of a non-existent object prop

See #25282.



git-svn-id: https://develop.svn.wordpress.org/trunk@25390 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Scott Taylor 2013-09-12 05:56:59 +00:00
parent 6acb4cf24d
commit b18d047edd
1 changed files with 3 additions and 2 deletions

View File

@ -130,7 +130,8 @@ class Tests_User extends WP_UnitTestCase {
$this->assertTrue(isset($out[$k]) && $out[$k][0] == $v);
// delete one key and check again
$key_to_delete = array_pop(array_keys($vals));
$keys = array_keys( $vals );
$key_to_delete = array_pop( $keys );
delete_user_meta($user_id, $key_to_delete);
$out = get_user_meta($user_id);
// make sure that key is excluded from the results
@ -572,7 +573,7 @@ class Tests_User extends WP_UnitTestCase {
$user = WP_User::get_data_by( 'id', false );
$this->assertEquals( false, $user );
$user = WP_User::get_data_by( 'id', $user->user_nicename );
$user = WP_User::get_data_by( 'id', @$user->user_nicename );
$this->assertEquals( false, $user );
$user = WP_User::get_data_by( 'id', 99999 );