From c9f049392b3e1c92f76e6a71df8985bbc46c2db1 Mon Sep 17 00:00:00 2001 From: Robin Cornett Date: Mon, 20 Apr 2015 17:02:16 -0400 Subject: [PATCH] refactor section/page variables --- ...s-displayfeaturedimagegenesis-settings.php | 39 ++++++++++--------- 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/includes/class-displayfeaturedimagegenesis-settings.php b/includes/class-displayfeaturedimagegenesis-settings.php index c948483..09dbf80 100644 --- a/includes/class-displayfeaturedimagegenesis-settings.php +++ b/includes/class-displayfeaturedimagegenesis-settings.php @@ -80,8 +80,11 @@ class Display_Featured_Image_Genesis_Settings { $this->displaysetting = get_option( 'displayfeaturedimagegenesis', $defaults ); + $page = 'displayfeaturedimagegenesis'; + $section = 'display_featured_image_section'; + add_settings_section( - 'display_featured_image_section', + $section, __( 'Optional Sitewide Settings', 'display-featured-image-genesis' ), array( $this, 'section_description'), 'displayfeaturedimagegenesis' @@ -91,56 +94,56 @@ class Display_Featured_Image_Genesis_Settings { 'displayfeaturedimagegenesis[less_header]', '', array( $this, 'header_size' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[default]', '', array( $this, 'set_default_image' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[exclude_front]', '', array( $this, 'exclude_front' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[keep_titles]', '', array( $this, 'keep_titles' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[move_excerpts]', '', array( $this, 'move_excerpts' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[is_paged]', '', array( $this, 'check_is_paged' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); add_settings_field( 'displayfeaturedimagegenesis[feed_image]', '', array( $this, 'add_image_to_feed' ), - 'displayfeaturedimagegenesis', - 'display_featured_image_section' + $page, + $section ); $args = array( @@ -157,7 +160,7 @@ class Display_Featured_Image_Genesis_Settings { 'display_featured_image_custom_post_types', __( 'Featured Images for Custom Post Types', 'display-featured-image-genesis' ), array( $this, 'cpt_section_description' ), - 'displayfeaturedimagegenesis' + $page ); @@ -165,7 +168,7 @@ class Display_Featured_Image_Genesis_Settings { "displayfeaturedimagegenesis[post_types]", __( 'Featured Images', 'display-featured-image-genesis' ), array( $this, 'set_cpt_image' ), - 'displayfeaturedimagegenesis', + $page, 'display_featured_image_custom_post_types' ); @@ -356,7 +359,7 @@ class Display_Featured_Image_Genesis_Settings { $cat_keys = array_keys( $_POST['displayfeaturedimagegenesis'] ); $is_updated = false; foreach ( $cat_keys as $key ) { - if ( isset ( $_POST['displayfeaturedimagegenesis'][$key] ) ) { + if ( isset( $_POST['displayfeaturedimagegenesis'][$key] ) ) { $displaysetting[$key] = $_POST['displayfeaturedimagegenesis'][$key]; if ( $_POST['displayfeaturedimagegenesis']['term_image'] === $displaysetting[$key] ) { $displaysetting[$key] = $this->validate_taxonomy_image( $_POST['displayfeaturedimagegenesis'][$key] );