WordPress.org
Get WordPress

Make WordPress Core

Changeset 15078


Ignore:
Timestamp:
05/31/2010 04:11:20 PM (16 years ago)
Author:
nacin
Message:

Cleanups.

Location:
trunk/wp-includes
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-includes/default-filters.php

    r14996 r15078  
    230230add_action( 'comment_form', 'wp_comment_form_unfiltered_html_nonce'        );
    231231add_action( 'wp_scheduled_delete',        'wp_scheduled_delete'            );
     232
     233// Navigation menu actions
    232234add_action( 'trash_post',                 '_wp_trash_menu_item'            );
    233235add_action( 'untrash_post',               '_wp_untrash_menu_item'          );
    234236add_action( 'delete_post',                '_wp_delete_post_menu_item'      );
    235237add_action( 'delete_term',                '_wp_delete_tax_menu_item'       );
     238add_action( 'transition_post_status', '_wp_auto_add_pages_to_menu',  10, 3 );
    236239
    237240// Post Thumbnail CSS class filtering

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

    r15008 r15078  
    1717 *
    1818 * @param string $menu Menu id, slug or name
    19  * @return mixed false if $menu param isn't supplied or term does not exist, menu object if successfull
     19 * @return mixed false if $menu param isn't supplied or term does not exist, menu object if successful.
    2020 */
    2121function wp_get_nav_menu_object( $menu ) {
    … …  
    3131                $menu_obj = get_term_by( 'name', $menu, 'nav_menu' );
    3232
    33         if ( ! $menu_obj ) {
     33        if ( ! $menu_obj )
    3434                $menu_obj = false;
    35         }
    3635
    3736        return $menu_obj;
    … …  
    3938
    4039/**
    41  * Check if the given ID is a nav menu.
     40 * Check if the given ID is a navigation menu.
    4241 *
    4342 * Returns true if it is; false otherwise.
    … …  
    6665
    6766/**
    68  * Register nav menus for a theme.
     67 * Register navigation menus for a theme.
    6968 *
    7069 * @since 3.0.0
    … …  
    8180
    8281/**
    83  * Register nav menu for a theme.
     82 * Register a navigation menu for a theme.
    8483 *
    8584 * @since 3.0.0
    … …  
    9291}
    9392/**
    94  * Returns an array of all registered nav menus in a theme
     93 * Returns an array of all registered navigation menus in a theme
    9594 *
    9695 * @since 3.0.0
    … …  
    105104
    106105/**
    107  * Returns an array with the registered nav menu locations and the menu assigned to it
     106 * Returns an array with the registered navigation menu locations and the menu assigned to it
    108107 *
    109108 * @since 3.0.0
    … …  
    112111
    113112function get_nav_menu_locations() {
    114         return get_theme_mod('nav_menu_locations');
     113        return get_theme_mod( 'nav_menu_locations' );
    115114}
    116115
    … …  
    130129 * Create a Navigation Menu.
    131130 *
    132  * Optional args:
    133  * slug - the url friendly version of the nav menu.
    134  *
    135131 * @since 3.0.0
    136132 *
    137133 * @param string $menu_name Menu Name
    138  * @param string $args Optional.
    139  * @return mixed Menu object on sucess|WP_Error on failure
    140  */
    141 function wp_create_nav_menu( $menu_name, $args = array() ) {
     134 * @return mixed Menu object on success|WP_Error on failure
     135 */
     136function wp_create_nav_menu( $menu_name ) {
    142137        $menu_exists = get_term_by( 'name', $menu_name, 'nav_menu' );
    143138
    … …  
    152147        $result = get_term( $menu['term_id'], 'nav_menu' );
    153148
    154         if ( $result && !is_wp_error($result) ) {
     149        if ( $result && !is_wp_error($result) )
    155150                do_action( 'wp_create_nav_menu', $menu['term_id'] );
    156                 return $result;
    157         } else {
    158                 return $result;
    159         }
     151
     152        return $result;
    160153}
    161154
    … …  
    166159 *
    167160 * @param string $menu name|id|slug
    168  * @return mixed Menu object on sucess|WP_Error on failure
     161 * @return mixed Menu object on success|WP_Error on failure
    169162 */
    170163function wp_delete_nav_menu( $menu ) {
    171164        $menu = wp_get_nav_menu_object( $menu );
    172         if ( ! $menu  )
     165        if ( ! $menu )
    173166                return false;
    174167
    175168        $menu_objects = get_objects_in_term( $menu->term_id, 'nav_menu' );
    176         if ( !empty( $menu_objects ) ) {
     169        if ( ! empty( $menu_objects ) ) {
    177170                foreach ( $menu_objects as $item ) {
    178171                        wp_delete_post( $item );
    … …  
    182175        $result = wp_delete_term( $menu->term_id, 'nav_menu' );
    183176
    184         if ( $result && !is_wp_error($result) ) {
     177        if ( $result && !is_wp_error($result) )
    185178                do_action( 'wp_delete_nav_menu', $menu->term_id );
    186                 return $result;
    187         } else {
    188                 return $result;
    189         }
     179
     180        return $result;
    190181}
    191182
    … …  
    211202                return $_menu;
    212203
    213         if ( $_menu && isset( $_menu->term_id ) ) {
    214                 $args = array(
    215                         'description' => ( isset( $menu_data['description'] ) ? $menu_data['description'] : '' ),
    216                         'name' => ( isset( $menu_data['menu-name'] ) ? $menu_data['menu-name'] : '' ),
    217                         'parent' => ( isset( $menu_data['parent'] ) ? (int) $menu_data['parent'] : 0 ),
    218                         'slug' => null,
    219                 );
    220 
    221                 $menu_id = (int) $_menu->term_id;
    222 
    223                 // double-check that we're not changing a menu to the name of another
    224                 $_possible_existing = get_term_by( 'name', $menu_data['menu-name'], 'nav_menu' );
    225                 if (
    226                         $_possible_existing &&
    227                         ! is_wp_error( $_possible_existing ) &&
    228                         isset( $_possible_existing->term_id ) &&
    229                         $_possible_existing->term_id != $menu_id
    230                 ) {
    231                         return new WP_Error( 'menu_exists', sprintf( __('The menu name <strong>%s</strong> conflicts with another menu name. Please try another.'), esc_html( $menu_data['menu-name'] ) ) );
    232                 }
    233 
    234                 $update_response = wp_update_term( $menu_id, 'nav_menu', $args );
    235 
    236                 if ( ! is_wp_error( $update_response ) )
    237                         return $menu_id;
    238                 else
    239                         return $update_response;
    240         } else {
     204        if ( ! $_menu || ! isset( $_menu->term_id ) )
    241205                return 0;
    242         }
     206
     207        $args = array(
     208                'description' => ( isset( $menu_data['description'] ) ? $menu_data['description']  : '' ),
     209                'name'        => ( isset( $menu_data['menu-name']   ) ? $menu_data['menu-name']    : '' ),
     210                'parent'      => ( isset( $menu_data['parent']      ) ? (int) $menu_data['parent'] : 0  ),
     211                'slug'        => null,
     212        );
     213
     214        $menu_id = (int) $_menu->term_id;
     215
     216        // double-check that we're not changing a menu to the name of another
     217        $_possible_existing = get_term_by( 'name', $menu_data['menu-name'], 'nav_menu' );
     218        if (
     219                $_possible_existing &&
     220                ! is_wp_error( $_possible_existing ) &&
     221                isset( $_possible_existing->term_id ) &&
     222                $_possible_existing->term_id != $menu_id
     223        )
     224                return new WP_Error( 'menu_exists', sprintf( __('The menu name <strong>%s</strong> conflicts with another menu name. Please try another.'), esc_html( $menu_data['menu-name'] ) ) );
     225
     226        $update_response = wp_update_term( $menu_id, 'nav_menu', $args );
     227
     228        if ( ! is_wp_error( $update_response ) )
     229                return $menu_id;
     230        else
     231                return $update_response;
    243232}
    244233
    … …  
    316305                        $original_title = $original_object->post_title;
    317306
    318                         if ( 'trash' == get_post_status( $args['menu-item-object-id'] ) ) {
     307                        if ( 'trash' == get_post_status( $args['menu-item-object-id'] ) )
    319308                                return new WP_Error('update_nav_menu_item_failed', sprintf(__('The menu item "%1$s" belongs to something that is in the trash, so it cannot be updated.'), $args['menu-item-title'] ) );
    320                         }
    321309                }
    322310
    … …  
    325313
    326314                        // hack to get wp to create a post object when too many properties are empty
    327                         if ( empty( $args['menu-item-description'] ) ) {
     315                        if ( empty( $args['menu-item-description'] ) )
    328316                                $args['menu-item-description'] = ' ';
    329                         }
    330317                }
    331318        }
    … …  
    363350        }
    364351
    365         if ( $menu_item_db_id && ! is_wp_error( $menu_item_db_id ) ) {
    366 
    367                 $menu_item_db_id = (int) $menu_item_db_id;
    368 
    369                 update_post_meta( $menu_item_db_id, '_menu_item_type', sanitize_key($args['menu-item-type']) );
    370                 update_post_meta( $menu_item_db_id, '_menu_item_menu_item_parent', (int) $args['menu-item-parent-id'] );
    371                 update_post_meta( $menu_item_db_id, '_menu_item_object_id', (int) $args['menu-item-object-id'] );
    372                 update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key($args['menu-item-object']) );
    373                 update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key($args['menu-item-target']) );
    374                
    375                 $args['menu-item-classes'] = array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-classes'] ) );
    376                 $args['menu-item-xfn'] = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-xfn'] ) ) );
    377                 update_post_meta( $menu_item_db_id, '_menu_item_classes', $args['menu-item-classes'] );
    378                 update_post_meta( $menu_item_db_id, '_menu_item_xfn', $args['menu-item-xfn'] );
    379                 update_post_meta( $menu_item_db_id, '_menu_item_url', esc_url_raw($args['menu-item-url']) );
    380                
    381                 if ( 0 == $menu_id )
    382                         update_post_meta( $menu_item_db_id, '_menu_item_orphaned', time() );
    383                 else
    384                         delete_post_meta( $menu_item_db_id, '_menu_item_orphaned' );
    385 
    386                 do_action('wp_update_nav_menu_item', $menu_id, $menu_item_db_id, $args );
    387         }
     352        if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) )
     353                return $menu_item_db_id;
     354
     355        $menu_item_db_id = (int) $menu_item_db_id;
     356
     357        update_post_meta( $menu_item_db_id, '_menu_item_type', sanitize_key($args['menu-item-type']) );
     358        update_post_meta( $menu_item_db_id, '_menu_item_menu_item_parent', (int) $args['menu-item-parent-id'] );
     359        update_post_meta( $menu_item_db_id, '_menu_item_object_id', (int) $args['menu-item-object-id'] );
     360        update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key($args['menu-item-object']) );
     361        update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key($args['menu-item-target']) );
     362       
     363        $args['menu-item-classes'] = array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-classes'] ) );
     364        $args['menu-item-xfn'] = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-xfn'] ) ) );
     365        update_post_meta( $menu_item_db_id, '_menu_item_classes', $args['menu-item-classes'] );
     366        update_post_meta( $menu_item_db_id, '_menu_item_xfn', $args['menu-item-xfn'] );
     367        update_post_meta( $menu_item_db_id, '_menu_item_url', esc_url_raw($args['menu-item-url']) );
     368       
     369        if ( 0 == $menu_id )
     370                update_post_meta( $menu_item_db_id, '_menu_item_orphaned', time() );
     371        else
     372                delete_post_meta( $menu_item_db_id, '_menu_item_orphaned' );
     373
     374        do_action('wp_update_nav_menu_item', $menu_id, $menu_item_db_id, $args );
    388375
    389376        return $menu_item_db_id;
    … …  
    404391}
    405392
    406 
    407393/**
    408394 * Sort menu items by the desired key.
    … …  
    415401 * @return int -1, 0, or 1 if $a is considered to be respectively less than, equal to, or greater than $b.
    416402 */
    417 function _sort_nav_menu_items($a, $b) {
     403function _sort_nav_menu_items( $a, $b ) {
    418404        global $_menu_item_sort_prop;
    419405
    420         if ( empty( $_menu_item_sort_prop ) ) {
     406        if ( empty( $_menu_item_sort_prop ) )
    421407                return 0;
    422         }
    423 
    424         if ( isset( $a->$_menu_item_sort_prop ) && isset( $b->$_menu_item_sort_prop ) ) {
    425                 $_a = (int) $a->$_menu_item_sort_prop;
    426                 $_b = (int) $b->$_menu_item_sort_prop;
    427 
    428                 if ( $a->$_menu_item_sort_prop == $b->$_menu_item_sort_prop ) {
    429                         return 0;
    430                 } elseif (
    431                         ( $_a == $a->$_menu_item_sort_prop ) &&
    432                         ( $_b == $b->$_menu_item_sort_prop )
    433                 ) {
    434                         return $_a < $_b ? -1 : 1;
    435                 } else {
    436                         return strcmp( $a->$_menu_item_sort_prop, $b->$_menu_item_sort_prop );
    437                 }
    438         } else {
     408
     409        if ( ! isset( $a->$_menu_item_sort_prop ) || ! isset( $b->$_menu_item_sort_prop ) )
    439410                return 0;
    440         }
     411
     412        $_a = (int) $a->$_menu_item_sort_prop;
     413        $_b = (int) $b->$_menu_item_sort_prop;
     414
     415        if ( $a->$_menu_item_sort_prop == $b->$_menu_item_sort_prop )
     416                return 0;
     417        elseif ( $_a == $a->$_menu_item_sort_prop && $_b == $b->$_menu_item_sort_prop )
     418                return $_a < $_b ? -1 : 1;
     419        else
     420                return strcmp( $a->$_menu_item_sort_prop, $b->$_menu_item_sort_prop );
    441421}
    442422
    … …  
    465445                return $items;
    466446
    467         $defaults = array( 'order' => 'ASC', 'orderby' => 'menu_order', 'post_type' => 'nav_menu_item', 'post_status' => 'publish', 'output' => ARRAY_A, 'output_key' => 'menu_order', 'nopaging' => true,
    468                                           'update_post_term_cache' => false);
     447        $defaults = array( 'order' => 'ASC', 'orderby' => 'menu_order', 'post_type' => 'nav_menu_item',
     448                'post_status' => 'publish', 'output' => ARRAY_A, 'output_key' => 'menu_order', 'nopaging' => true,
     449                'update_post_term_cache' => false );
    469450        $args = wp_parse_args( $args, $defaults );
    470451        if ( count( $items ) > 1 )
    … …  
    479460
    480461        // Get all posts and terms at once to prime the caches
    481         if ( empty($fetched[$menu->term_id]) || $_wp_using_ext_object_cache ) {
     462        if ( empty( $fetched[$menu->term_id] ) || $_wp_using_ext_object_cache ) {
    482463                $fetched[$menu->term_id] = true;
    483464                $posts = array();
    … …  
    485466                foreach ( $items as $item ) {
    486467                        $object_id = get_post_meta( $item->ID, '_menu_item_object_id', true );
    487                         $object = get_post_meta( $item->ID, '_menu_item_object', true );
    488                         $type = get_post_meta( $item->ID, '_menu_item_type', true );
     468                        $object    = get_post_meta( $item->ID, '_menu_item_object',    true );
     469                        $type      = get_post_meta( $item->ID, '_menu_item_type',      true );
    489470
    490471                        if ( 'post_type' == $type )
    … …  
    494475                }
    495476
    496                 if ( !empty($posts) ) {
     477                if ( ! empty( $posts ) ) {
    497478                        foreach ( array_keys($posts) as $post_type ) {
    498479                                get_posts( array('post__in' => $posts[$post_type], 'post_type' => $post_type, 'nopaging' => true, 'update_post_term_cache' => false) );
    … …  
    501482                unset($posts);
    502483
    503                 if ( !empty($terms) ) {
     484                if ( ! empty( $terms ) ) {
    504485                        foreach ( array_keys($terms) as $taxonomy ) {
    505486                                get_terms($taxonomy, array('include' => $terms[$taxonomy]) );
    … …  
    656637        foreach( (array) $menu_items as $menu_item ) {
    657638                if ( isset( $menu_item->ID ) && is_nav_menu_item( $menu_item->ID ) ) {
    658                         if ( get_post_meta($menu_item->ID, '_menu_item_type', true) != $object_type )
     639                        if ( get_post_meta( $menu_item->ID, '_menu_item_type', true ) != $object_type )
    659640                                continue;
    660641
    … …  
    785766        }
    786767}
    787 add_action( 'transition_post_status', '_wp_auto_add_pages_to_menu', 10, 3 );
    788768
    789769?>
Note: See TracChangeset for help on using the changeset viewer.