2012-05-23 2 views
0

Я пытаюсь понять, почему, когда идентификатор пользователя И/ИЛИ регистрационный ключ не помещается в URL-адрес, он не показывает страницу ошибки 404. То, что должно произойти, - это когда контроллер активации загружен, он должен видеть, находятся ли идентификаторы userID и registrationKey в URL-адресе или нет, и если не перейти на страницу с ошибкой.Не показывать страницу с ошибкой

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();» линия? то есть вы уверены, что ваш оператор if верен? – Laurence

+0

Почему бы не так? –

ответ

1

Я думаю, что код выглядит хорошо, но вам, возможно, придется немного поработать над тем, что вы сами. Если вы не передадите параметры через свой URI, и функция show_404() не вызывается, вам нужно увидеть, что содержат эти переменные. Поскольку либо идентификатор пользователя не является числовым, либо registerKey имеет больше, чем просто буквенно-цифровое. Если вы делаете var_dump перед заявлением и не передаете параметры в URI, что происходит?

+0

NULL для обеих переменных –

+0

О, я думаю, я вижу, похоже, что он должен вызывать функцию show_404, но он не возвращает ложь или что-то еще, и сценарий продолжается до тех пор, пока он не достигнет вашего файла вида. Попробуйте 'show_404(); return false; ' – thefoyer

+0

Я не могу добавить комментарий к другому ответу, но для Codeigniter вы правы, это просто' show_404' no '$ this->' – thefoyer

Смежные вопросы