2012-02-11 33 views
0

我有一个表单不输出任何错误消息我错过了什么吗?不输出表单错误消息

型号:

function createUser($username = NULL ,$passwordHash = NULL ,$firstname = NULL ,$lastname = NULL ,$email = NULL,$group = NULL ,$active = NULL) 
    { 
     $data = array('userName' => $username, 'userFirstName' => $firstname, 'userLastName' => $lastname, 'userEmail' => $email, 'userPassword' => sha1($passwordHash), 'userGroup' => $group, 'userActive' => $active); 

     $this->db->insert('users',$data); 

     return TRUE; 
    } 

查看:

<h1><?php echo $companyName; echo nbs(1);?> - <?php echo $pageTitle; ?></h1> 

<?php 
if($success == TRUE) { 
echo '<section id = "validation">Page Updated</section>'; 
} 
?> 
    <p>Error: <?php echo validation_errors();?> </p> 
    <div class="formContent"> 
     <form action="createUser" method="post"> 
      <fieldset class="control-group"> 
       <label for="userName">User Name: <input type="text" name="userName" value="<?php echo set_value('userName'); ?>" placeholder="User Name"></label> 
       <label for="userPassword">User Password: <input type="password" name="userPassword" value="<?php echo set_value('userPassword'); ?>" placeholder="User Password"></label> 
       <label for="userFirstName">First Name: <input type="text" name="userFirstName" value="<?php echo set_value('userFirstName'); ?>" placeholder="First Name"></label> 
       <label for="userLastName">Last Name: <input type="text" name="userLastName" value="<?php echo set_value('userLastName'); ?>" placeholder="Last Name"></label> 
       <label for="userEmail">E-Mail: <input type="text" name="userEmail" value="<?php echo set_value('userEmail'); ?>" placeholder="Admin E-mail"></label> 
       <label for="userGroup"> User Group: 
        <select name="userGroup" value="<?php echo set_value('userGroup'); ?>"> 
         <option value="select">Please Select</option> 
         <option value="admin">Admin Group</option> 
         <option value="user">User Group</option> 
        </select> 
       </label> 
       <label for="userActive"> User Active: 
         <select name="userActive" value="<?php echo set_value('userActive'); ?>"> 
          <option value="select">Please Select</option> 
          <option value="yes">Yes</option> 
          <option value="no">No</option> 
         </select> 
       </label> 
       <button type="submit" class="btn-primary">Create</button> 
       </fieldset> 
     </form> 
    </div> 

控制器:

public function index() 
    { 
     $data['companyName'] = $this->core_model->companyName(); 
     $data['success'] =""; 
     $data['pageTitle'] = "Create User"; 
     $this->load->view('admin/assets/header', $data); 
     $this->load->view('admin/createUser', $data); 
     $this->load->view('admin/assets/footer'); 

     if($this->input->post('submit')) 
     { 
       $this->form_validation->set_rules('userName', 'User Name', 'trim|required|xss_clean|callback_username_check'); 
       $this->form_validation->set_rules('userPassword', 'User Password', 'trim|required|xss_clean|sha1'); 
       $this->form_validation->set_rules('userFirstName', 'First Name', 'trim|required|xss_clean'); 
       $this->form_validation->set_rules('userLastName', 'Last Name', 'trim|required|xss_clean'); 
       $this->form_validation->set_rules('userEmail', 'E-Mail', 'trim|required|xss_clean'); 
       $this->form_validation->set_rules('userGroup', 'User Group', 'trim|required|xss_clean'); 
       $this->form_validation->set_rules('userActive', 'User Active', 'trim|required|xss_clean'); 

      if ($this->form_validation->run() == FALSE) { 

       $data['companyName'] = $this->core_model->companyName(); 
       $data['success'] =""; 
       $data['pageTitle'] = "Create User"; 
       $this->load->view('admin/assets/header', $data); 
       $this->load->view('admin/createUser', $data); 
       $this->load->view('admin/assets/footer'); 
      }else{ 

      $username = $this->input->post('userName',TRUE); 
      $password = $this->input->post('userPassword', TRUE); 
      $firstname = $this->input->post('userFirstName', TRUE); 
      $lastname = $this->input->post('userLastName',TRUE); 
      $email = $this->input->post('userEmail',TRUE); 
      $group = $this->input->post('userGroup',TRUE); 
      $active = $this->input->post('userActive', TRUE); 

      $this->db->escape($username); 
      $this->db->escape($password); 
      $this->db->escape($firstname); 
      $this->db->escape($lastname); 
      $this->db->escape($email); 
      $this->db->escape($group); 
      $this->db->escape($active); 

      $passwordHash = $this->encrypt->sha1($password); 

      if ($this->core_model->createUser($username,$passwordHash,$firstname,$lastname,$email,$group,$active)) 
      { 
       $data['success'] = TRUE; 
       $data['companyName'] = $this->core_model->companyName(); 
       $data['pageTitle'] = "Create User"; 
       $this->load->view('admin/assets/header', $data); 
       $this->load->view('admin/createUser', $data); 
       $this->load->view('admin/assets/footer'); 
      }else{ 
       $data['companyName'] = $this->core_model->companyName(); 
       $data['pageTitle'] = "Create User"; 
       $this->load->view('admin/assets/header', $data); 
       $this->load->view('admin/createUser', $data); 
       $this->load->view('admin/assets/footer'); 
      } 
     } 
     } 
    } 

     function __username_check($userName){ 
     { 
      if ($userName == $user->$userName) { 

       $this->form_validation->set_message('username_check','Sorry the chosen username %s is taken!'); 

       return false; 
      }else{ 
       return true; 
      } 

     } 

    } 

} 

