Extending form validation in Codeigniter

问题:

I have placed this class file called 'My_Form_validation.php' into 'application/core' and I have also tried placing it in 'application/libraries'.

In my controller I am using

$this->form_validation->set_rules('user_postcode', 'Postcode', 'valid_postcode|trim|required|xss_clean');

This is whats in My_Form_validation.php - the actual logic is not in question here because I have a couple of options to actually validate the postcode. What I need help with is understanding why it is not loading or getting called!

My CI version is define('CI_VERSION', '2.0.2');

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

/**
 * Form validation for UK Postcodes
 * 
 * Check that its a valid postcode
 * @author James Mills <[e-mail]>
 * @version 1.0
 * @package FriendsSavingMoney
 */

class MY_Form_validation extends CI_Form_validation
{

    function __construct()
    {
        parent::__construct();  
        log_message('debug', '*** Hello from MY_Form_validation ***');
    }

    function valid_postcode($postcode)
    {

        /**
         *
         * UK Postcode validation expression from Wikipedia
         * http://en.wikipedia.org/wiki/Postcodes_in_the_United_Kingdom
         *
         * Note: Remember to strtoupper() your postcode before inserting into database!
         *
         */

        $pattern = "/^(GIR 0AA)|(((A[BL]|B[ABDHLNRSTX]?|C[ABFHMORTVW]|D[ADEGHLNTY]|E[HNX]?|F[KY]|G[LUY]?|H[ADGPRSUX]|I[GMPV]|JE|K[ATWY]|L[ADELNSU]?|M[EKL]?|N[EGNPRW]?|O[LX]|P[AEHLOR]|R[GHM]|S[AEGKLMNOPRSTY]?|T[ADFNQRSW]|UB|W[ADFNRSV]|YO|ZE)[1-9]?[0-9]|((E|N|NW|SE|SW|W)1|EC[1-4]|WC[12])[A-HJKMNPR-Y]|(SW|W)([2-9]|[1-9][0-9])|EC[1-9][0-9]) [0-9][ABD-HJLNP-UW-Z]{2})$/";


        if (preg_match($pattern, strtoupper($postcode)))
    {
            return TRUE;
        } 
        else
        {
            $this->set_message('valid_postcode', 'Please enter a valid postcode');
            return FALSE;
        }
    }
}

回答1:

Because you're extending a CodeIgniter library and not a core component, you want to place that in application/libraries (not application/core).

And of course, don't forget to load the Form_validation library within your controller code.

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

Other things to check:

• Filename case sensitivity (MY_Form_validation.php loads while My_Form_validation.php won't)
• Class name case sensitivity (class MY_Form_validation extends CI_Form_validation)

Reference material:

• Extending Core Classes
• Extending Native Libraries

回答2:

You have to add $rules on your __construct method and also pass this to parent constructor

eg:

function __construct($rules = array())
{
    parent::__construct($rules);
}

Look at Form_validation and provide same variables.

回答3:

I know this is old, but just in case someone else stumbles on this in the modern day like I did, here's a quick example. (Currently using 3.0.6, but I believe this will work on 2 as well.)

class MY_Form_validation extends CI_Form_validation { // Capitalization matters

  protected $CI;

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

  /**
   * Valid Date
   *
   * Verify that the date value provided can be converted to a valid unix timestamp
   *
   * @param string  $str
   * @return    bool
   */

  public function valid_date($str) {
      $CI = $this->CI =& get_instance(); // Get your CodeIgniter instance

      if (($str = strtotime($str)) === FALSE) { // Basic timestamp check
          // Set error message by calling the method through the CI instance.
          // Obviously must be done BEFORE returning any value
          $this->CI->form_validation->set_message('valid_date', '{field} must be a valid date.');
          return FALSE;
      }

      return TRUE;
    }
}