WordPress.org

Make WordPress Core

Ticket #37746: 37746.diff

File 37746.diff, 1.6 KB (added by peterwilsoncc, 5 years ago)
  • src/wp-includes/meta.php

    diff --git src/wp-includes/meta.php src/wp-includes/meta.php
    index 8833d3e..5a2ac57 100644
    function get_metadata_by_mid( $meta_type, $meta_id ) { 
    571571                return false;
    572572        }
    573573
    574         $meta_id = absint( $meta_id );
    575         if ( ! $meta_id ) {
     574        $meta_id = intval( $meta_id );
     575        if ( $meta_id <= 0 ) {
    576576                return false;
    577577        }
    578578
    function update_metadata_by_mid( $meta_type, $meta_id, $meta_value, $meta_key = 
    615615                return false;
    616616        }
    617617
    618         $meta_id = absint( $meta_id );
    619         if ( ! $meta_id ) {
     618        $meta_id = intval( $meta_id );
     619        if ( $meta_id <= 0 ) {
    620620                return false;
    621621        }
    622622
    function delete_metadata_by_mid( $meta_type, $meta_id ) { 
    706706                return false;
    707707        }
    708708
    709         $meta_id = absint( $meta_id );
    710         if ( ! $meta_id ) {
     709        $meta_id = intval( $meta_id );
     710        if ( $meta_id <= 0 ) {
    711711                return false;
    712712        }
    713713
  • tests/phpunit/tests/meta.php

    diff --git tests/phpunit/tests/meta.php tests/phpunit/tests/meta.php
    index 0676c7e..b7d7ea2 100644
    class Tests_Meta extends WP_UnitTestCase { 
    293293        }
    294294
    295295        /**
     296         * @ticket 37746
     297         */
     298        function test_negative_meta_id() {
     299                $negative_mid = $this->meta_id * -1;
     300
     301                $this->assertTrue( $negative_mid < 0 );
     302                $this->assertFalse( get_metadata_by_mid( 'user', $negative_mid ) );
     303                $this->assertFalse( update_metadata_by_mid( 'user', $negative_mid, 'meta_new_value' ) );
     304                $this->assertFalse( delete_metadata_by_mid( 'user', $negative_mid ) );
     305        }
     306
     307        /**
    296308         * @ticket 15030
    297309         */
    298310        public function test_get_metadata_with_empty_key_array_value() {