php - Defining a private variable in a class -


i have following class

    class pietergoosen_widgets {      public function __construct() {          add_action( 'widgets_init', array( $this, 'sidebars_register' ) );         add_action('add_meta_boxes', array( $this, 'add_cspp' ) );         add_action('save_post', array( $this, 'save_cspp' ) );      }      public function sidebars_register() {          $mws = array (             'sidebar-2' => array (                 __( 'main sidebar', 'pietergoosen' ) => __( 'main sidebar website pages', 'pietergoosen' ),             ),             rest of code not constructive question             )         );          foreach ( $mws $mi => $mw ) {             foreach ($mw $mwn => $mwd) {                 register_sidebar(                     array (                             'name'          =>  $mwn,                             'id'            =>  $mi,                             'description'   =>  $mwd,                             'before_widget' => '<aside id="%1$s" class="widget %2$s">',                             'after_widget'  => '</aside>',                             'before_title'  => '<h1 class="widget-title">',                             'after_title'   => '</h1>',                     )                 );             }         }          $options = pietergoosen_get_theme_options();             global $sdwas;          if(!empty($options['_custom_sidebar_per_page']))             $sdwas = $options['_custom_sidebar_per_page'];          if(!empty($sdwas) && sizeof($sdwas) > 0) {               foreach($sdwas $sid => $sdwa) {                 $sid = self::sbslug($sdwa, 45);                  register_sidebar(                     array (                         'name'          => $sdwa,                         'id'            => $sid,                         'description'   => __( 'page specific sidebars can chosen per page', 'pietergoosen' ),                         'before_widget' => '<aside id="%1$s" class="widget %2$s">',                         'after_widget'  => '</aside>',                         'before_title'  => '<h1 class="widget-title">',                         'after_title'   => '</h1>',                     )                 );             }         }     }      public function sbslug($phrase, $maxlength) {         $result = strtolower($phrase);          $result = preg_replace("/[^a-z0-9\s-]/", "", $result);         $result = trim(preg_replace("/[\s-]+/", " ", $result));         $result = trim(substr($result, 0, $maxlength));         $result = preg_replace("/\s/", "-", $result);          return $result;     }      public function add_cspp() {          add_meta_box(              'custom_sidebar_per_page',              __( 'sidebar options', 'pietergoosen' ),              array( $this, 'cspp_link' ),             'page',              'side',              'default'         ); }      public function cspp_link( $post ) {          global $sdwas;          $custom = get_post_custom($post->id);          if(!empty($custom['_custom_sidebar_per_page']))             $val = $custom['_custom_sidebar_per_page'][0];         else             $val = "default";          // actual fields data entry         $output = '<p><label for="pietergoosen_new_field">'.__( 'choose sidebar display', 'pietergoosen' ).'</label></p>';         $output .= '<select name="custom_sidebar_per_page">';          // add default option         $output .= '<option';         if($val == "default")             $output .= ' selected="selected"';         $output .= ' value="default">'.__( 'no specified sidebar', 'pietergoosen' ).'</option>';          // fill select element registered sidebars         if(!empty($sdwas))             foreach($sdwas $sid => $sdwa) {                   $output .= '<option';                   if($sdwa == $val)                       $output .= ' selected="selected"';                   $output .= ' value="'.$sdwa.'">'.$sdwa.'</option>';               }            $output .= '</select>';            echo $output;       }        public function save_cspp($post_id){          if ( defined('doing_autosave') && doing_autosave )          return $post_id;          if ( !current_user_can( 'edit_page', $post_id ) )             return;          if(!empty( $_post['custom_sidebar_per_page'] ))          update_post_meta($post_id, '_custom_sidebar_per_page', $_post['custom_sidebar_per_page']);     }  }   $pgsidebar = new pietergoosen_widgets();

i need set $sdwas private variable use across class' public functions, i'm not sure how achieve that. used global $sdwas; set variable, know isn't realy correct way

any suggestions

i recommend making $sdwas private member of class.

at top, can declare such:

class pietergoosen_widgets {     private $sdwas;     //rest of code

then, access within functions, use $this->sdwas:

$sdwas = $options['_custom_sidebar_per_page'];

becomes:

$this->sdwas = $options['_custom_sidebar_per_page'];

make sure remove globals not needing them.


Comments

Post a Comment

Popular posts from this blog

apache - Remove .php and add trailing slash in url using htaccess not loading css -

i have add following code htaccess file removing .php extension , add trailing slash urls. worked fine reduced page loading speed , css code not loading @ all. pages ugly. please help rewriteengine on rewritebase / ## hide .php extension snippet # externally redirect /dir/foo.php /dir/foo rewritecond %{the_request} ^[a-z]{3,}\s([^.]+)\.php [nc] rewriterule ^ %1/ [r,l] # add trailing slash rewritecond %{request_filename} !-f rewritecond %{request_uri} !/$ rewriterule . %{request_uri}/ [l,r=301] # internally forward /dir/foo /dir/foo.php rewritecond %{request_filename} !-d rewritecond %{request_filename}.php -f rewriterule ^(.*?)/?$ $1.php [l] we can rewrite urls using .htaccess files. before writing in .htaccess files need verify kind of server using. can check server adding a <?php phpinfo(); ?> in php page. ensure mod_rewrite module enabled in apache server. can identified checking phpinfo included page. common methods used url rewriting follows ...
Read more

javascript - jQuery show full size image on click -

pretty self explanatory not workking . i want display full image when user click on image... yes know there tons of plugins but i want keep simple. any idea ? css : .full { display: none; width: 500px; height: 500px; margin-left: 100px; margin-top: -300px; position: fixed; background-color: #f00; z-index: 6; } .thumbnail { width: 50px; height: 50px; } html: <img class="thumbnail" src="http://4.bp.blogspot.com/-uhbdp2esxng/txneptvi92i/aaaaaaaaaky/upfqis7zto0/s1600/rihanna-wallpaper.jpg"> jquery: $(document).ready(function() { $('.thumbnail').on('click', function() { var img = $('<img />',{src:this.src,'class':'full'}); $('.full').html(img).show(); console.log('done!'); }); }); try looking how jquery works. $(document).ready(function() { $('.thumbnail').on('click', function(...
Read more