WordPress.org
Get WordPress

Make WordPress Core

Changeset 61454


Ignore:
Timestamp:
01/09/2026 02:38:07 AM (2 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.