WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (3 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/nav-menu.php

    r42228 r42343  
    6161 */
    6262function is_nav_menu( $menu ) {
    63     if ( ! $menu )
     63    if ( ! $menu ) {
    6464        return false;
     65    }
    6566
    6667    $menu_obj = wp_get_nav_menu_object( $menu );
     
    7172        ! empty( $menu_obj->taxonomy ) &&
    7273        'nav_menu' == $menu_obj->taxonomy
    73     )
     74    ) {
    7475        return true;
     76    }
    7577
    7678    return false;
     
    106108    global $_wp_registered_nav_menus;
    107109
    108     if ( is_array( $_wp_registered_nav_menus ) && isset( $_wp_registered_nav_menus[$location] ) ) {
    109         unset( $_wp_registered_nav_menus[$location] );
     110    if ( is_array( $_wp_registered_nav_menus ) && isset( $_wp_registered_nav_menus[ $location ] ) ) {
     111        unset( $_wp_registered_nav_menus[ $location ] );
    110112        if ( empty( $_wp_registered_nav_menus ) ) {
    111113            _remove_theme_support( 'menus' );
     
    138140function get_registered_nav_menus() {
    139141    global $_wp_registered_nav_menus;
    140     if ( isset( $_wp_registered_nav_menus ) )
     142    if ( isset( $_wp_registered_nav_menus ) ) {
    141143        return $_wp_registered_nav_menus;
     144    }
    142145    return array();
    143146}
     
    170173    $registered_nav_menus = get_registered_nav_menus();
    171174    if ( isset( $registered_nav_menus[ $location ] ) ) {
    172         $locations = get_nav_menu_locations();
     175        $locations    = get_nav_menu_locations();
    173176        $has_nav_menu = ! empty( $locations[ $location ] );
    174177    }
     
    254257function wp_delete_nav_menu( $menu ) {
    255258    $menu = wp_get_nav_menu_object( $menu );
    256     if ( ! $menu )
     259    if ( ! $menu ) {
    257260        return false;
     261    }
    258262
    259263    $menu_objects = get_objects_in_term( $menu->term_id, 'nav_menu' );
     
    269273    $locations = get_nav_menu_locations();
    270274    foreach ( $locations as $location => $menu_id ) {
    271         if ( $menu_id == $menu->term_id )
     275        if ( $menu_id == $menu->term_id ) {
    272276            $locations[ $location ] = 0;
     277        }
    273278    }
    274279    set_theme_mod( 'nav_menu_locations', $locations );
    275280
    276     if ( $result && !is_wp_error($result) )
     281    if ( $result && ! is_wp_error( $result ) ) {
    277282
    278283        /**
     
    284289         */
    285290        do_action( 'wp_delete_nav_menu', $menu->term_id );
     291    }
    286292
    287293    return $result;
     
    306312
    307313    $args = array(
    308         'description' => ( isset( $menu_data['description'] ) ? $menu_data['description']  : '' ),
    309         'name'        => ( isset( $menu_data['menu-name']   ) ? $menu_data['menu-name']    : '' ),
    310         'parent'      => ( isset( $menu_data['parent']      ) ? (int) $menu_data['parent'] : 0 ),
     314        'description' => ( isset( $menu_data['description'] ) ? $menu_data['description'] : '' ),
     315        'name'        => ( isset( $menu_data['menu-name'] ) ? $menu_data['menu-name'] : '' ),
     316        'parent'      => ( isset( $menu_data['parent'] ) ? (int) $menu_data['parent'] : 0 ),
    311317        'slug'        => null,
    312318    );
     
    320326        $_possible_existing->term_id != $menu_id
    321327    ) {
    322         return new WP_Error( 'menu_exists',
     328        return new WP_Error(
     329            'menu_exists',
    323330            /* translators: %s: menu name */
    324             sprintf( __( 'The menu name %s conflicts with another menu name. Please try another.' ),
     331            sprintf(
     332                __( 'The menu name %s conflicts with another menu name. Please try another.' ),
    325333                '<strong>' . esc_html( $menu_data['menu-name'] ) . '</strong>'
    326334            )
     
    333341
    334342        if ( $menu_exists ) {
    335             return new WP_Error( 'menu_exists',
     343            return new WP_Error(
     344                'menu_exists',
    336345                /* translators: %s: menu name */
    337                 sprintf( __( 'The menu name %s conflicts with another menu name. Please try another.' ),
     346                sprintf(
     347                    __( 'The menu name %s conflicts with another menu name. Please try another.' ),
    338348                    '<strong>' . esc_html( $menu_data['menu-name'] ) . '</strong>'
    339349                )
     
    343353        $_menu = wp_insert_term( $menu_data['menu-name'], 'nav_menu', $args );
    344354
    345         if ( is_wp_error( $_menu ) )
     355        if ( is_wp_error( $_menu ) ) {
    346356            return $_menu;
     357        }
    347358
    348359        /**
     
    359370    }
    360371
    361     if ( ! $_menu || ! isset( $_menu->term_id ) )
     372    if ( ! $_menu || ! isset( $_menu->term_id ) ) {
    362373        return 0;
     374    }
    363375
    364376    $menu_id = (int) $_menu->term_id;
     
    366378    $update_response = wp_update_term( $menu_id, 'nav_menu', $args );
    367379
    368     if ( is_wp_error( $update_response ) )
     380    if ( is_wp_error( $update_response ) ) {
    369381        return $update_response;
     382    }
    370383
    371384    $menu_id = (int) $update_response['term_id'];
     
    397410 */
    398411function wp_update_nav_menu_item( $menu_id = 0, $menu_item_db_id = 0, $menu_item_data = array() ) {
    399     $menu_id = (int) $menu_id;
     412    $menu_id         = (int) $menu_id;
    400413    $menu_item_db_id = (int) $menu_item_db_id;
    401414
    402415    // make sure that we don't convert non-nav_menu_item objects into nav_menu_item objects
    403     if ( ! empty( $menu_item_db_id ) && ! is_nav_menu_item( $menu_item_db_id ) )
     416    if ( ! empty( $menu_item_db_id ) && ! is_nav_menu_item( $menu_item_db_id ) ) {
    404417        return new WP_Error( 'update_nav_menu_item_failed', __( 'The given object ID is not that of a menu item.' ) );
     418    }
    405419
    406420    $menu = wp_get_nav_menu_object( $menu_id );
     
    415429
    416430    $defaults = array(
    417         'menu-item-db-id' => $menu_item_db_id,
    418         'menu-item-object-id' => 0,
    419         'menu-item-object' => '',
    420         'menu-item-parent-id' => 0,
    421         'menu-item-position' => 0,
    422         'menu-item-type' => 'custom',
    423         'menu-item-title' => '',
    424         'menu-item-url' => '',
     431        'menu-item-db-id'       => $menu_item_db_id,
     432        'menu-item-object-id'   => 0,
     433        'menu-item-object'      => '',
     434        'menu-item-parent-id'   => 0,
     435        'menu-item-position'    => 0,
     436        'menu-item-type'        => 'custom',
     437        'menu-item-title'       => '',
     438        'menu-item-url'         => '',
    425439        'menu-item-description' => '',
    426         'menu-item-attr-title' => '',
    427         'menu-item-target' => '',
    428         'menu-item-classes' => '',
    429         'menu-item-xfn' => '',
    430         'menu-item-status' => '',
     440        'menu-item-attr-title'  => '',
     441        'menu-item-target'      => '',
     442        'menu-item-classes'     => '',
     443        'menu-item-xfn'         => '',
     444        'menu-item-status'      => '',
    431445    );
    432446
     
    436450        $args['menu-item-position'] = 1;
    437451    } elseif ( 0 == (int) $args['menu-item-position'] ) {
    438         $menu_items = 0 == $menu_id ? array() : (array) wp_get_nav_menu_items( $menu_id, array( 'post_status' => 'publish,draft' ) );
    439         $last_item = array_pop( $menu_items );
     452        $menu_items                 = 0 == $menu_id ? array() : (array) wp_get_nav_menu_items( $menu_id, array( 'post_status' => 'publish,draft' ) );
     453        $last_item                  = array_pop( $menu_items );
    440454        $args['menu-item-position'] = ( $last_item && isset( $last_item->menu_order ) ) ? 1 + $last_item->menu_order : count( $menu_items );
    441455    }
     
    454468        if ( 'taxonomy' == $args['menu-item-type'] ) {
    455469            $original_parent = get_term_field( 'parent', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
    456             $original_title = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
     470            $original_title  = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
    457471        } elseif ( 'post_type' == $args['menu-item-type'] ) {
    458472
    459473            $original_object = get_post( $args['menu-item-object-id'] );
    460474            $original_parent = (int) $original_object->post_parent;
    461             $original_title = $original_object->post_title;
     475            $original_title  = $original_object->post_title;
    462476        } elseif ( 'post_type_archive' == $args['menu-item-type'] ) {
    463477            $original_object = get_post_type_object( $args['menu-item-object'] );
     
    467481        }
    468482
    469         if ( $args['menu-item-title'] == $original_title )
     483        if ( $args['menu-item-title'] == $original_title ) {
    470484            $args['menu-item-title'] = '';
     485        }
    471486
    472487        // hack to get wp to create a post object when too many properties are empty
    473         if ( '' ==  $args['menu-item-title'] && '' == $args['menu-item-description'] )
     488        if ( '' == $args['menu-item-title'] && '' == $args['menu-item-description'] ) {
    474489            $args['menu-item-description'] = ' ';
     490        }
    475491    }
    476492
    477493    // Populate the menu item object
    478494    $post = array(
    479         'menu_order' => $args['menu-item-position'],
    480         'ping_status' => 0,
     495        'menu_order'   => $args['menu-item-position'],
     496        'ping_status'  => 0,
    481497        'post_content' => $args['menu-item-description'],
    482498        'post_excerpt' => $args['menu-item-attr-title'],
    483         'post_parent' => $original_parent,
    484         'post_title' => $args['menu-item-title'],
    485         'post_type' => 'nav_menu_item',
     499        'post_parent'  => $original_parent,
     500        'post_title'   => $args['menu-item-title'],
     501        'post_type'    => 'nav_menu_item',
    486502    );
    487503
     
    490506    // New menu item. Default is draft status
    491507    if ( ! $update ) {
    492         $post['ID'] = 0;
     508        $post['ID']          = 0;
    493509        $post['post_status'] = 'publish' == $args['menu-item-status'] ? 'publish' : 'draft';
    494         $menu_item_db_id = wp_insert_post( $post );
    495         if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) )
     510        $menu_item_db_id     = wp_insert_post( $post );
     511        if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) ) {
    496512            return $menu_item_db_id;
     513        }
    497514
    498515        /**
     
    518535    if ( 'custom' == $args['menu-item-type'] ) {
    519536        $args['menu-item-object-id'] = $menu_item_db_id;
    520         $args['menu-item-object'] = 'custom';
     537        $args['menu-item-object']    = 'custom';
    521538    }
    522539
    523540    $menu_item_db_id = (int) $menu_item_db_id;
    524541
    525     update_post_meta( $menu_item_db_id, '_menu_item_type', sanitize_key($args['menu-item-type']) );
     542    update_post_meta( $menu_item_db_id, '_menu_item_type', sanitize_key( $args['menu-item-type'] ) );
    526543    update_post_meta( $menu_item_db_id, '_menu_item_menu_item_parent', strval( (int) $args['menu-item-parent-id'] ) );
    527544    update_post_meta( $menu_item_db_id, '_menu_item_object_id', strval( (int) $args['menu-item-object-id'] ) );
    528     update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key($args['menu-item-object']) );
    529     update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key($args['menu-item-target']) );
     545    update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key( $args['menu-item-object'] ) );
     546    update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key( $args['menu-item-target'] ) );
    530547
    531548    $args['menu-item-classes'] = array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-classes'] ) );
    532     $args['menu-item-xfn'] = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-xfn'] ) ) );
     549    $args['menu-item-xfn']     = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-xfn'] ) ) );
    533550    update_post_meta( $menu_item_db_id, '_menu_item_classes', $args['menu-item-classes'] );
    534551    update_post_meta( $menu_item_db_id, '_menu_item_xfn', $args['menu-item-xfn'] );
    535     update_post_meta( $menu_item_db_id, '_menu_item_url', esc_url_raw($args['menu-item-url']) );
    536 
    537     if ( 0 == $menu_id )
     552    update_post_meta( $menu_item_db_id, '_menu_item_url', esc_url_raw( $args['menu-item-url'] ) );
     553
     554    if ( 0 == $menu_id ) {
    538555        update_post_meta( $menu_item_db_id, '_menu_item_orphaned', (string) time() );
    539     elseif ( get_post_meta( $menu_item_db_id, '_menu_item_orphaned' ) )
     556    } elseif ( get_post_meta( $menu_item_db_id, '_menu_item_orphaned' ) ) {
    540557        delete_post_meta( $menu_item_db_id, '_menu_item_orphaned' );
     558    }
    541559
    542560    // Update existing menu item. Default is publish status
    543561    if ( $update ) {
    544         $post['ID'] = $menu_item_db_id;
     562        $post['ID']          = $menu_item_db_id;
    545563        $post['post_status'] = 'draft' == $args['menu-item-status'] ? 'draft' : 'publish';
    546564        wp_update_post( $post );
     
    575593 */
    576594function wp_get_nav_menus( $args = array() ) {
    577     $defaults = array( 'hide_empty' => false, 'orderby' => 'name' );
    578     $args = wp_parse_args( $args, $defaults );
     595    $defaults = array(
     596        'hide_empty' => false,
     597        'orderby'    => 'name',
     598    );
     599    $args     = wp_parse_args( $args, $defaults );
    579600
    580601    /**
     
    588609     * @param array $args  An array of arguments used to retrieve menu objects.
    589610     */
    590     return apply_filters( 'wp_get_nav_menus', get_terms( 'nav_menu',  $args), $args );
     611    return apply_filters( 'wp_get_nav_menus', get_terms( 'nav_menu', $args ), $args );
    591612}
    592613
     
    652673    }
    653674
    654     $defaults = array( 'order' => 'ASC', 'orderby' => 'menu_order', 'post_type' => 'nav_menu_item',
    655         'post_status' => 'publish', 'output' => ARRAY_A, 'output_key' => 'menu_order', 'nopaging' => true );
    656     $args = wp_parse_args( $args, $defaults );
     675    $defaults        = array(
     676        'order'       => 'ASC',
     677        'orderby'     => 'menu_order',
     678        'post_type'   => 'nav_menu_item',
     679        'post_status' => 'publish',
     680        'output'      => ARRAY_A,
     681        'output_key'  => 'menu_order',
     682        'nopaging'    => true,
     683    );
     684    $args            = wp_parse_args( $args, $defaults );
    657685    $args['include'] = $items;
    658686
     
    665693    // Get all posts and terms at once to prime the caches
    666694    if ( empty( $fetched[ $menu->term_id ] ) && ! wp_using_ext_object_cache() ) {
    667         $fetched[$menu->term_id] = true;
    668         $posts = array();
    669         $terms = array();
     695        $fetched[ $menu->term_id ] = true;
     696        $posts                     = array();
     697        $terms                     = array();
    670698        foreach ( $items as $item ) {
    671699            $object_id = get_post_meta( $item->ID, '_menu_item_object_id', true );
    672             $object    = get_post_meta( $item->ID, '_menu_item_object',    true );
    673             $type      = get_post_meta( $item->ID, '_menu_item_type',      true );
    674 
    675             if ( 'post_type' == $type )
    676                 $posts[$object][] = $object_id;
    677             elseif ( 'taxonomy' == $type)
    678                 $terms[$object][] = $object_id;
     700            $object    = get_post_meta( $item->ID, '_menu_item_object', true );
     701            $type      = get_post_meta( $item->ID, '_menu_item_type', true );
     702
     703            if ( 'post_type' == $type ) {
     704                $posts[ $object ][] = $object_id;
     705            } elseif ( 'taxonomy' == $type ) {
     706                $terms[ $object ][] = $object_id;
     707            }
    679708        }
    680709
    681710        if ( ! empty( $posts ) ) {
    682             foreach ( array_keys($posts) as $post_type ) {
    683                 get_posts( array('post__in' => $posts[$post_type], 'post_type' => $post_type, 'nopaging' => true, 'update_post_term_cache' => false) );
     711            foreach ( array_keys( $posts ) as $post_type ) {
     712                get_posts(
     713                    array(
     714                        'post__in'               => $posts[ $post_type ],
     715                        'post_type'              => $post_type,
     716                        'nopaging'               => true,
     717                        'update_post_term_cache' => false,
     718                    )
     719                );
    684720            }
    685721        }
    686         unset($posts);
     722        unset( $posts );
    687723
    688724        if ( ! empty( $terms ) ) {
    689             foreach ( array_keys($terms) as $taxonomy ) {
    690                 get_terms( $taxonomy, array(
    691                     'include' => $terms[ $taxonomy ],
    692                     'hierarchical' => false,
    693                 ) );
     725            foreach ( array_keys( $terms ) as $taxonomy ) {
     726                get_terms(
     727                    $taxonomy, array(
     728                        'include'      => $terms[ $taxonomy ],
     729                        'hierarchical' => false,
     730                    )
     731                );
    694732            }
    695733        }
    696         unset($terms);
     734        unset( $terms );
    697735    }
    698736
     
    704742
    705743    if ( ARRAY_A == $args['output'] ) {
    706         $items = wp_list_sort( $items, array(
    707             $args['output_key'] => 'ASC',
    708         ) );
    709         $i = 1;
     744        $items = wp_list_sort(
     745            $items, array(
     746                $args['output_key'] => 'ASC',
     747            )
     748        );
     749        $i     = 1;
    710750        foreach ( $items as $k => $item ) {
    711             $items[$k]->{$args['output_key']} = $i++;
     751            $items[ $k ]->{$args['output_key']} = $i++;
    712752        }
    713753    }
     
    755795    if ( isset( $menu_item->post_type ) ) {
    756796        if ( 'nav_menu_item' == $menu_item->post_type ) {
    757             $menu_item->db_id = (int) $menu_item->ID;
     797            $menu_item->db_id            = (int) $menu_item->ID;
    758798            $menu_item->menu_item_parent = ! isset( $menu_item->menu_item_parent ) ? get_post_meta( $menu_item->ID, '_menu_item_menu_item_parent', true ) : $menu_item->menu_item_parent;
    759             $menu_item->object_id = ! isset( $menu_item->object_id ) ? get_post_meta( $menu_item->ID, '_menu_item_object_id', true ) : $menu_item->object_id;
    760             $menu_item->object = ! isset( $menu_item->object ) ? get_post_meta( $menu_item->ID, '_menu_item_object', true ) : $menu_item->object;
    761             $menu_item->type = ! isset( $menu_item->type ) ? get_post_meta( $menu_item->ID, '_menu_item_type', true ) : $menu_item->type;
     799            $menu_item->object_id        = ! isset( $menu_item->object_id ) ? get_post_meta( $menu_item->ID, '_menu_item_object_id', true ) : $menu_item->object_id;
     800            $menu_item->object           = ! isset( $menu_item->object ) ? get_post_meta( $menu_item->ID, '_menu_item_object', true ) : $menu_item->object;
     801            $menu_item->type             = ! isset( $menu_item->type ) ? get_post_meta( $menu_item->ID, '_menu_item_type', true ) : $menu_item->type;
    762802
    763803            if ( 'post_type' == $menu_item->type ) {
     
    767807                } else {
    768808                    $menu_item->type_label = $menu_item->object;
    769                     $menu_item->_invalid = true;
     809                    $menu_item->_invalid   = true;
    770810                }
    771811
     
    788828
    789829            } elseif ( 'post_type_archive' == $menu_item->type ) {
    790                 $object =  get_post_type_object( $menu_item->object );
     830                $object = get_post_type_object( $menu_item->object );
    791831                if ( $object ) {
    792                     $menu_item->title = '' == $menu_item->post_title ? $object->labels->archives : $menu_item->post_title;
     832                    $menu_item->title      = '' == $menu_item->post_title ? $object->labels->archives : $menu_item->post_title;
    793833                    $post_type_description = $object->description;
    794834                } else {
    795                     $menu_item->_invalid = true;
     835                    $menu_item->_invalid   = true;
    796836                    $post_type_description = '';
    797837                }
    798838
    799839                $menu_item->type_label = __( 'Post Type Archive' );
    800                 $post_content = wp_trim_words( $menu_item->post_content, 200 );
     840                $post_content          = wp_trim_words( $menu_item->post_content, 200 );
    801841                $post_type_description = '' == $post_content ? $post_type_description : $post_content;
    802                 $menu_item->url = get_post_type_archive_link( $menu_item->object );
     842                $menu_item->url        = get_post_type_archive_link( $menu_item->object );
    803843            } elseif ( 'taxonomy' == $menu_item->type ) {
    804844                $object = get_taxonomy( $menu_item->object );
     
    807847                } else {
    808848                    $menu_item->type_label = $menu_item->object;
    809                     $menu_item->_invalid = true;
     849                    $menu_item->_invalid   = true;
    810850                }
    811851
    812                 $term_url = get_term_link( (int) $menu_item->object_id, $menu_item->object );
    813                 $menu_item->url = !is_wp_error( $term_url ) ? $term_url : '';
     852                $term_url       = get_term_link( (int) $menu_item->object_id, $menu_item->object );
     853                $menu_item->url = ! is_wp_error( $term_url ) ? $term_url : '';
    814854
    815855                $original_title = get_term_field( 'name', $menu_item->object_id, $menu_item->object, 'raw' );
    816                 if ( is_wp_error( $original_title ) )
     856                if ( is_wp_error( $original_title ) ) {
    817857                    $original_title = false;
     858                }
    818859                $menu_item->title = '' == $menu_item->post_title ? $original_title : $menu_item->post_title;
    819860
    820861            } else {
    821                 $menu_item->type_label = __('Custom Link');
    822                 $menu_item->title = $menu_item->post_title;
    823                 $menu_item->url = ! isset( $menu_item->url ) ? get_post_meta( $menu_item->ID, '_menu_item_url', true ) : $menu_item->url;
     862                $menu_item->type_label = __( 'Custom Link' );
     863                $menu_item->title      = $menu_item->post_title;
     864                $menu_item->url        = ! isset( $menu_item->url ) ? get_post_meta( $menu_item->ID, '_menu_item_url', true ) : $menu_item->url;
    824865            }
    825866
     
    847888
    848889            $menu_item->classes = ! isset( $menu_item->classes ) ? (array) get_post_meta( $menu_item->ID, '_menu_item_classes', true ) : $menu_item->classes;
    849             $menu_item->xfn = ! isset( $menu_item->xfn ) ? get_post_meta( $menu_item->ID, '_menu_item_xfn', true ) : $menu_item->xfn;
     890            $menu_item->xfn     = ! isset( $menu_item->xfn ) ? get_post_meta( $menu_item->ID, '_menu_item_xfn', true ) : $menu_item->xfn;
    850891        } else {
    851             $menu_item->db_id = 0;
     892            $menu_item->db_id            = 0;
    852893            $menu_item->menu_item_parent = 0;
    853             $menu_item->object_id = (int) $menu_item->ID;
    854             $menu_item->type = 'post_type';
    855 
    856             $object = get_post_type_object( $menu_item->post_type );
    857             $menu_item->object = $object->name;
     894            $menu_item->object_id        = (int) $menu_item->ID;
     895            $menu_item->type             = 'post_type';
     896
     897            $object                = get_post_type_object( $menu_item->post_type );
     898            $menu_item->object     = $object->name;
    858899            $menu_item->type_label = $object->labels->singular_name;
    859900
     
    863904            }
    864905
    865             $menu_item->title = $menu_item->post_title;
    866             $menu_item->url = get_permalink( $menu_item->ID );
     906            $menu_item->title  = $menu_item->post_title;
     907            $menu_item->url    = get_permalink( $menu_item->ID );
    867908            $menu_item->target = '';
    868909
     
    872913            /** This filter is documented in wp-includes/nav-menu.php */
    873914            $menu_item->description = apply_filters( 'nav_menu_description', '' );
    874             $menu_item->classes = array();
    875             $menu_item->xfn = '';
     915            $menu_item->classes     = array();
     916            $menu_item->xfn         = '';
    876917        }
    877918    } elseif ( isset( $menu_item->taxonomy ) ) {
    878         $menu_item->ID = $menu_item->term_id;
    879         $menu_item->db_id = 0;
     919        $menu_item->ID               = $menu_item->term_id;
     920        $menu_item->db_id            = 0;
    880921        $menu_item->menu_item_parent = 0;
    881         $menu_item->object_id = (int) $menu_item->term_id;
    882         $menu_item->post_parent = (int) $menu_item->parent;
    883         $menu_item->type = 'taxonomy';
    884 
    885         $object = get_taxonomy( $menu_item->taxonomy );
    886         $menu_item->object = $object->name;
     922        $menu_item->object_id        = (int) $menu_item->term_id;
     923        $menu_item->post_parent      = (int) $menu_item->parent;
     924        $menu_item->type             = 'taxonomy';
     925
     926        $object                = get_taxonomy( $menu_item->taxonomy );
     927        $menu_item->object     = $object->name;
    887928        $menu_item->type_label = $object->labels->singular_name;
    888929
    889         $menu_item->title = $menu_item->name;
    890         $menu_item->url = get_term_link( $menu_item, $menu_item->taxonomy );
    891         $menu_item->target = '';
    892         $menu_item->attr_title = '';
     930        $menu_item->title       = $menu_item->name;
     931        $menu_item->url         = get_term_link( $menu_item, $menu_item->taxonomy );
     932        $menu_item->target      = '';
     933        $menu_item->attr_title  = '';
    893934        $menu_item->description = get_term_field( 'description', $menu_item->term_id, $menu_item->taxonomy );
    894         $menu_item->classes = array();
    895         $menu_item->xfn = '';
     935        $menu_item->classes     = array();
     936        $menu_item->xfn         = '';
    896937
    897938    }
     
    918959 */
    919960function wp_get_associated_nav_menu_items( $object_id = 0, $object_type = 'post_type', $taxonomy = '' ) {
    920     $object_id = (int) $object_id;
     961    $object_id     = (int) $object_id;
    921962    $menu_item_ids = array();
    922963
    923     $query = new WP_Query;
     964    $query      = new WP_Query;
    924965    $menu_items = $query->query(
    925966        array(
    926             'meta_key' => '_menu_item_object_id',
    927             'meta_value' => $object_id,
    928             'post_status' => 'any',
    929             'post_type' => 'nav_menu_item',
     967            'meta_key'       => '_menu_item_object_id',
     968            'meta_value'     => $object_id,
     969            'post_status'    => 'any',
     970            'post_type'      => 'nav_menu_item',
    930971            'posts_per_page' => -1,
    931972        )
     
    9591000 *
    9601001 * @param int $object_id The ID of the original object being trashed.
    961  *
    9621002 */
    9631003function _wp_delete_post_menu_item( $object_id = 0 ) {
     
    10021042 */
    10031043function _wp_auto_add_pages_to_menu( $new_status, $old_status, $post ) {
    1004     if ( 'publish' != $new_status || 'publish' == $old_status || 'page' != $post->post_type )
     1044    if ( 'publish' != $new_status || 'publish' == $old_status || 'page' != $post->post_type ) {
    10051045        return;
    1006     if ( ! empty( $post->post_parent ) )
     1046    }
     1047    if ( ! empty( $post->post_parent ) ) {
    10071048        return;
     1049    }
    10081050    $auto_add = get_option( 'nav_menu_options' );
    1009     if ( empty( $auto_add ) || ! is_array( $auto_add ) || ! isset( $auto_add['auto_add'] ) )
     1051    if ( empty( $auto_add ) || ! is_array( $auto_add ) || ! isset( $auto_add['auto_add'] ) ) {
    10101052        return;
     1053    }
    10111054    $auto_add = $auto_add['auto_add'];
    1012     if ( empty( $auto_add ) || ! is_array( $auto_add ) )
     1055    if ( empty( $auto_add ) || ! is_array( $auto_add ) ) {
    10131056        return;
     1057    }
    10141058
    10151059    $args = array(
    10161060        'menu-item-object-id' => $post->ID,
    1017         'menu-item-object' => $post->post_type,
    1018         'menu-item-type' => 'post_type',
    1019         'menu-item-status' => 'publish',
     1061        'menu-item-object'    => $post->post_type,
     1062        'menu-item-type'      => 'post_type',
     1063        'menu-item-status'    => 'publish',
    10201064    );
    10211065
    10221066    foreach ( $auto_add as $menu_id ) {
    10231067        $items = wp_get_nav_menu_items( $menu_id, array( 'post_status' => 'publish,draft' ) );
    1024         if ( ! is_array( $items ) )
     1068        if ( ! is_array( $items ) ) {
    10251069            continue;
     1070        }
    10261071        foreach ( $items as $item ) {
    1027             if ( $post->ID == $item->object_id )
     1072            if ( $post->ID == $item->object_id ) {
    10281073                continue 2;
     1074            }
    10291075        }
    10301076        wp_update_nav_menu_item( $menu_id, 0, $args );
Note: See TracChangeset for help on using the changeset viewer.