Minahito
minah****@users*****
2006年 7月 11日 (火) 12:04:18 JST
Index: xoops2jp/html/modules/legacyRender/admin/actions/TplfileCloneAction.class.php diff -u xoops2jp/html/modules/legacyRender/admin/actions/TplfileCloneAction.class.php:1.1.2.3 xoops2jp/html/modules/legacyRender/admin/actions/TplfileCloneAction.class.php:1.1.2.4 --- xoops2jp/html/modules/legacyRender/admin/actions/TplfileCloneAction.class.php:1.1.2.3 Wed May 17 18:43:27 2006 +++ xoops2jp/html/modules/legacyRender/admin/actions/TplfileCloneAction.class.php Tue Jul 11 12:04:17 2006 @@ -1,9 +1,13 @@ <?php +/** + * @package legacyRender + * @version $Id: TplfileCloneAction.class.php,v 1.1.2.4 2006/07/11 03:04:17 minahito Exp $ + */ if (!defined('XOOPS_ROOT_PATH')) exit(); +require_once XOOPS_MODULE_PATH . "/legacyRender/class/AbstractEditAction.class.php"; require_once XOOPS_MODULE_PATH . "/legacyRender/admin/forms/TplfileCloneForm.class.php"; -require_once XOOPS_MODULE_PATH . "/legacyRender/admin/class/AbstractEditAction.class.php"; class LegacyRender_TplfileCloneAction extends LegacyRender_AbstractEditAction { @@ -11,12 +15,12 @@ function _getId() { - return isset($_REQUEST['tpl_id']) ? intval($_REQUEST['tpl_id']) : 0; + return xoops_getrequest('tpl_id'); } function &_getHandler() { - $handler = xoops_getmodulehandler('tplfile'); + $handler =& xoops_getmodulehandler('tplfile'); return $handler; } @@ -27,6 +31,10 @@ $this->mObjectHandler =& $this->_getHandler(); $obj =& $this->mObjectHandler->get($id); + // + // The following code uses 'tpl_tplset' directly. This input value will + // be checked by ActionForm. + // if (is_object($obj) && $obj->get('tpl_tplset') == 'default') { $this->mObject =& $obj->createClone(xoops_getrequest('tpl_tplset')); } @@ -60,7 +68,7 @@ function executeViewError(&$controller, &$xoopsUser, &$render) { - redirect_header("./index.php?action=TplsetList", 1, _AD_LEGACYRENDER_ERROR_DBUPDATE_FAILED); + $controller->executeRedirect("./index.php?action=TplsetList", 1, _AD_LEGACYRENDER_ERROR_DBUPDATE_FAILED); } }