Changeset 35337

Timestamp:

10/21/2015 05:38:44 PM (10 years ago)

Author:

ocean90

Message:

Options: Hide the week starts on setting for installs that have the default setting already.

The default setting is the value of $wp_locale->start_of_week which holds the value per locale, see [35336].

Props swissspidy, ocean90.
Fixes #28344.

Location:

trunk/src/wp-admin

Files:

• includes/options.php (modified) (1 diff)
• options-general.php (modified) (2 diffs)
• options.php (modified) (2 diffs)

trunk/src/wp-admin/includes/options.php

@@ -140 +140 @@
     echo '<p class="description">' . __( 'The <a href="https://codex.wordpress.org/Glossary#Character_set">character encoding</a> of your site (UTF-8 is recommended)' ) . '</p>';
 }
+
+/**
+ * Render the week starts on setting.
+ *
+ * @global WP_Locale $wp_locale
+ *
+ * @since 4.4.0
+ */
+function options_general_start_of_week() {
+    global $wp_locale;
+    ?>
+    <select name="start_of_week" id="start_of_week">
+        <?php
+        $start_of_week = get_option( 'start_of_week' );
+        for ( $day_index = 0; $day_index <= 6; $day_index++ ) {
+            echo "\n\t<option value='" . esc_attr( $day_index ) . "'" . selected( $start_of_week, $day_index, false ) . ">" . $wp_locale->get_weekday( $day_index ) . '</option>';
+        }
+        ?>
+    </select>
+    <?php
+}

trunk/src/wp-admin/options-general.php

@@ -54 +54 @@
 
 <form method="post" action="options.php" novalidate="novalidate">
-<?php settings_fields('general'); ?>
+<?php
+settings_fields( 'general' );
+
+/**
+ * @global WP_Locale $wp_locale
+ */
+global $wp_locale;
+if ( get_option( 'start_of_week' ) != $wp_locale->start_of_week ) {
+    add_settings_field( 'start_of_week', __( 'Week Starts On' ), 'options_general_start_of_week', 'general', 'default', array( 'label_for' => 'start_of_week' ) );
+}
+?>
 
 <table class="form-table">
@@ -275 +285 @@
 </td>
 </tr>
-<tr>
-<th scope="row"><label for="start_of_week"><?php _e('Week Starts On') ?></label></th>
-<td><select name="start_of_week" id="start_of_week">
-<?php
-/**
- * @global WP_Locale $wp_locale
- */
-global $wp_locale;
-
-for ($day_index = 0; $day_index <= 6; $day_index++) :
-    $selected = (get_option('start_of_week') == $day_index) ? 'selected="selected"' : '';
-    echo "\n\t<option value='" . esc_attr($day_index) . "' $selected>" . $wp_locale->get_weekday($day_index) . '</option>';
-endfor;
-?>
-</select></td>
-</tr>
-<?php do_settings_fields('general', 'default'); ?>
+
+<?php do_settings_fields( 'general', 'default' ); ?>
 
 <?php

trunk/src/wp-admin/options.php

@@ -83 +83 @@
 
 $whitelist_options = array(
-    'general' => array( 'blogname', 'blogdescription', 'gmt_offset', 'date_format', 'time_format', 'start_of_week', 'timezone_string', 'WPLANG' ),
+    'general' => array( 'blogname', 'blogdescription', 'gmt_offset', 'date_format', 'time_format', 'timezone_string', 'WPLANG' ),
     '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' ),
     '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' ),
@@ -93 +93 @@
 $mail_options = array('mailserver_url', 'mailserver_port', 'mailserver_login', 'mailserver_pass');
 
-if ( ! in_array( get_option( 'blog_charset' ), array( 'utf8', 'utf-8', 'UTF8', 'UTF-8' ) ) )
+if ( ! in_array( get_option( 'blog_charset' ), array( 'utf8', 'utf-8', 'UTF8', 'UTF-8' ) ) ) {
     $whitelist_options['reading'][] = 'blog_charset';
+}
+
+/**
+ * @global WP_Locale $wp_locale
+ */
+global $wp_locale;
+if ( get_option( 'start_of_week' ) != $wp_locale->start_of_week ) {
+    $whitelist_options['general'][] = 'start_of_week';
+}
 
 if ( get_site_option( 'initial_db_version' ) < 32453 ) {