WordPress.org
Get WordPress

Make WordPress Core

Changeset 21838


Ignore:
Timestamp:
09/13/2012 05:28:57 PM (12 years ago)
Author:
nacin
Message:

Fold Privacy Settings into Reading Settings, moving blog_public (search engine/robots) to options-reading and removing options-privacy.

When blog_public only has two values (as judged by whether the blog_privacy_selector action is used), convert from radio buttons to a checkbox, and rename from 'Site Visibility' to a more specific 'Search Engine Visibility'.

The text and implementation may change a bit. see #16416.

Location:
trunk/wp-admin
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/includes/dashboard.php

    r21789 r21838  
    393393    echo '</p>';
    394394
    395     // Check if search engines are blocked.
     395    // Check if search engines are asked not to index this site.
    396396    if ( !is_network_admin() && !is_user_admin() && current_user_can('manage_options') && '1' != get_option('blog_public') ) {
    397397        $title = apply_filters('privacy_on_link_title', __('Your site is asking search engines not to index its content') );
    398         $content = apply_filters('privacy_on_link_text', __('Search Engines Blocked') );
    399 
    400         echo "<p><a href='options-privacy.php' title='$title'>$content</a></p>";
     398        $content = apply_filters('privacy_on_link_text', __('Search Engines Discouraged') );
     399
     400        echo "<p><a href='options-reading.php' title='$title'>$content</a></p>";
    401401    }
    402402
    … …  
    12611261        <p><?php _e( 'Here are a few easy things you can do to get your feet wet. Make sure to click Save on each Settings screen.' ); ?></p>
    12621262        <ul>
    1263         <li><?php echo sprintf( __( '<a href="%s">Choose your privacy setting</a>' ), esc_url( admin_url('options-privacy.php') ) ); ?></li>
    12641263        <li><?php echo sprintf( __( '<a href="%s">Select your tagline and time zone</a>' ), esc_url( admin_url('options-general.php') ) ); ?></li>
    12651264        <li><?php echo sprintf( __( '<a href="%s">Turn comments on or off</a>' ), esc_url( admin_url('options-discussion.php') ) ); ?></li>

  • trunk/wp-admin/includes/plugin.php

    r21812 r21838  
    16251625
    16261626    if ( 'misc' == $option_group ) {
    1627         _deprecated_argument( __FUNCTION__, '3.0', __( 'The miscellaneous options group has been removed. Use another settings group.' ) );
     1627        _deprecated_argument( __FUNCTION__, '3.0', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'misc' ) );
    16281628        $option_group = 'general';
     1629    }
     1630
     1631    if ( 'privacy' == $option_group ) {
     1632        _deprecated_argument( __FUNCTION__, '3.5', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'privacy' ) );
     1633        $option_group = 'reading';
    16291634    }
    16301635
    … …  
    16481653
    16491654    if ( 'misc' == $option_group ) {
    1650         _deprecated_argument( __FUNCTION__, '3.0', __( 'The miscellaneous options group has been removed. Use another settings group.' ) );
     1655        _deprecated_argument( __FUNCTION__, '3.0', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'misc' ) );
    16511656        $option_group = 'general';
     1657    }
     1658
     1659    if ( 'privacy' == $option_group ) {
     1660        _deprecated_argument( __FUNCTION__, '3.5', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'privacy' ) );
     1661        $option_group = 'reading';
    16521662    }
    16531663

  • trunk/wp-admin/includes/template.php

    r21791 r21838  
    10411041
    10421042    if ( 'misc' == $page ) {
    1043         _deprecated_argument( __FUNCTION__, '3.0', __( 'The miscellaneous options group has been removed. Use another settings group.' ) );
     1043        _deprecated_argument( __FUNCTION__, '3.0', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'misc' ) );
    10441044        $page = 'general';
     1045    }
     1046
     1047    if ( 'privacy' == $page ) {
     1048        _deprecated_argument( __FUNCTION__, '3.5', sprintf( __( 'The "%s" options group has been removed. Use another settings group.' ), 'privacy' ) );
     1049        $page = 'reading';
    10451050    }
    10461051
    … …  
    10831088        _deprecated_argument( __FUNCTION__, '3.0', __( 'The miscellaneous options group has been removed. Use another settings group.' ) );
    10841089        $page = 'general';
     1090    }
     1091
     1092    if ( 'privacy' == $page ) {
     1093        _deprecated_argument( __FUNCTION__, '3.5', __( 'The privacy options group has been removed. Use another settings group.' ) );
     1094        $page = 'reading';
    10851095    }
    10861096

  • trunk/wp-admin/menu.php

    r21200 r21838  
    207207    $submenu['options-general.php'][25] = array(__('Discussion'), 'manage_options', 'options-discussion.php');
    208208    $submenu['options-general.php'][30] = array(__('Media'), 'manage_options', 'options-media.php');
    209     $submenu['options-general.php'][35] = array(__('Privacy'), 'manage_options', 'options-privacy.php');
    210209    $submenu['options-general.php'][40] = array(__('Permalinks'), 'manage_options', 'options-permalink.php');
    211210

  • trunk/wp-admin/options-privacy.php

    r19507 r21838  
    1 <?php
    2 /**
    3  * Privacy Options Settings Administration Screen.
    4  *
    5  * @package WordPress
    6  * @subpackage Administration
    7  */
    8 
    9 /** Load WordPress Administration Bootstrap */
    10 require_once('./admin.php');
    11 
    12 if ( ! current_user_can( 'manage_options' ) )
    13     wp_die( __( 'You do not have sufficient permissions to manage options for this site.' ) );
    14 
    15 $title = __('Privacy Settings');
    16 $parent_file = 'options-general.php';
    17 
    18 get_current_screen()->add_help_tab( array(
    19     'id'      => 'overview',
    20     'title'   => __('Overview'),
    21     'content' => '<p>' . __('You can choose whether or not your site will be crawled by robots, ping services, and spiders. If you want those services to ignore your site, click the radio button next to &#8220;Ask search engines not to index this site&#8221; and click the Save Changes button at the bottom of the screen. Note that your privacy is not complete; your site is still visible on the web.') . '</p>' .
    22         '<p>' . __('When this setting is in effect a reminder is shown in the Right Now box of the Dashboard that says, &#8220;Search Engines Blocked,&#8221; to remind you that your site is not being crawled.') . '</p>',
    23 ) );
    24 
    25 get_current_screen()->set_help_sidebar(
    26     '<p><strong>' . __('For more information:') . '</strong></p>' .
    27     '<p>' . __('<a href="http://codex.wordpress.org/Settings_Privacy_Screen" target="_blank">Documentation on Privacy Settings</a>') . '</p>' .
    28     '<p>' . __('<a href="http://wordpress.org/support/" target="_blank">Support Forums</a>') . '</p>'
    29 );
    30 
    31 include('./admin-header.php');
    32 ?>
    33 
    34 <div class="wrap">
    35 <?php screen_icon(); ?>
    36 <h2><?php echo esc_html( $title ); ?></h2>
    37 
    38 <form method="post" action="options.php">
    39 <?php settings_fields('privacy'); ?>
    40 
    41 <table class="form-table">
    42 <tr valign="top">
    43 <th scope="row"><?php _e( 'Site Visibility' ); ?> </th>
    44 <td><fieldset><legend class="screen-reader-text"><span><?php _e( 'Site Visibility' ); ?> </span></legend>
    45 <input id="blog-public" type="radio" name="blog_public" value="1" <?php checked('1', get_option('blog_public')); ?> />
    46 <label for="blog-public"><?php _e( 'Allow search engines to index this site.' );?></label><br/>
    47 <input id="blog-norobots" type="radio" name="blog_public" value="0" <?php checked('0', get_option('blog_public')); ?> />
    48 <label for="blog-norobots"><?php _e( 'Ask search engines not to index this site.' ); ?></label>
    49 <p class="description"><?php _e( 'Note: Neither of these options blocks access to your site &mdash; it is up to search engines to honor your request.' ); ?></p>
    50 <?php do_action('blog_privacy_selector'); ?>
    51 </fieldset></td>
    52 </tr>
    53 <?php do_settings_fields('privacy', 'default'); ?>
    54 </table>
    55 
    56 <?php do_settings_sections('privacy'); ?>
    57 
    58 <?php submit_button(); ?>
    59 </form>
    60 
    61 </div>
    62 
    63 <?php include('./admin-footer.php') ?>

  • trunk/wp-admin/options-reading.php

    r21653 r21838  
    5151) );
    5252
     53get_current_screen()->add_help_tab( array(
     54    'id'      => 'site-visibility',
     55    'title'   => has_action( 'blog_privacy_selector' ) ? __( 'Site Visibility' ) : __( 'Search Engine Visibility' ),
     56    'content' => '<p>' . __( 'You can choose whether or not your site will be crawled by robots, ping services, and spiders. If you want those services to ignore your site, click the radio button next to &#8220;Ask search engines not to index this site&#8221; and click the Save Changes button at the bottom of the screen. Note that your privacy is not complete; your site is still visible on the web.' ) . '</p>' .
     57        '<p>' . __( 'When this setting is in effect a reminder is shown in the Right Now box of the Dashboard that says, &#8220;Search Engines Discouraged,&#8221; to remind you that your site is not being crawled.' ) . '</p>',
     58) );
     59
    5360get_current_screen()->set_help_sidebar(
    5461    '<p><strong>' . __('For more information:') . '</strong></p>' .
    … …  
    6471<h2><?php echo esc_html( $title ); ?></h2>
    6572
    66 <form name="form1" method="post" action="options.php">
     73<form method="post" action="options.php">
    6774<?php
    6875settings_fields( 'reading' );
    … …  
    133140</tr>
    134141
     142<tr valign="top">
     143<th scope="row"><?php has_action( 'blog_privacy_selector' ) ? _e( 'Site Visibility' ) : _e( 'Search Engine Visibility' ); ?> </th>
     144<td><fieldset><legend class="screen-reader-text"><span><?php has_action( 'blog_privacy_selector' ) ? _e( 'Site Visibility' ) : _e( 'Search Engine Visibility' ); ?> </span></legend>
     145<?php if ( has_action( 'blog_privacy_selector' ) ) : ?>
     146    <input id="blog-public" type="radio" name="blog_public" value="1" <?php checked('1', get_option('blog_public')); ?> />
     147    <label for="blog-public"><?php _e( 'Allow search engines to index this site.' );?></label><br/>
     148    <input id="blog-norobots" type="radio" name="blog_public" value="0" <?php checked('0', get_option('blog_public')); ?> />
     149    <label for="blog-norobots"><?php _e( 'Ask search engines not to index this site.' ); ?></label>
     150    <p class="description"><?php _e( 'Note: Neither of these options blocks access to your site &mdash; it is up to search engines to honor your request.' ); ?></p>
     151    <?php do_action('blog_privacy_selector'); ?>
     152<?php else : ?>
     153    <label for="blog_public"><input name="blog_public" type="checkbox" id="blog_public" value="1" <?php checked( '1', get_option( 'blog_public' ) ); ?> />
     154    <?php _e( 'Allow search engines to index this site' ); ?>
     155    <p class="description"><?php _e( 'It is up to search engines to honor a request to not index this site.' ); ?></p>
     156<?php endif; ?>
     157</fieldset></td>
     158</tr>
     159
    135160<?php do_settings_fields( 'reading', 'default' ); ?>
    136161</table>

  • trunk/wp-admin/options-writing.php

    r21818 r21838  
    183183<?php else : ?>
    184184
    185     <p><?php printf(__('WordPress is not notifying any <a href="http://codex.wordpress.org/Update_Services">Update Services</a> because of your site&#8217;s <a href="%s">privacy settings</a>.'), 'options-privacy.php'); ?></p>
     185    <p><?php printf(__('WordPress is not notifying any <a href="http://codex.wordpress.org/Update_Services">Update Services</a> because of your site&#8217;s <a href="%s">visibility settings</a>.'), 'options-reading.php'); ?></p>
    186186
    187187<?php endif; ?>

  • trunk/wp-admin/options.php

    r21818 r21838  
    6363    'discussion' => array( 'default_pingback_flag', 'default_ping_status', 'default_comment_status', 'comments_notify', 'moderation_notify', 'comment_moderation', 'require_name_email', 'comment_whitelist', 'comment_max_links', 'moderation_keys', 'blacklist_keys', 'show_avatars', 'avatar_rating', 'avatar_default', 'close_comments_for_old_posts', 'close_comments_days_old', 'thread_comments', 'thread_comments_depth', 'page_comments', 'comments_per_page', 'default_comments_page', 'comment_order', 'comment_registration' ),
    6464    'media' => array( 'thumbnail_size_w', 'thumbnail_size_h', 'thumbnail_crop', 'medium_size_w', 'medium_size_h', 'large_size_w', 'large_size_h', 'image_default_size', 'image_default_align', 'image_default_link_type', 'embed_autourls', 'embed_size_w', 'embed_size_h' ),
    65     'privacy' => array( 'blog_public' ),
    66     'reading' => array( 'posts_per_page', 'posts_per_rss', 'rss_use_excerpt', 'blog_charset', 'show_on_front', 'page_on_front', 'page_for_posts' ),
    67     'writing' => array( 'default_post_edit_rows', 'use_smilies', 'default_category', 'default_email_category', 'use_balanceTags', 'default_link_category', 'default_post_format' ),
    68     'options' => array( '' ) );
     65    'reading' => array( 'posts_per_page', 'posts_per_rss', 'rss_use_excerpt', 'blog_charset', 'show_on_front', 'page_on_front', 'page_for_posts', 'blog_public' ),
     66    'writing' => array( 'default_post_edit_rows', 'use_smilies', 'default_category', 'default_email_category', 'use_balanceTags', 'default_link_category', 'default_post_format' )
     67);
     68$whitelist_options['misc'] = $whitelist_options['options'] = $whitelist_options['privacy'] = array();
    6969
    7070$mail_options = array('mailserver_url', 'mailserver_port', 'mailserver_login', 'mailserver_pass');
    … …  
    9191    if ( apply_filters( 'enable_post_by_email_configuration', true ) )
    9292        $whitelist_options['writing'] = array_merge($whitelist_options['writing'], $mail_options);
    93 
    94     $whitelist_options[ 'misc' ] = array();
    9593}
    9694
Note: See TracChangeset for help on using the changeset viewer.