WordPress.org
Get WordPress

Make WordPress Core

Changeset 60837


Ignore:
Timestamp:
09/30/2025 05:06:24 PM (5 weeks ago)
Author:
desrosj
Message:

Grouped backports for the 4.9 branch.

  • REST API: Increase the specificity of capability checks for collections when the edit context is in use.
  • Menus: Prevent HTML in menu item titles from being rendered unexpectedly.

Merges [60814], [60815], [60816] to the 4.9 branch.

Props andraganescu, desrosj, ehti, hurayraiit, iandunn, joehoyle, johnbillion, jorbin, mnelson4, noisysocks, peterwilsoncc, phillsav, rmccue, timothyblynjacobs, vortfu, westonruter , whyisjake, zieladam.

Location:
branches/4.9
Files:
11 edited

Legend:

Unmodified
Added
Removed

  • branches/4.9

  • branches/4.9/src/wp-admin/js/customize-nav-menus.js

    r42846 r60837  
    518518            }
    519519
    520             this.currentMenuControl.addItemToMenu( menu_item.attributes );
     520            // Leave the title as empty to reuse the original title as a placeholder if set.
     521            var nav_menu_item = Object.assign( {}, menu_item.attributes );
     522            if ( nav_menu_item.title === nav_menu_item.original_title ) {
     523                nav_menu_item.title = '';
     524            }
     525
     526            this.currentMenuControl.addItemToMenu( nav_menu_item );
    521527
    522528            $( menuitemTpl ).find( '.menu-item-handle' ).addClass( 'item-added' );
    … …  
    29722978                {
    29732979                    nav_menu_term_id: menuControl.params.menu_id,
    2974                     original_title: item.title,
    29752980                    position: position
    29762981                }

  • branches/4.9/src/wp-admin/js/nav-menu.js

    r41227 r60837  
    11691169
    11701170        eventOnClickMenuSave : function() {
    1171             var locs = '',
    1172             menuName = $('#menu-name'),
    1173             menuNameVal = menuName.val();
    1174             // Cancel and warn if invalid menu name
     1171            var menuName = $('#menu-name'),
     1172                menuNameVal = menuName.val();
     1173
     1174            // Cancel and warn if invalid menu name.
    11751175            if( !menuNameVal || menuNameVal == menuName.attr('title') || !menuNameVal.replace(/\s+/, '') ) {
    11761176                menuName.parent().addClass('form-invalid');
    11771177                return false;
    11781178            }
    1179             // Copy menu theme locations
     1179            // Copy menu theme locations.
     1180            // Note: This appears to be dead code since #nav-menu-theme-locations no longer exists, perhaps removed in r32842.
     1181            var $updateNavMenu = $('#update-nav-menu');
    11801182            $('#nav-menu-theme-locations select').each(function() {
    1181                 locs += '<input type="hidden" name="' + this.name + '" value="' + $(this).val() + '" />';
    1182             });
    1183             $('#update-nav-menu').append( locs );
    1184             // Update menu item position data
     1183                $updateNavMenu.append(
     1184                    $( '<input>', {
     1185                        type: 'hidden',
     1186                        name: this.name,
     1187                        value: $( this ).val()
     1188                    } )
     1189                );
     1190            });
     1191            // Update menu item position data.
    11851192            api.menuList.find('.menu-item-data-position').val( function(index) { return index + 1; } );
    11861193            window.onbeforeunload = null;
    … …  
    12221229
    12231230            if( ! $items.length ) {
    1224                 $('.categorychecklist', panel).html( '<li><p>' + navMenuL10n.noResultsFound + '</p></li>' );
     1231                var li = $( '<li>' );
     1232                var p = $( '<p>', { text: navMenuL10n.noResultsFound } );
     1233                li.append( p );
     1234                $('.categorychecklist', panel).empty().append( li );
    12251235                $( '.spinner', panel ).removeClass( 'is-active' );
    12261236                wrapper.addClass( 'has-no-menu-item' );

  • branches/4.9/src/wp-includes/class-wp-customize-nav-menus.php

    r43481 r60837  
    159159            } elseif ( 'post' !== $object && 0 === $page && $post_type->has_archive ) {
    160160                // Add a post type archive link.
     161                $title   = $post_type->labels->archives;
    161162                $items[] = array(
    162                     'id'         => $object . '-archive',
    163                     'title'      => $post_type->labels->archives,
    164                     'type'       => 'post_type_archive',
    165                     'type_label' => __( 'Post Type Archive' ),
    166                     'object'     => $object,
    167                     'url'        => get_post_type_archive_link( $object ),
     163                    'id'             => $object_name . '-archive',
     164                    'title'          => $title,
     165                    'original_title' => $title,
     166                    'type'           => 'post_type_archive',
     167                    'type_label'     => __( 'Post Type Archive' ),
     168                    'object'         => $object_name,
     169                    'url'            => get_post_type_archive_link( $object_name ),
    168170                );
    169171            }
    … …  
    194196                    $post_title = sprintf( __( '#%d (no title)' ), $post->ID );
    195197                }
     198
     199                $title   = html_entity_decode( $post_title, ENT_QUOTES, get_bloginfo( 'charset' ) );
    196200                $items[] = array(
    197                     'id'         => "post-{$post->ID}",
    198                     'title'      => html_entity_decode( $post_title, ENT_QUOTES, get_bloginfo( 'charset' ) ),
    199                     'type'       => 'post_type',
    200                     'type_label' => get_post_type_object( $post->post_type )->labels->singular_name,
    201                     'object'     => $post->post_type,
    202                     'object_id'  => intval( $post->ID ),
    203                     'url'        => get_permalink( intval( $post->ID ) ),
     201                    'id'             => "post-{$post->ID}",
     202                    'title'          => $title,
     203                    'original_title' => $title,
     204                    'type'           => 'post_type',
     205                    'type_label'     => get_post_type_object( $post->post_type )->labels->singular_name,
     206                    'object'         => $post->post_type,
     207                    'object_id'      => (int) $post->ID,
     208                    'url'            => get_permalink( (int) $post->ID ),
    204209                );
    205210            }
    … …  
    222227
    223228            foreach ( $terms as $term ) {
     229                $title   = html_entity_decode( $term->name, ENT_QUOTES, get_bloginfo( 'charset' ) );
    224230                $items[] = array(
    225                     'id'         => "term-{$term->term_id}",
    226                     'title'      => html_entity_decode( $term->name, ENT_QUOTES, get_bloginfo( 'charset' ) ),
    227                     'type'       => 'taxonomy',
    228                     'type_label' => get_taxonomy( $term->taxonomy )->labels->singular_name,
    229                     'object'     => $term->taxonomy,
    230                     'object_id'  => intval( $term->term_id ),
    231                     'url'        => get_term_link( intval( $term->term_id ), $term->taxonomy ),
     231                    'id'             => "term-{$term->term_id}",
     232                    'title'          => $title,
     233                    'original_title' => $title,
     234                    'type'           => 'taxonomy',
     235                    'type_label'     => get_taxonomy( $term->taxonomy )->labels->singular_name,
     236                    'object'         => $term->taxonomy,
     237                    'object_id'      => (int) $term->term_id,
     238                    'url'            => get_term_link( (int) $term->term_id, $term->taxonomy ),
    232239                );
    233240            }

  • branches/4.9/src/wp-includes/customize/class-wp-customize-nav-menu-item-setting.php

    r41894 r60837  
    5757        'xfn'              => '',
    5858        'status'           => 'publish',
    59         'original_title'   => '',
    6059        'nav_menu_term_id' => 0, // This will be supplied as the $menu_id arg for wp_update_nav_menu_item().
    6160        '_invalid'         => false,
    … …  
    212211     */
    213212    public function value() {
    214         if ( $this->is_previewed && $this->_previewed_blog_id === get_current_blog_id() ) {
     213        $type_label = null;
     214        if ( $this->is_previewed && get_current_blog_id() === $this->_previewed_blog_id ) {
    215215            $undefined  = new stdClass(); // Symbol.
    216216            $post_value = $this->post_value( $undefined );
    … …  
    220220            } else {
    221221                $value = $post_value;
    222             }
    223             if ( ! empty( $value ) && empty( $value['original_title'] ) ) {
    224                 $value['original_title'] = $this->get_original_title( (object) $value );
    225222            }
    226223        } elseif ( isset( $this->value ) ) {
    … …  
    235232                    $is_title_empty = empty( $post->post_title );
    236233                    $value = (array) wp_setup_nav_menu_item( $post );
     234                    if ( isset( $value['type_label'] ) ) {
     235                        $type_label = $value['type_label'];
     236                    }
    237237                    if ( $is_title_empty ) {
    238238                        $value['title'] = '';
    … …  
    251251        }
    252252
    253         if ( ! empty( $value ) && empty( $value['type_label'] ) ) {
    254             $value['type_label'] = $this->get_type_label( (object) $value );
     253        // These properties are read-only and are part of the setting for use in the Customizer UI.
     254        if ( is_array( $value ) ) {
     255            $value_obj               = (object) $value;
     256            $value['type_label']     = isset( $type_label ) ? $type_label : $this->get_type_label( $value_obj );
     257            $value['original_title'] = $this->get_original_title( $value_obj );
    255258        }
    256259
    … …  
    259262
    260263    /**
     264     * Prepares the value for editing on the client.
     265     *
     266     * @since 6.8.3
     267     *
     268     * @return array|false Value prepared for the client.
     269     */
     270    public function js_value() {
     271        $value = parent::js_value();
     272        if ( is_array( $value ) && isset( $value['original_title'] ) ) {
     273            // Decode entities for the sake of displaying the original title as a placeholder.
     274            $value['original_title'] = html_entity_decode( $value['original_title'], ENT_QUOTES, get_bloginfo( 'charset' ) );
     275        }
     276        return $value;
     277    }
     278
     279    /**
    261280     * Get original title.
    262281     *
    … …  
    264283     *
    265284     * @param object $item Nav menu item.
    266      * @return string The original title.
     285     * @return string The original title, without entity decoding.
    267286     */
    268287    protected function get_original_title( $item ) {
    … …  
    290309            }
    291310        }
    292         $original_title = html_entity_decode( $original_title, ENT_QUOTES, get_bloginfo( 'charset' ) );
    293311        return $original_title;
    294312    }
    … …  
    346364            $this->value['status'] = $this->value['post_status'];
    347365            unset( $this->value['post_status'] );
    348         }
    349 
    350         if ( ! isset( $this->value['original_title'] ) ) {
    351             $this->value['original_title'] = $this->get_original_title( (object) $this->value );
    352366        }
    353367
    … …  
    587601        unset( $item->position );
    588602
    589         if ( empty( $item->original_title ) ) {
    590             $item->original_title = $this->get_original_title( $item );
    591         }
    592603        if ( empty( $item->title ) && ! empty( $item->original_title ) ) {
    593             $item->title = $item->original_title;
     604            $item->title = $item->original_title; // This is NOT entity-decoded. It comes from self::get_original_title().
    594605        }
    595606        if ( $item->title ) {
    … …  
    641652     * @since 4.3.0
    642653     *
    643      * @param array $menu_item_value The value to sanitize.
     654     * @param array|false $value The menu item value to sanitize.
    644655     * @return array|false|null|WP_Error Null or WP_Error if an input isn't valid. False if it is marked for deletion.
    645656     *                                   Otherwise the sanitized value.
    … …  
    695706        }
    696707
    697         $menu_item_value['original_title'] = sanitize_text_field( $menu_item_value['original_title'] );
    698 
    699708        // Apply the same filters as when calling wp_insert_post().
    700709

  • branches/4.9/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

    r50732 r60837  
    339339
    340340        foreach ( $query_result as $post ) {
    341             if ( ! $this->check_read_permission( $post ) ) {
     341            if ( 'edit' === $request['context'] ) {
     342                $permission = $this->check_update_permission( $post );
     343            } else {
     344                $permission = $this->check_read_permission( $post );
     345            }
     346
     347            if ( ! $permission ) {
    342348                continue;
    343349            }

  • branches/4.9/src/wp-includes/rest-api/endpoints/class-wp-rest-terms-controller.php

    r54569 r60837  
    319319
    320320        foreach ( $query_result as $term ) {
     321            if ( 'edit' === $request['context'] && ! current_user_can( 'edit_term', $term->term_id ) ) {
     322                continue;
     323            }
    321324            $data = $this->prepare_item_for_response( $term, $request );
    322325            $response[] = $this->prepare_response_for_collection( $data );

  • branches/4.9/src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

    r56865 r60837  
    180180
    181181        if ( 'edit' === $request['context'] && ! current_user_can( 'list_users' ) ) {
    182             return new WP_Error( 'rest_forbidden_context', __( 'Sorry, you are not allowed to list users.' ), array( 'status' => rest_authorization_required_code() ) );
     182            return new WP_Error( 'rest_forbidden_context', __( 'Sorry, you are not allowed to edit users.' ), array( 'status' => rest_authorization_required_code() ) );
    183183        }
    184184
    … …  
    294294
    295295        foreach ( $query->results as $user ) {
     296            if ( 'edit' === $request['context'] && ! current_user_can( 'edit_user', $user->ID ) ) {
     297                continue;
     298            }
     299
    296300            $data = $this->prepare_item_for_response( $user, $request );
    297301            $users[] = $this->prepare_response_for_collection( $data );
    … …  
    388392        }
    389393
    390         if ( 'edit' === $request['context'] && ! current_user_can( 'list_users' ) ) {
    391             return new WP_Error( 'rest_user_cannot_view', __( 'Sorry, you are not allowed to list users.' ), array( 'status' => rest_authorization_required_code() ) );
    392         } elseif ( ! count_user_posts( $user->ID, $types ) && ! current_user_can( 'edit_user', $user->ID ) && ! current_user_can( 'list_users' ) ) {
     394        if ( 'edit' === $request['context'] && ! current_user_can( 'edit_user', $user->ID ) ) {
     395            return new WP_Error( 'rest_forbidden_context', __( 'Sorry, you are not allowed to edit this user.' ), array( 'status' => rest_authorization_required_code() ) );
     396        }
     397
     398        if ( ! current_user_can( 'edit_user', $user->ID ) && ! current_user_can( 'list_users' ) && ! count_user_posts( $user->ID, $types ) ) {
    393399            return new WP_Error( 'rest_user_cannot_view', __( 'Sorry, you are not allowed to list users.' ), array( 'status' => rest_authorization_required_code() ) );
    394400        }
    … …  
    891897        }
    892898
    893         if ( in_array( 'roles', $fields, true ) ) {
     899        if ( in_array( 'roles', $fields, true ) && ( current_user_can( 'list_users' ) || current_user_can( 'edit_user', $user->ID ) ) ) {
    894900            // Defensively call array_values() to ensure an array is returned.
    895901            $data['roles'] = array_values( $user->roles );

  • branches/4.9/tests/phpunit/tests/customize/nav-menu-item-setting.php

    r41697 r60837  
    9090            'xfn' => '',
    9191            'status' => 'publish',
    92             'original_title' => '',
    9392            'nav_menu_term_id' => 0,
    9493            '_invalid' => false,
    … …  
    544543            'xfn' => 'hello  inject',
    545544            'status' => 'draft',
    546             'original_title' => 'Hi',
     545            'original_title'   => 'Hi<script>unfilteredHtml()</script>',
    547546            'nav_menu_term_id' => 0,
    548547        );
    … …  
    842841            'xfn'              => '',
    843842            'status'           => 'publish',
    844             'original_title'   => '',
    845843            'nav_menu_term_id' => 0,
    846844            '_invalid'         => false,

  • branches/4.9/tests/phpunit/tests/customize/nav-menus.php

    r43481 r60837  
    171171            'id'         => "post-{$post_id}",
    172172            'title'      => 'Post Title',
     173            'original_title'      => 'Post Title',
    173174            'type'       => 'post_type',
    174175            'type_label' => 'Post',
    … …  
    198199            'id'         => "post-{$page_id}",
    199200            'title'      => 'Page Title',
     201            'original_title'      => 'Page Title',
    200202            'type'       => 'post_type',
    201203            'type_label' => 'Page',
    … …  
    224226            'id'         => "post-{$post_id}",
    225227            'title'      => 'Post Title',
     228            'original_title'      => 'Post Title',
    226229            'type'       => 'post_type',
    227230            'type_label' => 'Post',
    … …  
    250253            'id'         => "term-{$term_id}",
    251254            'title'      => 'Term Title',
     255            'original_title' => 'Term Title',
    252256            'type'       => 'taxonomy',
    253257            'type_label' => 'Category',

  • branches/4.9/tests/phpunit/tests/rest-api/rest-users-controller.php

    r43445 r60837  
    934934        $request->set_param( 'context', 'edit' );
    935935        $response = $this->server->dispatch( $request );
    936         $this->assertErrorResponse( 'rest_user_cannot_view', $response, 401 );
     936        $this->assertErrorResponse( 'rest_forbidden_context', $response, 401 );
    937937    }
    938938
Note: See TracChangeset for help on using the changeset viewer.