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 17 years ago by Yammy
- Owner changed from somebody to Yammy
- Status changed from new to assigned
comment:2 Changed 17 years ago by Yammy
- Status changed from assigned to closed
- Resolution set to 修正済
r18014としてコミット
comment:4 Changed 17 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.
