[Svn-src-all:3213] [version-2_12-dev 22114] #1973 (対応状況管理画面 SC_Helper_Purchase#sfUpdateOrderStatus の第1引数を空文字で呼んでいる)
Seasoft
admin ＠ mail.ec-cube.net
2012年 12月 1日 (土) 17:15:33 JST
Subversion committed to /home/svn/open 22114
http://svn.ec-cube.net/open_trac/changeset/22114
┌────────────────────────────┐
│更新者 : Seasoft │
│更新日時: 2012-12-01 17:15:33 +0900 (土, 01 12月 2012)│
└────────────────────────────┘
Log:
--------------------------------------------------------
#1973 (対応状況管理画面 SC_Helper_Purchase#sfUpdateOrderStatus の第1引数を空文字で呼んでいる)
#1903 (無駄な処理を改善する)
#1905 (typo修正・ソース整形・ソースコメントの改善)
Changed: [U:修正，A:追加，D:削除]
--------------------------------------------------------
U branches/version-2_12-dev/data/Smarty/templates/admin/order/status.tpl
U branches/version-2_12-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Status.php
変更: branches/version-2_12-dev/data/Smarty/templates/admin/order/status.tpl
===================================================================
--- branches/version-2_12-dev/data/Smarty/templates/admin/order/status.tpl 2012-11-29 06:14:47 UTC (rev 22113)
+++ branches/version-2_12-dev/data/Smarty/templates/admin/order/status.tpl 2012-12-01 08:15:33 UTC (rev 22114)
@@ -101,7 +101,6 @@
</tr>
<!--{/section}-->
</table>
- <input type="hidden" name="move[]" value="" />
<p><!--{$tpl_strnavi}--></p>
@@ -119,15 +118,14 @@
<script type="text/javascript">
<!--
function fnSelectCheckSubmit(){
-
var selectflag = 0;
var fm = document.form1;
- if(fm.change_status.options[document.form1.change_status.selectedIndex].value == ""){
- selectflag = 1;
+ if (fm.change_status.options[document.form1.change_status.selectedIndex].value == "") {
+ selectflag = 1;
}
- if(selectflag == 1){
+ if (selectflag == 1) {
alert('セレクトボックスが選択されていません');
return false;
}
@@ -135,26 +133,26 @@
var checkflag = 0;
var max = fm["move[]"].length;
- if(max) {
+ if (max) {
for (i=0;i<max;i++){
- if(fm["move[]"][i].checked == true){
+ if(fm["move[]"][i].checked == true) {
checkflag = 1;
}
}
} else {
- if(fm["move[]"].checked == true) {
+ if (fm["move[]"].checked == true) {
checkflag = 1;
}
}
- if(checkflag == 0){
+ if (checkflag == 0){
alert('チェックボックスが選択されていません');
return false;
}
- if(selectflag == 0 && checkflag == 1){
- document.form1.mode.value = 'update';
- document.form1.submit();
+ if (selectflag == 0 && checkflag == 1) {
+ document.form1.mode.value = 'update';
+ document.form1.submit();
}
}
//-->
変更: branches/version-2_12-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Status.php
===================================================================
--- branches/version-2_12-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Status.php 2012-11-29 06:14:47 UTC (rev 22113)
+++ branches/version-2_12-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Status.php 2012-12-01 08:15:33 UTC (rev 22114)
@@ -82,7 +82,6 @@
$objFormParam->convParam();
$this->arrForm = $objFormParam->getHashArray();
- // $this->arrForm = $_POST;
//支払方法の取得
$this->arrPayment = $objDb->sfGetIDValueList('dtb_payment', 'payment_id', 'payment_method');
@@ -90,35 +89,28 @@
switch ($this->getMode()) {
case 'update':
switch ($objFormParam->getValue('change_status')) {
- case '':
- break;
- // 削除
+ // 削除
case 'delete':
$this->lfDelete($objFormParam->getValue('move'));
break;
- // 更新
+ // 更新
default:
$this->lfStatusMove($objFormParam->getValue('change_status'), $objFormParam->getValue('move'));
break;
}
-
- // 対応状況
- $status = !is_null($objFormParam->getValue('status')) ? $objFormParam->getValue('status') : '';
break;
case 'search':
- // 対応状況
- $status = !is_null($_POST['status']) ? $objFormParam->getValue('status') : '';
- break;
-
default:
- // 対応状況
- //デフォルトで新規受付一覧表示
- $status = ORDER_NEW;
break;
}
// 対応状況
+ $status = $objFormParam->getValue('status');
+ if (strlen($status) === 0) {
+ //デフォルトで新規受付一覧表示
+ $status = ORDER_NEW;
+ }
$this->SelectedStatus = $status;
//検索結果の表示
$this->lfStatusDisp($status, $objFormParam->getValue('search_pageno'));
@@ -132,9 +124,11 @@
function lfInitParam(&$objFormParam) {
$objFormParam->addParam('注文番号', 'order_id', INT_LEN, 'n', array('MAX_LENGTH_CHECK', 'NUM_CHECK'));
$objFormParam->addParam('変更前対応状況', 'status', INT_LEN, 'n', array('MAX_LENGTH_CHECK', 'NUM_CHECK'));
- $objFormParam->addParam('変更後対応状況', 'change_status', STEXT_LEN, 'KVa', array('MAX_LENGTH_CHECK', 'NUM_CHECK'));
$objFormParam->addParam('ページ番号', 'search_pageno', INT_LEN, 'n', array('MAX_LENGTH_CHECK', 'NUM_CHECK'));
- $objFormParam->addParam('移動注文番号', 'move', INT_LEN, 'n', array('MAX_LENGTH_CHECK', 'NUM_CHECK'));
+ if ($this->getMode() == 'update') {
+ $objFormParam->addParam('変更後対応状況', 'change_status', STEXT_LEN, 'KVa', array('EXIST_CHECK', 'MAX_LENGTH_CHECK', 'NUM_CHECK'));
+ $objFormParam->addParam('移動注文番号', 'move', INT_LEN, 'n', array('EXIST_CHECK', 'MAX_LENGTH_CHECK', 'NUM_CHECK'));
+ }
}
/**
@@ -148,13 +142,6 @@
if (is_null($objFormParam->getValue('search_pageno'))) {
$objFormParam->setValue('search_pageno', 1);
}
-
- if ($this->getMode() == 'change') {
- if (is_null($objFormParam->getValue('change_status'))) {
- $objFormParam->setValue('change_status','');
- }
- }
-
}
/**
Svn-src-all メーリングリストの案内