WordPress、PHPの$_POSTの入力値チェックについて。

Question

いつもありがとうございます。

下記の行がサニタイズ≒値チェックできてないとのことですが、保存するデータはそれ以降の行でチェックを行っているつもりです。下記の行について、どういった処理が必要でしょうか？

$this->options = $_POST[$this->text_domain];

That sets your option name to be a NON sanitized value.

function admin_setting_form() {
	$post_types = wp_list_filter( get_post_types(array('public'=>true)),array('attachment'), 'NOT' );
	?>
	<div class="wrap">
		<h2><?php echo esc_attr($this->name); ?> &raquo; <?php _e('Settings'); ?></h2>
		<form id="<?php echo esc_attr($this->text_domain); ?>" method="post" action="">
			<?php wp_nonce_field( 'csr-nonce-key', 'csr-key' ); ?>
            <h3><?php _e('有効にする投稿タイプを選択してください'); ?></h3>
			<?php
				foreach ( $post_types  as $post_type ) {
			?>
			<p>
				<strong><?php echo esc_attr($post_type); ?>ページ上で有効にします</strong>
                <input type="checkbox" name="<?php echo esc_attr($this->text_domain); ?>[post_type][<?php echo esc_attr($post_type); ?>]"  value="1" <?php if( isset( $this->options['post_type'][$post_type] ) && $this->options['post_type'][$post_type] == '1' ) echo 'checked'; ?> />
            </p>
            <?php 
        		} 
            ?>
            <h3>コメントの入力から外したい要素を選択</h3>
            <p>
				<strong>URLを外す</strong>
                <input type="checkbox" name="<?php echo esc_attr($this->text_domain); ?>[url]"  value="1" <?php if( isset( $this->options['url'] ) && $this->options['url'] == '1' ) echo 'checked'; ?> />
           	</p>
           	<p>
				<strong>メールアドレスを外す</strong>
                <input type="checkbox" name="<?php echo esc_attr($this->text_domain); ?>[email]"  value="1" <?php if( isset( $this->options['email'] ) && $this->options['email'] == '1' ) echo 'checked'; ?> />
            </p>
		    <p class="submit">
		    	<input class="button-primary" type="submit" name='save' value='<?php _e('Save Changes') ?>' />
		    </p>
		</form>
	</div>
	<?php
}
// save
function admin_save_options() {
    $post_types = wp_list_filter( get_post_types(array('public'=>true)),array('attachment'), 'NOT' );
    if (isset($_POST['save'])) {
		if( check_admin_referer( 'csr-nonce-key', 'csr-key' ) ) {
        	if (isset($_POST[$this->text_domain]) && is_array($_POST[$this->text_domain]) ) {
        		$this->options = $_POST[$this->text_domain];
        		// post_type sanitai
        		foreach ( $post_types as $post_type  ) {
        			if( isset($this->options['post_type'][$post_type]) ) {
	        			$this->options['post_type'][$post_type] = '1';
	        		}
        		}
        		if( isset( $this->options['url'] ) && $this->options['url'] == '1' ) {
        			$this->options['url'] = '1';
        		}
        		if( isset( $this->options['email'] ) && $this->options['email'] == '1' ) {
        			$this->options['email'] = '1';
        		}
        		update_option($this->text_domain, $this->options );
			}
			//wp_safe_redirect( menu_page_url( $this->text_domain, false ) );
        }
    }
    $this->options = get_option($this->text_domain);
	$this->admin_setting_form();
}

Answer 1

filter_input()が手軽でラクかと

Comments

[matometaru]

ありがとうございます。しかし、
Warning: filter_input() expects parameter 2 to be string, array given in
となりました。

if (isset($_POST[$this->text_domain]) && is_array($_POST[$this->text_domain]) ) {
ひとつ前のこのチェックではだめなのでしょうか？