WordPress.org

Make WordPress Core

Ticket #21297: 21297.2.diff

File 21297.2.diff, 2.8 KB (added by obenland, 6 years ago)

Adds visibility keywords

  • wp-content/themes/twentytwelve/inc/theme-options.php

     
    1212         * The option value in the database will be based on get_stylesheet()
    1313         * so child themes don't share the parent theme's option value.
    1414         */
    15         var $option_key = 'twentytwelve_theme_options';
     15        public $option_key = 'twentytwelve_theme_options';
    1616
    1717        /**
    1818         * Initialize our options.
    1919         */
    20         var $options = array();
     20        public $options = array();
    2121
    22         function Twenty_Twelve_Options() {
     22        public function __construct() {
    2323                // Set option key based on get_stylesheet()
    2424                if ( 'twentytwelve' != get_stylesheet() )
    2525                        $this->option_key = get_stylesheet() . '_theme_options';
     
    3838         * which is used when the option is saved, to ensure that our option values are properly
    3939         * formatted, and safe.
    4040         */
    41         function options_init() {
     41        public function options_init() {
    4242                // Load our options for use in any method.
    4343                $this->options = $this->get_theme_options();
    4444
     
    7272         *
    7373         * This function is attached to the admin_menu action hook.
    7474         */
    75         function add_page() {
     75        public function add_page() {
    7676                $theme_page = add_theme_page(
    7777                        __( 'Theme Options', 'twentytwelve' ), // Name of page
    7878                        __( 'Theme Options', 'twentytwelve' ), // Label in menu
     
    8585        /**
    8686         * Returns the default options.
    8787         */
    88         function get_default_theme_options() {
     88        public function get_default_theme_options() {
    8989                $default_theme_options = array(
    9090                        'enable_fonts' => false,
    9191                );
     
    9696        /**
    9797         * Returns the options array.
    9898         */
    99         function get_theme_options() {
     99        public function get_theme_options() {
    100100                return get_option( $this->option_key, $this->get_default_theme_options() );
    101101        }
    102102
    103103        /**
    104104         * Renders the enable fonts checkbox setting field.
    105105         */
    106         function settings_field_enable_fonts() {
     106        public function settings_field_enable_fonts() {
    107107                $options = $this->options;
    108108                ?>
    109109                <label for="enable-fonts">
     
    118118         *
    119119         * @uses get_current_theme() for back compat, fallback for < 3.4
    120120         */
    121         function render_page() {
     121        public function render_page() {
    122122                ?>
    123123                <div class="wrap">
    124124                        <?php screen_icon(); ?>
     
    142142         *
    143143         * @see options_init()
    144144         */
    145         function validate( $input ) {
     145        public function validate( $input ) {
    146146                $output = $defaults = $this->get_default_theme_options();
    147147
    148148                // The enable fonts checkbox should boolean true or false
     
    161161         *
    162162         * @since Twenty Twelve 1.0
    163163         */
    164         function customize_register( $wp_customize ) {
     164        public function customize_register( $wp_customize ) {
    165165                // Enable Web Fonts
    166166                $wp_customize->add_section( $this->option_key . '_enable_fonts', array(
    167167                        'title'    => __( 'Fonts', 'twentytwelve' ),