Changeset 22114

Timestamp:

2012/12/01 17:15:33 (11 years ago)

Author:

Seasoft

Message:

#1973 (対応状況管理画面 SC_Helper_Purchase#sfUpdateOrderStatus の第1引数を空文字で呼んでいる)
#1903 (無駄な処理を改善する)
#1905 (typo修正・ソース整形・ソースコメントの改善)

Location:

branches/version-2_12-dev/data

Files:

• Smarty/templates/admin/order/status.tpl (modified) (3 diffs)
• class/pages/admin/order/LC_Page_Admin_Order_Status.php (modified) (4 diffs)

branches/version-2_12-dev/data/Smarty/templates/admin/order/status.tpl

@@ -102 +102 @@
             <!--{/section}-->
         </table>
-        <input type="hidden" name="move[]" value="" />
 
         <p><!--{$tpl_strnavi}--></p>
@@ -120 +119 @@
 <!--
 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;
@@ -136 +134 @@
     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

@@ -83 +83 @@
 
         $this->arrForm = $objFormParam->getHashArray();
-        //        $this->arrForm = $_POST;
 
         //支払方法の取得
@@ -91 +90 @@
             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') : '';
+            default:
                 break;
-
-            default:
-                // 対応状況
+        }
+
+        // 対応状況
+        $status = $objFormParam->getValue('status');
+        if (strlen($status) === 0) {
                 //デフォルトで新規受付一覧表示
                 $status = ORDER_NEW;
-                break;
-        }
-
-        // 対応状況
+        }
         $this->SelectedStatus = $status;
         //検索結果の表示
@@ -133 +125 @@
         $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'));
+        }
     }
 
@@ -149 +143 @@
             $objFormParam->setValue('search_pageno', 1);
         }
-
-        if ($this->getMode() == 'change') {
-            if (is_null($objFormParam->getValue('change_status'))) {
-                $objFormParam->setValue('change_status','');
-            }
-        }
-
     }