/* End of file login.php */ 
/* Location: ./application/controllers/admin/createUser.php */ 

回答

2

你需要把<input> s outside the <label> </label>标签!这是主要问题。

另外:

  1. 没有一个名为“提交”输入:你的提交按钮,其实没有名字属性。而且,顺便说一句,既然你已经在使用form_validation类,那么检查(if input->post('submit'))是多余的;
  2. 我看到的另一个多余的事情是将TRUE(即使用xss_cleaned)传递给input-> post方法:您已经有了大量的xss_clean验证规则,那么为什么还要将再次传递给那么昂贵的额外处理在验证检查过程中通过它?
  3. 旁注,如果你使用的Active Record,或查询绑定,你没有逃跑的变量,所以我会删除部分太:)
  4. 我相信你对__username_check()调用将失败:在函数,关于“callback_”验证规则的是“username_check”;另外双下划线通常用于PHP中的“魔术方法”;您可以安全地删除两者,或者如果您确实需要函数名称的下划线(只有一个),则可能需要调用“callback__check_username”。
  5. 而且你三次加载相同的视图,为什么?我相信你可以重写整个指数的方法是这样的:

    function index() 
    { 
        $this->form_validation->set_rules('userName', 'User Name', 'trim|required|xss_clean|callback_username_check'); 
        $this->form_validation->set_rules('userPassword', 'User Password', 'trim|required|xss_clean|sha1'); 
        $this->form_validation->set_rules('userFirstName', 'First Name', 'trim|required|xss_clean'); 
        $this->form_validation->set_rules('userLastName', 'Last Name', 'trim|required|xss_clean'); 
        $this->form_validation->set_rules('userEmail', 'E-Mail', 'trim|required|xss_clean'); 
        $this->form_validation->set_rules('userGroup', 'User Group', 'trim|required|xss_clean'); 
        $this->form_validation->set_rules('userActive', 'User Active', 'trim|required|xss_clean'); 
    
        if ($this->form_validation->run() == FALSE) { 
    
         $data['success'] =""; 
    
        }else{ 
    
         $username = $this->input->post('userName'); 
         $password = $this->input->post('userPassword'); 
         $firstname = $this->input->post('userFirstName'); 
         $lastname = $this->input->post('userLastName'); 
         $email = $this->input->post('userEmail'); 
         $group = $this->input->post('userGroup'); 
         $active = $this->input->post('userActive'); 
         $passwordHash = $this->encrypt->sha1($password); 
    
         if ($this->core_model->createUser($username,$passwordHash,$firstname,$lastname,$email,$group,$active)) 
         { 
          $data['success'] = TRUE; 
    
         }  
        } 
        $data['companyName'] = $this->core_model->companyName(); 
        $data['pageTitle'] = "Create User"; 
        $this->load->view('admin/assets/header', $data); 
        $this->load->view('admin/createUser', $data); 
        $this->load->view('admin/assets/footer'); 
    } 
    

UPDATE:

为用户名检查,因为笨的2.0版你有能力验证中精选规则:如果您放置了is_unique规则,实际上它会自动查询数据库以检查该规则。语法是:

is_unique[table.field] 

在你的情况,可能是

$this->form_validation->set_rules('userName', 'User Name', 'trim|required|is_unique[users.userName]|xss_clean'); 
+0

太感谢你了达米安,我是立足一个网站的验证码,我不得不建立一个文凭我没有和我可以现在看到我已经花了很多时间,然后我应该有代码写作 - >我们在哪里告诉要逃避所有数据等等。RE:4我现在想创建的是一个检查器,检查用户名是否有被采取,然后吐出一个错误,如果它被采取或处理,如果它确定。我将如何做到这一点?我会上传模型以获得非常明确的反馈,希望我在一年前找到您。 – 2012-02-12 02:55:34

+0

那么,你只需要查询数据库:如果用户名存在,返回FALSE(验证失败:用户名存在),否则返回TRUE(验证通过)。但是:如果你愿意将你的CI升级到大于2的版本,你会注意到有一个新的验证规则可以做到这一点。我已经用这个更新了我的答案。至于转义数据,你被告知是正确的,但关于框架的好处是他们可以减轻你的工作量:因为你使用的是Active Record,所以你的查询已经被框架转义了 – 2012-02-12 08:41:36