WordPress.org

Plugin Directory

Changeset 614694


Ignore:
Timestamp:
10/19/12 19:30:53 (18 months ago)
Author:
MattyRob
Message:

Fixes for bugs in new Settings page

File:
1 edited

Legend:

Unmodified
Added
Removed
  • subscribe2/trunk/admin/settings.php

    r614682 r614694  
    3737                    $this->subscribe2_options[$key] = (int)$_POST[$key]; 
    3838                } 
    39             } elseif ( in_array($key, array('show_meta', 'show_button', 'ajax', 'widget', 'counterwidget', 's2meta_default', 'reg_override')) ){ 
     39            } elseif ( in_array($key, array('show_meta', 'show_button', 'ajax', 'widget', 'counterwidget', 's2meta_default', 'reg_override')) ) { 
    4040                // check box entries 
    4141                ( isset($_POST[$key]) && $_POST[$key] == '1' ) ? $newvalue = '1' : $newvalue = '0'; 
     
    4444                            // email subject and body templates 
    4545                $this->subscribe2_options[$key] = $_POST[$key]; 
    46             } elseif ( in_array($key, array('compulsory', 'category', 'format')) ) { 
    47                 if ( !empty($_POST[$key]) ) { 
    48                     sort($_POST[$key]); 
    49                     $newvalue = implode(',', $_POST[$key]); 
    50                 } else { 
    51                     $newvalue = ''; 
    52                 } 
    53                 if ( $key === 'category' ) { 
    54                     $this->subscribe2_options['exclude'] = $newvalue; 
    55                 } elseif ($key === 'format') { 
     46            } elseif ( in_array($key, array('compulsory', 'exclude', 'format')) ) { 
     47                sort($_POST[$key]); 
     48                $newvalue = implode(',', $_POST[$key]); 
     49 
     50                if ($key === 'format') { 
    5651                    $this->subscribe2_options['exclude_formats'] = $newvalue; 
    5752                } else { 
    5853                    $this->subscribe2_options[$key] = $newvalue; 
     54                } 
     55            } elseif ( $key === 'registered_users_tab' ) { 
     56                $options = array('compulsory', 'exclude', 'format', 'reg_override'); 
     57                foreach ( $options as $option ) { 
     58                    if ( !isset($_POST[$option]) ) { 
     59                        if ($option === 'format') { 
     60                            $this->subscribe2_options['exclude_formats'] = ''; 
     61                        } else { 
     62                            $this->subscribe2_options[$option] = ''; 
     63                        } 
     64                    } 
    5965                } 
    6066            } elseif ( $key === 'email_freq' ) { 
     
    272278    case 'registered': 
    273279        // compulsory categories 
     280        echo "<input type=\"hidden\" name=\"registered_users_tab\" value=\"options\" />\r\n"; 
    274281        echo "<div class=\"s2_admin\" id=\"s2_compulsory_categories\">\r\n"; 
    275282        echo "<h3>" . __('Compulsory Categories', 'subscribe2') . "</h3>\r\n"; 
     
    286293        echo "<strong><em style=\"color: red\">" . __('Posts assigned to any Excluded Category do not generate notifications and are not included in digest notifications', 'subscribe2') . "</em></strong><br />\r\n"; 
    287294        echo "</p>"; 
    288         $this->display_category_form(explode(',', $this->subscribe2_options['exclude'])); 
     295        $this->display_category_form(explode(',', $this->subscribe2_options['exclude']), 1, array(), 'exclude'); 
    289296        echo "<p style=\"text-align: center;\"><label><input type=\"checkbox\" name=\"reg_override\" value=\"1\"" . checked($this->subscribe2_options['reg_override'], '1', false) . " /> "; 
    290297        echo __('Allow registered users to subscribe to excluded categories?', 'subscribe2') . "</label></p>\r\n"; 
Note: See TracChangeset for help on using the changeset viewer.