WordPress.org
Get WordPress

Make WordPress Core

Ticket #17850: 17850.6.diff

File 17850.6.diff, 5.7 KB (added by ryan, 14 years ago)

All underscore prefixed are protected unless sanitizer is registered

  • wp-includes/post-template.php

     
    735735                echo "<ul class='post-meta'>\n";
    736736                foreach ( (array) $keys as $key ) {
    737737                        $keyt = trim($key);
    738                         if ( '_' == $keyt[0] )
     738                        if ( is_hidden_meta( $keyt ) )
    739739                                continue;
    740740                        $values = array_map('trim', get_post_custom_values($key));
    741741                        $value = implode($values,', ');

  • wp-includes/class-wp-xmlrpc-server.php

     
    234234
    235235                foreach ( (array) has_meta($post_id) as $meta ) {
    236236                        // Don't expose protected fields.
    237                         if ( strpos($meta['meta_key'], '_wp_') === 0 ) {
     237                        if ( is_protected_meta( $meta['meta_key'], 'post' ) ) {
    238238                                continue;
    239239                        }
    240240
    … …  
    264264                                $meta['id'] = (int) $meta['id'];
    265265
    266266                                if ( isset($meta['key']) ) {
    267                                         update_meta($meta['id'], $meta['key'], $meta['value']);
     267                                        if ( ! is_protected_meta( $meta['key'], 'post' ) )
     268                                                update_meta($meta['id'], $meta['key'], $meta['value']);
     269                                } else {
     270                                        if ( ! is_protected_meta( $meta['key'] ) )
     271                                                delete_meta($meta['id']);
    268272                                }
    269                                 else {
    270                                         delete_meta($meta['id']);
    271                                 }
     273                        } else {
     274                                if ( ! is_protected_meta( $meta['key'] ) )
     275                                        add_post_meta( $post_id, $meta['key'], $meta['value'] );
    272276                        }
    273                         else {
    274                                 $_POST['metakeyinput'] = $meta['key'];
    275                                 $_POST['metavalue'] = $meta['value'];
    276                                 add_meta($post_id);
    277                         }
    278277                }
    279278        }
    280279

  • wp-includes/meta.php

     
    588588 * @return bool True if the key is protected, false otherwise.
    589589 */
    590590function is_protected_meta( $meta_key, $meta_type = null ) {
    591         $protected = (  '_' == $meta_key[0] );
     591        $protected = ( '_' == $meta_key[0] );
    592592
     593        if ( !empty( $meta_type ) && has_filter( "sanitize_{$meta_type}_meta_{$meta_key}" ) )
     594                $protected = false;
     595
    593596        return apply_filters( 'is_protected_meta', $protected, $meta_key, $meta_type );
    594597}
    595598
    596599/**
     600 * Determine whether a meta key is hidden
     601 *
     602 * @since 3.2.0
     603 *
     604 * @param string $meta_key Meta key
     605 * @return bool True if the key is hidden, false otherwise.
     606 */
     607function is_hidden_meta( $meta_key, $meta_type = null ) {
     608        $hidden = ( '_' == $meta_key[0] );
     609
     610        return apply_filters( 'is_hidden_meta', $hidden, $meta_key, $meta_type );
     611}
     612
     613/**
    597614 * Sanitize meta value
    598615 *
    599616 * @since 3.1.3
    … …  
    603620 * @param string $meta_type Type of meta
    604621 * @return mixed Sanitized $meta_value
    605622 */
    606 function sanitize_meta( $meta_key, $meta_value, $meta_type = null ) {
    607         return apply_filters( 'sanitize_meta', $meta_value, $meta_key, $meta_type );
     623function sanitize_meta( $meta_key, $meta_value, $meta_type ) {
     624        return apply_filters( "sanitize_{$meta_type}_meta_{$meta_key}", $meta_value, $meta_key, $meta_type );
    608625}
    609626
    610627?>

  • wp-admin/admin-ajax.php

     
    396396        if ( !$meta = get_post_meta_by_id( $id ) )
    397397                die('1');
    398398
    399         if ( !current_user_can( 'edit_post', $meta->post_id ) || is_protected_meta( $meta->meta_key ) )
     399        if ( !current_user_can( 'edit_post', $meta->post_id ) || is_hidden_meta( $meta->meta_key ) || is_protected_meta( $meta->meta_key, 'post' ) )
    400400                die('-1');
    401401        if ( delete_meta( $meta->meta_id ) )
    402402                die('1');
    … …  
    870870                        die('0'); // if meta doesn't exist
    871871                if ( !current_user_can( 'edit_post', $meta->post_id ) )
    872872                        die('-1');
    873                 if ( is_protected_meta( $meta->meta_key ) )
     873                if ( is_hidden_meta( $meta->meta_key ) || is_protected_meta( $meta->meta_key, 'post' ) )
    874874                        die('-1');
    875875                if ( $meta->meta_value != stripslashes($value) || $meta->meta_key != stripslashes($key) ) {
    876876                        if ( !$u = update_meta( $mid, $key, $value ) )

  • wp-admin/includes/post.php

     
    207207                                continue;
    208208                        if ( $meta->post_id != $post_ID )
    209209                                continue;
    210                         if ( is_protected_meta( $value['key'] ) )
     210                        if ( is_hidden_meta( $value['key'] ) || is_protected_meta( $value['key'], 'post' ) )
    211211                                continue;
    212212                        update_meta( $key, $value['key'], $value['value'] );
    213213                }
    … …  
    219219                                continue;
    220220                        if ( $meta->post_id != $post_ID )
    221221                                continue;
    222                         if ( is_protected_meta( $meta->meta_key ) )
     222                        if ( is_hidden_meta( $meta->meta_key ) || is_protected_meta( $meta->meta_key, 'post' ) )
    223223                                continue;
    224224                        delete_meta( $key );
    225225                }
    … …  
    662662                if ( $metakeyinput)
    663663                        $metakey = $metakeyinput; // default
    664664
    665                 if ( is_protected_meta( $metakey ) )
     665                if ( is_hidden_meta( $metakey ) || is_protected_meta( $metakey, 'post' ) )
    666666                        return false;
    667667
    668668                wp_cache_delete($post_ID, 'post_meta');
    … …  
    770770
    771771        $meta_key = stripslashes($meta_key);
    772772
    773         if ( is_protected_meta( $meta_key ) )
     773        if ( is_protected_meta( $meta_key, 'post' ) )
    774774                return false;
    775775
    776776        if ( '' === trim( $meta_value ) )

  • wp-admin/includes/template.php

     
    466466function _list_meta_row( $entry, &$count ) {
    467467        static $update_nonce = false;
    468468
    469         if ( is_protected_meta( $entry['meta_key'] ) )
     469        if ( is_hidden_meta( $entry['meta_key'] ) || is_protected_meta( $entry['meta_key'], 'post' ) )
    470470                return;
    471471
    472472        if ( !$update_nonce )