sergiotarxz/Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

In Twenty Eleven: Register the widget from functions.php. Kill create_function. Attach twentyeleven_widgets_init() to widgets_init, rather than init. Eliminate use of the $post global in the widget. see #17198.

Browse Source 
git-svn-id: https://develop.svn.wordpress.org/trunk@17738 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Andrew Nacin 2011-04-28 09:47:30 +00:00
parent 45677c6e26
commit f8a460675c
2 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
wp-content/themes/twentyeleven/functions.php
View File

@ -366,9 +366,14 @@ function twentyeleven_page_menu_args( $args ) {
add_filter( 'wp_page_menu_args', 'twentyeleven_page_menu_args' ); add_filter( 'wp_page_menu_args', 'twentyeleven_page_menu_args' );
/** /**
* Register widgetized area and update sidebar with default widgets * Register our sidebars and widgetized areas. Also register the default Epherma widget.
*
* @since Twenty Eleven 1.0
*/ */
function twentyeleven_widgets_init() { function twentyeleven_widgets_init() {
register_widget( 'Twenty_Eleven_Ephemera_Widget' );
register_sidebar( array( register_sidebar( array(
'name' => __( 'Main Sidebar', 'twentyeleven' ), 'name' => __( 'Main Sidebar', 'twentyeleven' ),
'id' => 'sidebar-1', 'id' => 'sidebar-1',
@ -418,7 +423,7 @@ function twentyeleven_widgets_init() {
'after_title' => '</h1>', 'after_title' => '</h1>',
) ); ) );
} }
add_action( 'init', 'twentyeleven_widgets_init' ); add_action( 'widgets_init', 'twentyeleven_widgets_init' );
/** /**
* Display navigation to next/previous pages when applicable * Display navigation to next/previous pages when applicable

8
wp-content/themes/twentyeleven/inc/widgets.php
View File

@ -78,13 +78,11 @@ class Twenty_Eleven_Ephemera_Widget extends WP_Widget {
echo $title; // Can set this with a widget option, or omit altogether echo $title; // Can set this with a widget option, or omit altogether
echo $after_title; echo $after_title;
global $post;
?> ?>
<ol> <ol>
<?php while ( $ephemera->have_posts() ) : $ephemera->the_post(); ?> <?php while ( $ephemera->have_posts() ) : $ephemera->the_post(); ?>
<?php if ( 'link' != get_post_format( $post->ID ) ) : ?> <?php if ( 'link' != get_post_format() ) : ?>
<li class="entry-title"> <li class="entry-title">
<a href="<?php the_permalink(); ?>" title="<?php printf( esc_attr__( 'Permalink to %s', 'twentyeleven' ), the_title_attribute( 'echo=0' ) ); ?>" rel="bookmark"><?php the_title(); ?></a> <a href="<?php the_permalink(); ?>" title="<?php printf( esc_attr__( 'Permalink to %s', 'twentyeleven' ), the_title_attribute( 'echo=0' ) ); ?>" rel="bookmark"><?php the_title(); ?></a>
@ -116,7 +114,7 @@ class Twenty_Eleven_Ephemera_Widget extends WP_Widget {
echo $after_widget; echo $after_widget;
// Reset the global $the_post as this query will have stomped on it // Reset the post globals as this query will have stomped on it
wp_reset_postdata(); wp_reset_postdata();
// end check for ephemeral posts // end check for ephemeral posts
@ -162,5 +160,3 @@ class Twenty_Eleven_Ephemera_Widget extends WP_Widget {
<?php <?php
} }
} }
add_action( 'widgets_init', create_function( '', "register_widget( 'Twenty_Eleven_Ephemera_Widget' );" ) );