WordPress-C
/
WordPress
mirror of https://github.com/WordPress/WordPress.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Eliminate use of `extract()` in `wp_dropdown_pages()`. 

Adds unit tests to: `tests/post/template.php`. 
There was previously only one wimpy assertion for `wp_dropdown_pages()`.
	
See #22400.

Built from https://develop.svn.wordpress.org/trunk@28399


git-svn-id: http://core.svn.wordpress.org/trunk@28227 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Scott Taylor 2014-05-15 01:12:13 +00:00
parent fe2d90ab4f
commit 29d8832625
1 changed files with 20 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
wp-includes/post-template.php
View File

@ -918,7 +918,7 @@ function the_meta() {
* @param array|string $args Optional. Override default arguments. * @param array|string $args Optional. Override default arguments.
* @return string HTML content, if not displaying. * @return string HTML content, if not displaying.
*/ */
function wp_dropdown_pages($args = '') { function wp_dropdown_pages( $args = '' ) {
$defaults = array( $defaults = array(
'depth' => 0, 'child_of' => 0, 'depth' => 0, 'child_of' => 0,
'selected' => 0, 'echo' => 1, 'selected' => 0, 'echo' => 1,
@ -928,21 +928,23 @@ function wp_dropdown_pages($args = '') {
); );
$r = wp_parse_args( $args, $defaults ); $r = wp_parse_args( $args, $defaults );
extract( $r, EXTR_SKIP );
$pages = get_pages($r); $pages = get_pages( $r );
$output = ''; $output = '';
// Back-compat with old system where both id and name were based on $name argument // Back-compat with old system where both id and name were based on $name argument
if ( empty($id) ) if ( empty( $r['id'] ) ) {
$id = $name; $r['id'] = $r['name'];
}
if ( ! empty($pages) ) { if ( ! empty( $pages ) ) {
$output = "<select name='" . esc_attr( $name ) . "' id='" . esc_attr( $id ) . "'>\n"; $output = "<select name='" . esc_attr( $r['name'] ) . "' id='" . esc_attr( $r['id'] ) . "'>\n";
if ( $show_option_no_change ) if ( $r['show_option_no_change'] ) {
$output .= "\t<option value=\"-1\">$show_option_no_change</option>"; $output .= "\t<option value=\"-1\">" . $r['show_option_no_change'] . "</option>\n";
if ( $show_option_none ) }
$output .= "\t<option value=\"" . esc_attr($option_none_value) . "\">$show_option_none</option>\n"; if ( $r['show_option_none'] ) {
$output .= walk_page_dropdown_tree($pages, $depth, $r); $output .= "\t<option value=\"" . esc_attr( $r['option_none_value'] ) . '">' . $r['show_option_none'] . "</option>\n";
}
$output .= walk_page_dropdown_tree( $pages, $r['depth'], $r );
$output .= "</select>\n"; $output .= "</select>\n";
} }
@ -951,14 +953,14 @@ function wp_dropdown_pages($args = '') {
* *
* @since 2.1.0 * @since 2.1.0
* *
* @param string $output HTML output for drop down list of pages. * @param string $html HTML output for drop down list of pages.
*/ */
$output = apply_filters( 'wp_dropdown_pages', $output ); $html = apply_filters( 'wp_dropdown_pages', $output );
if ( $echo ) if ( $r['echo'] ) {
echo $output; echo $html;
}
return $output; return $html;
} }
/** /**