WordPress.org
Get WordPress

Make WordPress Core

Changeset 61454


Ignore:
Timestamp:
01/09/2026 02:38:07 AM (8 months ago)
Author:
westonruter
Message:

Code Modernization: Menus, Template, XML-RPC: Use null coalescing operator instead of isset() ternaries.

Developed as a subset of ​https://github.com/WordPress/wordpress-develop/pull/10654
Initially developed in ​https://github.com/WordPress/wordpress-develop/pull/4886

Follow-up to [61453], [61445], [61444], [61443], [61442], [61436], [61435], [61434], [61403], [61433], [61432], [61431], [61430], [61429], [61424], [61404], [61403].

Props costdev, westonruter.
See #58874, #63430.

Location:
trunk/src
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/nav-menu.php

    r60991 r61454  
    2323function _wp_ajax_menu_quick_search( $request = array() ) {
    2424        $args            = array();
    25         $type            = isset( $request['type'] ) ? $request['type'] : '';
    26         $object_type     = isset( $request['object_type'] ) ? $request['object_type'] : '';
    27         $query           = isset( $request['q'] ) ? $request['q'] : '';
    28         $response_format = isset( $request['response-format'] ) ? $request['response-format'] : '';
     25        $type            = $request['type'] ?? '';
     26        $object_type     = $request['object_type'] ?? '';
     27        $query           = $request['q'] ?? '';
     28        $response_format = $request['response-format'] ?? '';
    2929
    3030        if ( ! $response_format || ! in_array( $response_format, array( 'json', 'markup' ), true ) ) {
    … …  
    11841184
    11851185                        $args = array(
    1186                                 'menu-item-db-id'       => ( isset( $_item_object_data['menu-item-db-id'] ) ? $_item_object_data['menu-item-db-id'] : '' ),
    1187                                 'menu-item-object-id'   => ( isset( $_item_object_data['menu-item-object-id'] ) ? $_item_object_data['menu-item-object-id'] : '' ),
    1188                                 'menu-item-object'      => ( isset( $_item_object_data['menu-item-object'] ) ? $_item_object_data['menu-item-object'] : '' ),
    1189                                 'menu-item-parent-id'   => ( isset( $_item_object_data['menu-item-parent-id'] ) ? $_item_object_data['menu-item-parent-id'] : '' ),
    1190                                 'menu-item-position'    => ( isset( $_item_object_data['menu-item-position'] ) ? $_item_object_data['menu-item-position'] : '' ),
    1191                                 'menu-item-type'        => ( isset( $_item_object_data['menu-item-type'] ) ? $_item_object_data['menu-item-type'] : '' ),
    1192                                 'menu-item-title'       => ( isset( $_item_object_data['menu-item-title'] ) ? $_item_object_data['menu-item-title'] : '' ),
    1193                                 'menu-item-url'         => ( isset( $_item_object_data['menu-item-url'] ) ? $_item_object_data['menu-item-url'] : '' ),
    1194                                 'menu-item-description' => ( isset( $_item_object_data['menu-item-description'] ) ? $_item_object_data['menu-item-description'] : '' ),
    1195                                 'menu-item-attr-title'  => ( isset( $_item_object_data['menu-item-attr-title'] ) ? $_item_object_data['menu-item-attr-title'] : '' ),
    1196                                 'menu-item-target'      => ( isset( $_item_object_data['menu-item-target'] ) ? $_item_object_data['menu-item-target'] : '' ),
    1197                                 'menu-item-classes'     => ( isset( $_item_object_data['menu-item-classes'] ) ? $_item_object_data['menu-item-classes'] : '' ),
    1198                                 'menu-item-xfn'         => ( isset( $_item_object_data['menu-item-xfn'] ) ? $_item_object_data['menu-item-xfn'] : '' ),
     1186                                'menu-item-db-id'       => $_item_object_data['menu-item-db-id'] ?? '',
     1187                                'menu-item-object-id'   => $_item_object_data['menu-item-object-id'] ?? '',
     1188                                'menu-item-object'      => $_item_object_data['menu-item-object'] ?? '',
     1189                                'menu-item-parent-id'   => $_item_object_data['menu-item-parent-id'] ?? '',
     1190                                'menu-item-position'    => $_item_object_data['menu-item-position'] ?? '',
     1191                                'menu-item-type'        => $_item_object_data['menu-item-type'] ?? '',
     1192                                'menu-item-title'       => $_item_object_data['menu-item-title'] ?? '',
     1193                                'menu-item-url'         => $_item_object_data['menu-item-url'] ?? '',
     1194                                'menu-item-description' => $_item_object_data['menu-item-description'] ?? '',
     1195                                'menu-item-attr-title'  => $_item_object_data['menu-item-attr-title'] ?? '',
     1196                                'menu-item-target'      => $_item_object_data['menu-item-target'] ?? '',
     1197                                'menu-item-classes'     => $_item_object_data['menu-item-classes'] ?? '',
     1198                                'menu-item-xfn'         => $_item_object_data['menu-item-xfn'] ?? '',
    11991199                        );
    12001200
    … …  
    14481448                        $args = array();
    14491449                        foreach ( $post_fields as $field ) {
    1450                                 $args[ $field ] = isset( $_POST[ $field ][ $_key ] ) ? $_POST[ $field ][ $_key ] : '';
     1450                                $args[ $field ] = $_POST[ $field ][ $_key ] ?? '';
    14511451                        }
    14521452

  • trunk/src/wp-admin/link-parse-opml.php

    r60703 r61454  
    5656                $names[]        = $name;
    5757                $urls[]         = $url;
    58                 $targets[]      = isset( $attrs['TARGET'] ) ? $attrs['TARGET'] : '';
    59                 $feeds[]        = isset( $attrs['XMLURL'] ) ? $attrs['XMLURL'] : '';
    60                 $descriptions[] = isset( $attrs['DESCRIPTION'] ) ? $attrs['DESCRIPTION'] : '';
     58                $targets[]      = $attrs['TARGET'] ?? '';
     59                $feeds[]        = $attrs['XMLURL'] ?? '';
     60                $descriptions[] = $attrs['DESCRIPTION'] ?? '';
    6161        } // End if outline.
    6262}

  • trunk/src/wp-admin/nav-menus.php

    r60682 r61454  
    5353
    5454// Allowed actions: add, update, delete.
    55 $action = isset( $_REQUEST['action'] ) ? $_REQUEST['action'] : 'edit';
     55$action = $_REQUEST['action'] ?? 'edit';
    5656
    5757/*

  • trunk/src/wp-includes/author-template.php

    r61057 r61454  
    170170        if ( ! $user_id ) {
    171171                global $authordata;
    172                 $user_id = isset( $authordata->ID ) ? $authordata->ID : 0;
     172                $user_id = $authordata->ID ?? 0;
    173173        } else {
    174174                $authordata = get_userdata( $user_id );
    … …  
    179179        }
    180180
    181         $value = isset( $authordata->$field ) ? $authordata->$field : '';
     181        $value = $authordata->$field ?? '';
    182182
    183183        /**
    … …  
    503503
    504504        foreach ( $authors as $author_id ) {
    505                 $posts = isset( $post_counts[ $author_id ] ) ? $post_counts[ $author_id ] : 0;
     505                $posts = $post_counts[ $author_id ] ?? 0;
    506506
    507507                if ( ! $posts && $parsed_args['hide_empty'] ) {

  • trunk/src/wp-includes/category-template.php

    r60275 r61454  
    964964        $tags_data = array();
    965965        foreach ( $tags as $key => $tag ) {
    966                 $tag_id = isset( $tag->id ) ? $tag->id : $key;
     966                $tag_id = $tag->id ?? $key;
    967967
    968968                $count      = $counts[ $key ];

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

    r60655 r61454  
    16811681
    16821682                // Handle enclosures.
    1683                 $enclosure = isset( $post_data['enclosure'] ) ? $post_data['enclosure'] : null;
     1683                $enclosure = $post_data['enclosure'] ?? null;
    16841684                $this->add_enclosure_if_new( $post_id, $enclosure );
    16851685
    … …  
    19871987                $username = $args[1];
    19881988                $password = $args[2];
    1989                 $filter   = isset( $args[3] ) ? $args[3] : array();
     1989                $filter   = $args[3] ?? array();
    19901990
    19911991                if ( isset( $args[4] ) ) {
    … …  
    24582458                $password = $args[2];
    24592459                $taxonomy = $args[3];
    2460                 $filter   = isset( $args[4] ) ? $args[4] : array();
     2460                $filter   = $args[4] ?? array();
    24612461
    24622462                $user = $this->login( $username, $password );
    … …  
    26162616                $username = $args[1];
    26172617                $password = $args[2];
    2618                 $filter   = isset( $args[3] ) ? $args[3] : array( 'public' => true );
     2618                $filter   = $args[3] ?? array( 'public' => true );
    26192619
    26202620                if ( isset( $args[4] ) ) {
    … …  
    27652765                $username = $args[1];
    27662766                $password = $args[2];
    2767                 $filter   = isset( $args[3] ) ? $args[3] : array();
     2767                $filter   = $args[3] ?? array();
    27682768
    27692769                if ( isset( $args[4] ) ) {
    … …  
    36593659                $username = $args[1];
    36603660                $password = $args[2];
    3661                 $struct   = isset( $args[3] ) ? $args[3] : array();
     3661                $struct   = $args[3] ?? array();
    36623662
    36633663                $user = $this->login( $username, $password );
    … …  
    44344434                $username = $args[1];
    44354435                $password = $args[2];
    4436                 $struct   = isset( $args[3] ) ? $args[3] : array();
     4436                $struct   = $args[3] ?? array();
    44374437
    44384438                $user = $this->login( $username, $password );
    … …  
    46254625                $username = $args[1];
    46264626                $password = $args[2];
    4627                 $filter   = isset( $args[3] ) ? $args[3] : array( 'public' => true );
     4627                $filter   = $args[3] ?? array( 'public' => true );
    46284628
    46294629                if ( isset( $args[4] ) ) {
    … …  
    53765376                $password       = $args[2];
    53775377                $content_struct = $args[3];
    5378                 $publish        = isset( $args[4] ) ? $args[4] : 0;
     5378                $publish        = $args[4] ?? 0;
    53795379
    53805380                $user = $this->login( $username, $password );
    … …  
    54925492                }
    54935493
    5494                 $post_title   = isset( $content_struct['title'] ) ? $content_struct['title'] : '';
    5495                 $post_content = isset( $content_struct['description'] ) ? $content_struct['description'] : '';
     5494                $post_title   = $content_struct['title'] ?? '';
     5495                $post_content = $content_struct['description'] ?? '';
    54965496
    54975497                $post_status = $publish ? 'publish' : 'draft';
    … …  
    55115511                }
    55125512
    5513                 $post_excerpt = isset( $content_struct['mt_excerpt'] ) ? $content_struct['mt_excerpt'] : '';
    5514                 $post_more    = isset( $content_struct['mt_text_more'] ) ? $content_struct['mt_text_more'] : '';
    5515 
    5516                 $tags_input = isset( $content_struct['mt_keywords'] ) ? $content_struct['mt_keywords'] : array();
     5513                $post_excerpt = $content_struct['mt_excerpt'] ?? '';
     5514                $post_more    = $content_struct['mt_text_more'] ?? '';
     5515
     5516                $tags_input = $content_struct['mt_keywords'] ?? array();
    55175517
    55185518                if ( isset( $content_struct['mt_allow_comments'] ) ) {
    … …  
    56625662
    56635663                // Handle enclosures.
    5664                 $enclosure = isset( $content_struct['enclosure'] ) ? $content_struct['enclosure'] : null;
     5664                $enclosure = $content_struct['enclosure'] ?? null;
    56655665                $this->add_enclosure_if_new( $post_id, $enclosure );
    56665666
    … …  
    57725772                $password       = $args[2];
    57735773                $content_struct = $args[3];
    5774                 $publish        = isset( $args[4] ) ? $args[4] : 0;
     5774                $publish        = $args[4] ?? 0;
    57755775
    57765776                $user = $this->login( $username, $password );
    … …  
    59565956                }
    59575957
    5958                 $post_more = isset( $content_struct['mt_text_more'] ) ? $content_struct['mt_text_more'] : '';
     5958                $post_more = $content_struct['mt_text_more'] ?? '';
    59595959
    59605960                $post_status = $publish ? 'publish' : 'draft';
    … …  
    59735973                }
    59745974
    5975                 $tags_input = isset( $content_struct['mt_keywords'] ) ? $content_struct['mt_keywords'] : array();
     5975                $tags_input = $content_struct['mt_keywords'] ?? array();
    59765976
    59775977                if ( 'publish' === $post_status || 'private' === $post_status ) {
    … …  
    60816081
    60826082                // Handle enclosures.
    6083                 $enclosure = isset( $content_struct['enclosure'] ) ? $content_struct['enclosure'] : null;
     6083                $enclosure = $content_struct['enclosure'] ?? null;
    60846084                $this->add_enclosure_if_new( $post_id, $enclosure );
    60856085
    … …  
    70607060
    70617061                preg_match( '|<title>([^<]*?)</title>|is', $remote_source, $matchtitle );
    7062                 $title = isset( $matchtitle[1] ) ? $matchtitle[1] : '';
     7062                $title = $matchtitle[1] ?? '';
    70637063                if ( empty( $title ) ) {
    70647064                        return $this->pingback_error( 32, __( 'A title on that page cannot be found.' ) );

  • trunk/src/wp-includes/general-template.php

    r61411 r61454  
    46594659
    46604660        // Get max pages and current page out of the current query, if available.
    4661         $total   = isset( $wp_query->max_num_pages ) ? $wp_query->max_num_pages : 1;
     4661        $total   = $wp_query->max_num_pages ?? 1;
    46624662        $current = get_query_var( 'paged' ) ? (int) get_query_var( 'paged' ) : 1;
    46634663
    … …  
    46984698                // Find the format argument.
    46994699                $format       = explode( '?', str_replace( '%_%', $args['format'], $args['base'] ) );
    4700                 $format_query = isset( $format[1] ) ? $format[1] : '';
     4700                $format_query = $format[1] ?? '';
    47014701                wp_parse_str( $format_query, $format_args );
    47024702

  • trunk/src/wp-includes/link-template.php

    r61387 r61454  
    24392439
    24402440        $home_root = parse_url( home_url() );
    2441         $home_root = ( isset( $home_root['path'] ) ) ? $home_root['path'] : '';
     2441        $home_root = $home_root['path'] ?? '';
    24422442        $home_root = preg_quote( $home_root, '|' );
    24432443

  • trunk/src/wp-includes/nav-menu.php

    r58854 r61454  
    320320
    321321        $args = array(
    322                 'description' => ( isset( $menu_data['description'] ) ? $menu_data['description'] : '' ),
    323                 'name'        => ( isset( $menu_data['menu-name'] ) ? $menu_data['menu-name'] : '' ),
     322                'description' => $menu_data['description'] ?? '',
     323                'name'        => $menu_data['menu-name'] ?? '',
    324324                'parent'      => ( isset( $menu_data['parent'] ) ? (int) $menu_data['parent'] : 0 ),
    325325                'slug'        => null,
Note: See TracChangeset for help on using the changeset viewer.