Minahito
minah****@users*****
2006年 7月 10日 (月) 16:24:55 JST
Index: xoops2jp/html/modules/legacyRender/admin/forms/BannerFilterForm.class.php diff -u xoops2jp/html/modules/legacyRender/admin/forms/BannerFilterForm.class.php:1.1.2.3 xoops2jp/html/modules/legacyRender/admin/forms/BannerFilterForm.class.php:1.1.2.4 --- xoops2jp/html/modules/legacyRender/admin/forms/BannerFilterForm.class.php:1.1.2.3 Tue Apr 25 14:24:16 2006 +++ xoops2jp/html/modules/legacyRender/admin/forms/BannerFilterForm.class.php Mon Jul 10 16:24:55 2006 @@ -1,4 +1,8 @@ <?php +/** + * @package legacyRender + * @version $Id: BannerFilterForm.class.php,v 1.1.2.4 2006/07/10 07:24:55 minahito Exp $ + */ if (!defined('XOOPS_ROOT_PATH')) exit(); @@ -21,7 +25,7 @@ class LegacyRender_BannerFilterForm extends LegacyRender_AbstractFilterForm { var $mSort = 0; - var $_mSortKeys = array( + var $mSortKeys = array( BANNER_SORT_KEY_BID => 'bid', BANNER_SORT_KEY_CID => 'cid', BANNER_SORT_KEY_IMPTOTAL => 'imptotal', @@ -39,18 +43,18 @@ { $this->mSort = isset($_REQUEST['sort']) ? intval($_REQUEST['sort']) : BANNER_SORT_KEY_DEFAULT; - if ($this->mSort > BANNER_SORT_KEY_MAXVALUE) { + if (!isset($this->mSortKeys[abs($this->mSort)])) { $this->mSort = BANNER_SORT_KEY_DEFAULT; } if (isset($_REQUEST['cid'])) { $this->_mNavi->addExtra('cid', xoops_getrequest('cid')); - $this->_mCriteria->add(new Criteria('cid', array(XOBJ_DTYPE_INT, xoops_getrequest('cid')))); + $this->_mCriteria->add(new Criteria('cid', xoops_getrequest('cid'))); } if (isset($_REQUEST['htmlbanner'])) { $this->_mNavi->addExtra('htmlbanner', xoops_getrequest('htmlbanner')); - $this->_mCriteria->add(new Criteria('htmlbanner', array(XOBJ_DTYPE_BOOL, xoops_getrequest('htmlbanner')))); + $this->_mCriteria->add(new Criteria('htmlbanner', xoops_getrequest('htmlbanner'))); } $this->_mCriteria->addSort($this->getSort(), $this->getOrder());