fanbinit/rhymix
1
0
Fork 0
mirror of https://github.com/Lastorder-DC/rhymix.git synced 2026-05-04 01:23:32 +09:00
Code Issues Projects Releases Packages Wiki Activity

문서가 없을 경우 404 에러를 내도록 개선

Browse source
This commit is contained in:
conory 2016-03-11 16:15:47 +09:00
parent 1b8a41b077
commit 10e8962fa1
1 changed files with 12 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

25
classes/module/ModuleHandler.class.php
View file

@ -168,12 +168,14 @@ class ModuleHandler extends Handler
// Get module's information based on document_srl, if it's specified // Get module's information based on document_srl, if it's specified
if($this->document_srl) if($this->document_srl)
{ {
$module_info = $oModuleModel->getModuleInfoByDocumentSrl($this->document_srl); $module_info = $oModuleModel->getModuleInfoByDocumentSrl($this->document_srl);
// If the document does not exist, remove document_srl
// redirect, if the document does not exist
if(!$module_info) if(!$module_info)
{ {
unset($this->document_srl); $this->error = 'The document does not exist';
$this->httpStatusCode = '404';
return true;
} }
else else
{ {
@ -181,20 +183,17 @@ class ModuleHandler extends Handler
// if mids are not matching, set it as the document's mid // if mids are not matching, set it as the document's mid
if(!$this->mid || ($this->mid != $module_info->mid)) if(!$this->mid || ($this->mid != $module_info->mid))
{ {
if(Context::getRequestMethod() == 'GET') if(Context::getRequestMethod() == 'GET')
{ {
Context::setCacheControl(0); Context::setCacheControl(0);
$this->mid = $module_info->mid; header('location: ' . getNotEncodedSiteUrl($site_module_info->domain, 'mid', $module_info->mid, 'document_srl', $this->document_srl), true, 301);
header('location: ' . getNotEncodedSiteUrl($site_module_info->domain, 'mid', $this->mid, 'document_srl', $this->document_srl), true, 301); return false;
return FALSE;
} }
else else
{ {
$this->mid = $module_info->mid; $this->mid = $module_info->mid;
Context::set('mid', $this->mid); Context::set('mid', $this->mid);
} }
} }
// if requested module is different from one of the document, remove the module information retrieved based on the document number // if requested module is different from one of the document, remove the module information retrieved based on the document number
if($this->module && $module_info->module != $this->module) if($this->module && $module_info->module != $this->module)
@ -202,7 +201,6 @@ class ModuleHandler extends Handler
unset($module_info); unset($module_info);
} }
} }
} }
// If module_info is not set yet, and there exists mid information, get module information based on the mid // If module_info is not set yet, and there exists mid information, get module information based on the mid
@ -218,7 +216,7 @@ class ModuleHandler extends Handler
Context::setCacheControl(0); Context::setCacheControl(0);
$site_info = $oModuleModel->getSiteInfo($site_module_info->module_site_srl); $site_info = $oModuleModel->getSiteInfo($site_module_info->module_site_srl);
header('location: ' . getNotEncodedSiteUrl($site_info->domain, 'mid', $site_module_info->mid), true, 301); header('location: ' . getNotEncodedSiteUrl($site_info->domain, 'mid', $site_module_info->mid), true, 301);
return FALSE; return false;
} }
// If module_info is not set still, and $module does not exist, find the default module // If module_info is not set still, and $module does not exist, find the default module
@ -249,7 +247,7 @@ class ModuleHandler extends Handler
Context::setCacheControl(0); Context::setCacheControl(0);
header("Location: $redirect_url", true, 301); header("Location: $redirect_url", true, 301);
return FALSE; return false;
} }
// If module info was set, retrieve variables from the module information // If module info was set, retrieve variables from the module information
@ -297,6 +295,7 @@ class ModuleHandler extends Handler
{ {
$this->error = 'msg_module_is_not_exists'; $this->error = 'msg_module_is_not_exists';
$this->httpStatusCode = '404'; $this->httpStatusCode = '404';
return true;
} }
// If mid exists, set mid into context // If mid exists, set mid into context
@ -310,13 +309,13 @@ class ModuleHandler extends Handler
if(!$output->toBool()) if(!$output->toBool())
{ {
$this->error = $output->getMessage(); $this->error = $output->getMessage();
return TRUE; return true;
} }
// Set current module info into context // Set current module info into context
Context::set('current_module_info', $this->module_info); Context::set('current_module_info', $this->module_info);
return TRUE; return true;
} }
/** /**