2012-05-23 165 views
0

我想弄清楚为什么当一个userID和/或registrationKey没有放入URL中时,它不显示404错误页面。应该发生什么情况是激活控制器被加载时,它应该查看userID和registrationKey是否在URL中,如果不是,则转到错误页面。不显示错误页面

网址:siteurl.com/cms/activate/1/d87f0d8a7f0a8

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

class Activate extends CI_Controller { 

public function __construct() 
{ 
    parent::__construct(); 
    $this->load->model('users/usersmodel'); 
} 

public function index($userID = NULL, $registrationKey = NULL) 
{ 
    //Config Defaults Start 
    $msgBoxMsgs = array();//msgType = dl, info, warn, note, msg 
    $cssPageAddons = '';//If you have extra CSS for this view append it here 
    $jsPageAddons = '';//If you have extra JS for this view append it here 
    $metaAddons = '';//Sometimes there is a need for additional Meta Data such in the case of Facebook addon's 
    $siteTitle = '';//alter only if you need something other than the default for this view. 
    //Config Defaults Start 


    //examples of how to use the message box system (css not included). 
    //$msgBoxMsgs[] = array('msgType' => 'dl', 'theMsg' => 'This is a Blank Message Box...'); 

    /**********************************************************Your Coding Logic Here, Start*/ 

    if(!is_numeric($userID) || !preg_match('/^[A-Za-z0-9]+$/', $registrationKey)) 
    { 
     show_404(); 
    } 
    else 
    { 
     $registrationDetails = $this->usersmodel->getRegistrationDetails($userID); 
     if (!is_null($registrationDetails)) 
     { 
      $regKeyDate = $registrationDetails->registrationKeyDate; 

      if ($this->usersmodel->activateUser($userID, $registrationKey)) 
      { 
       $message = 'User was activated successfully! You may now login!';     
      } 
      else 
      { 
       $message = 'User was not activated successfully! Please try again with the right credentials!'; 
      } 
     } 
     else 
     { 
      $message = 'Two or more days have passed since you registered! You will need to register again!';  
     } 
     $bodyContent = $this->config->item('defaultTemplate') ."/usermanagement/forms/activate";//which view file 
    } 

    $this->data['message'] = $message; 
    $bodyType = "full";//type of template 

    /***********************************************************Your Coding Logic Here, End*/ 

    //Double checks if any default variables have been changed, Start. 
    //If msgBoxMsgs array has anything in it, if so displays it in view, else does nothing.  
    if(count($msgBoxMsgs) !== 0) 
    { 
     $msgBoxes = $this->msgboxes->buildMsgBoxesOutput(array('display' => 'show', 'msgs' =>$msgBoxMsgs)); 
    } 
    else 
    { 
     $msgBoxes = array('display' => 'none'); 
    } 

    if($siteTitle == '') 
    { 
     $siteTitle = $this->metatags->SiteTitle(); //reads 
    } 

    //Double checks if any default variables have been changed, End. 

    $this->data['msgBoxes'] = $msgBoxes; 
    $this->data['cssPageAddons'] = $cssPageAddons;//if there is any additional CSS to add from above Variable this will send it to the view. 
    $this->data['jsPageAddons'] = $jsPageAddons;//if there is any addictional JS to add from the above variable this will send it to the view. 
    $this->data['siteTitle'] = $siteTitle;//defaults can be changed via models/metatags.php 
    $this->data['bodyType'] = $bodyType; 
    $this->data['bodyContent'] = $bodyContent; 
    $this->load->view($this->config->item('defaultTemplate') .'/usermanagement/index', $this->data); 
} 

} 

/* End of file activate.php */ 
/* Location: ./application/controllers/activate.php */ 

任何想法?

+0

你确定你确实到达了“show_404();”线?即你确定你的陈述是正确的吗? – Laurence

+0

为什么不对? –

回答

1

我认为代码看起来很好,但是您可能需要对自己的问题进行一些解决。如果你不通过你的URI传递参数,并且show_404()函数没有被调用,那么你需要看看这些变量包含什么。由于userID不是数字或registrationKey不仅仅是字母数字。如果你在他们之前var_dump声明,并且不通过你的URI参数,会发生什么?

+0

NULL这两个变量 –

+0

哦,我想我明白了,它看起来应该调用show_404函数,但它不会返回false或任何内容,脚本会一直持续到您的视图文件。试试'show_404();返回false;' – thefoyer

+0

我不能为其他答案添加评论,但对于Codeigniter你是正确的,它只是'show_404' no'$ this->' – thefoyer