Ticket #464 (closed 改善提案: 修正済)

Opened 11 years ago

Last modified 11 years ago

受注新規追加機能での同一商品追加時の処理

Reported by: Yammy Owned by: Yammy
Priority: Milestone: EC-CUBE2.4.1
Component: 管理画面 Version: 2.4.0RC-2
Keywords: Cc:
修正済み:

Description

受注新規追加機能で、同一商品追加時の処理ですが、現状同じ商品が既にあっても、もう一行追加されます。 既に同じ商品がある場合は、行を追加せずに数量を+1するように処理を変更しました。

Index: LC_Page_Admin_Order_Edit.php
===================================================================
--- LC_Page_Admin_Order_Edit.php        (リビジョン 18013)
+++ LC_Page_Admin_Order_Edit.php        (作業コピー)
@@ -702,7 +702,25 @@
     function lfInsertProduct($product_id, $classcategory_id1, $classcategory_id2) {
         $arrProduct = $this->lfGetProductsClass($product_id, $classcategory_id1, $classcategory_id2);
         $this->arrForm = $this->objFormParam->getFormParamList();
-        $this->lfSetProductData($arrProduct);
+        $existes = false;
+        $existes_key = NULL;
+        if (!empty($this->arrForm['product_id']['value'])) {
+            foreach ($this->arrForm['product_id']['value'] AS $key=>$val) {
+                if ($val == $product_id && $this->arrForm['product_id']['classcategory_id1'][$key] == $classcategory_id1 && $this->arrForm['product_id']['classcategory_id2'][$key] == $classcategory_id2) {
+                    // 既に同じ賞品がある場合
+                    $existes = true;
+                    $existes_key = $key;
+                }
+            }
+        }
+
+        if ($existes) {
+            // 既に同じ賞品がある場合
+            ++$this->arrForm['quantity']['value'][$existes_key];
+        } else {
+            // 既に同じ賞品がない場合
+            $this->lfSetProductData($arrProduct);
+        }
     }

     function lfUpdateProduct($product_id, $classcategory_id1, $classcategory_id2, $no) {

Change History

comment:1 Changed 11 years ago by Yammy

  • Owner changed from somebody to Yammy
  • Status changed from new to assigned

comment:2 Changed 11 years ago by Yammy

  • Status changed from assigned to closed
  • Resolution set to 修正済

r18014としてコミット

comment:3 Changed 11 years ago by Yammy

r18015 でコメント中の「賞品」を「商品」に修正。

comment:4 Changed 11 years ago by kajiwara

  • Priority changed from to
  • Status changed from closed to reopened
  • Resolution 修正済 deleted

Yammy様 本件、ご対応ありがとうございます。

正式版でも事象を確認いたしました。

正式版では追って対応させていただきますので、正式版対応完了後、改めて修正済のステータスに変更させていただきたいと思います。

また、本事象に関しては、代替操作が容易に行えるため、優先度を低に下げさせていただきました。

以上、よろしくお願いいたします。

comment:5 Changed 11 years ago by kajiwara

  • Status changed from reopened to closed
  • Resolution set to 修正済

r18160 にて本件対応させていただきました。

正常に、通常での商品追加、同商品の追加 ができるこを確認いたしました。

以上、よろしくお願いいたします。

Note: See TracTickets for help on using tickets.