2

codeIgniter と ReCaptcha を使用すると、奇妙な動作が発生します。

ReCaptcha を単純なアプリケーション/ヘルパーとして使用しました。私が行った唯一の変更は、API「recaptcha_helper.php」の名前を変更し、追加したことです

if(!defined('RECAPTCHA")){ 
   define('RECAPTCHA',true);
   [API code]
}

コントローラーにデータを投稿すると、結果は... 期待どおりではありません。

ReCaptcha valid / form valid = works fine!
ReCaptcha valid / form not valid = works fine!
ReCaptcha not valid / form valid = all form data lost
ReCaptcha not valid / form not valid = all form data lost + validation lost

また、私が作成した他のすべての Web サイトと同様に、「set_value('input_name')」を使用します。今日まで、レキャプチャをフォームに入れるまで、それは魅力のように機能していました。

コントローラーは次のとおりです。

<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');

class Register extends MY_Controller {


    var $form_rules = array(    
                        array(
                            'field' => 'user_first_name',
                            'label' => 'Prénom ',
                            'rules' => 'trim|required|max_lenght[50]|xss_clean'),
                        array(
                            'field' => 'user_last_name',
                            'label' => 'Nom ',
                            'rules' => 'trim|required|max_lenght[50]|xss_clean'),
                        array(
                            'field' => 'user_email',
                            'label' => 'Courriel ',
                            'rules' => 'trim|required|max_lenght[100]|valid_email|xss_clean'),
                        array(
                            'field' => 'user_email_confirm',
                            'label' => 'Confirmation du courriel ',
                            'rules' => 'trim|required|max_lenght[100]|matches[user_email]|xss_clean'));

    public function __construct(){
        parent::__construct();
    }
    public function register(){

        $this->load->library('form_validation');
        $this->form_validation->set_rules($this->form_rules);

        // RECAPTCHA STUFF
        $this->load->helper('recaptcha');
        $publickey = "****";
        $privatekey = "****";
        # the response from reCAPTCHA
        $resp = null;
        # the error code from reCAPTCHA, if any
        $error = null;

        if(isset($_POST) && count($_POST)>0){
            $resp = recaptcha_check_answer ($privatekey,
                                            $_SERVER["REMOTE_ADDR"],
                                            $_POST["recaptcha_challenge_field"],
                                            $_POST["recaptcha_response_field"]);
            if ($resp->is_valid) {
                if($this->form_validation->run()){
                    $new_data = array(  'user_first_name' => $_POST['user_first_name'],
                                        'user_last_name' => $_POST['user_last_name'],
                                        'user_email' => $_POST['user_email']);
                    $this->db->insert('user', $new_data);
                    $new_user_id = $this->db->insert_id();
                    $this->load->view('header');
                    $this->load->view('sent');
                    $this->load->view('footer');
                    return;
                }
            }else{
                 $error = $resp->error;
            }
            $data = $_POST;
        }

        $data['recaptcha'] = recaptcha_get_html($publickey, $error);

        $this->load->view('header');
        $this->load->view('inscription_form', $data);
        $this->load->view('footer');
    }
}

そしてフォームはこちら

<div id="inscription">        
    <?php echo form_open(); ?>
        <p>
            <label for="user_first_name">Pr&eacute;nom *</label>
            <input type="text" name="user_first_name" value="<?php echo set_value('user_first_name'); ?>" maxlength="50" />
            <?php echo form_error('user_first_name'); ?>
        </p>
        <p class="clear">
            <label for="user_last_name">Nom *</label>
            <input type="text" name="user_last_name" value="<?php echo set_value('user_last_name'); ?>" maxlength="50" />
            <?php echo form_error('user_last_name'); ?>
        </p>
        <p class="clear">
            <label for="user_email">Courriel *</label>
            <input type="text" name="user_email" value="<?php echo set_value('user_email'); ?>" maxlength="100" />
            <?php echo form_error('user_email'); ?>
        </p>
        <p class="clear">
            <label for="user_email_confirm">Confirmation du courriel *</label>
            <input type="text" name="user_email_confirm" value="<?php echo set_value('user_email_confirm'); ?>" maxlength="100" />
            <?php echo form_error('user_email_confirm'); ?>
       </p>
       <div class="clear"></div>
        <div><?php echo $recaptcha; ?></div>
        <p><input type="submit" value="Envoyer"/></p>
        <p class="clear">* Tous les champs de ce formulaire sont requis.</p>
    </form>
</div>

なぜこれが起こるのですか?

4

1 に答える 1

2

フォームを再作成するには、form_validation コマンドを実行する必要があります。

しかし、あなたの問題は、あなたのコードでは、reCaptcha が失敗した場合、検証を実行しないため、フォームに再度入力するものが何もないことです (実際には検証に到達しないため)。

したがって、コードを変更して、最初にフォームの検証を実行してください。フォームの検証で true が返された場合は、キャプチャを確認してください。

if(isset($_POST) && count($_POST)>0){
        $resp = recaptcha_check_answer ($privatekey,
                                        $_SERVER["REMOTE_ADDR"],
                                        $_POST["recaptcha_challenge_field"],
                                        $_POST["recaptcha_response_field"]);
         if($this->form_validation->run()) 
          {
            if ($resp->is_valid) 
            {
                $new_data = array(  'user_first_name' => $_POST['user_first_name'],
                                    'user_last_name' => $_POST['user_last_name'],
                                    'user_email' => $_POST['user_email']);
                $this->db->insert('user', $new_data);
                $new_user_id = $this->db->insert_id();
                $this->load->view('header');
                $this->load->view('sent');
                $this->load->view('footer');
                return;
            }
            else
            {
                  $error = $resp->error;
            }
        }
        $data = $_POST;
    }

コードを改善するには、代わりに reCaptcha ($resp->is_valid) をコールバック メソッドにします。そうすれば、個別のメソッドではなく、フォーム検証の一部としてキャプチャを実際に実行できます。

コールバックについてはこちら

public function _recaptcha_check($str)
    {
              $resp = recaptcha_check_answer ($str,
                                    $_SERVER["REMOTE_ADDR"],
                                    $_POST["recaptcha_challenge_field"],
                                    $_POST["recaptcha_response_field"]);
          if ( ! $resp->is_valid)
          {
            $this->form_validation->set_message('_recaptcha_check', 'Your reCaptcha was wrong!');
            return FALSE;
        }
        else
        {
            return TRUE;
        }
    }

ルールを設定する場所にこれを設定します(コールバック名の二重「_」に注意してください)

array(
                            'field' => 'recaptcha_response_field',
                            'label' => 'Recaptcha',
                            'rules' => 'required|callback__recaptcha_check'),
于 2012-05-09T15:47:33.390 に答える