Ticket #464 (closed 改善提案: 修正済)
受注新規追加機能での同一商品追加時の処理
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 14 years ago by Yammy
- Owner changed from somebody to Yammy
- Status changed from new to assigned
comment:2 Changed 14 years ago by Yammy
- Status changed from assigned to closed
- Resolution set to 修正済
r18014としてコミット
comment:4 Changed 14 years ago by kajiwara
- Priority changed from 中 to 低
- Status changed from closed to reopened
- Resolution 修正済 deleted
Yammy様 本件、ご対応ありがとうございます。
正式版でも事象を確認いたしました。
正式版では追って対応させていただきますので、正式版対応完了後、改めて修正済のステータスに変更させていただきたいと思います。
また、本事象に関しては、代替操作が容易に行えるため、優先度を低に下げさせていただきました。
以上、よろしくお願いいたします。
Note: See
TracTickets for help on using
tickets.