バージョン選択

フォーラム

メニュー

オンライン状況

37 人のユーザが現在オンラインです。 (17 人のユーザが フォーラム を参照しています。)
登録ユーザ: 0
ゲスト: 37
もっと...

サイト内検索

バグ報告 > 管理機能 > 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。

管理機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
sumida
投稿日時: 2014/6/21 21:59
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
管理画面:受注情報編集において、「変更」ボタンをクリックし、商品を選択する。
(1)変更前と同じ商品を選ぶ
  現状:数量が+1される。
     本来は、数量が変化してはいけないと思う。
     これについては、既にチケット#2486に登録されています。
修正案
data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

        //既にあるデータは1つだけ数量を1増やす
        $pre_shipment_product_class_id = $arrShipmentProducts['shipment_product_class_id'][$select_shipping_id][$change_no];
        if ($pre_shipment_product_class_id == $edit_product_class_id) {
/* deleted
            $arrShipmentProducts['shipment_quantity'][$select_shipping_id][$change_no] ++;
deleted */
        } elseif (in_array($edit_product_class_id, $arrShipmentProducts['shipment_product_class_id'][$select_shipping_id])) {



(2)既にある商品と同じ商品を選ぶ
  現状:数量が+1される。
     本来は、「変更前の商品の数量+既にある商品の数量」になるのではないかと思う。

この(2)についてのスレッドです。
修正案
data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

        } elseif (in_array($edit_product_class_id, $arrShipmentProducts['shipment_product_class_id'][$select_shipping_id])) {
// added
            $pre_shipment_quantity = $arrShipmentProducts['shipment_quantity'][$select_shipping_id][$change_no];
// added
            //配送先データ削除
            $arrShipmentProducts = $this->deleteShipment($objFormParam, $this->arrShipmentItemKeys , $select_shipping_id, $change_no);
            foreach ($arrShipmentProducts['shipment_product_class_id'][$select_shipping_id] as $relation_index => $shipment_product_class_id) {
                if ($shipment_product_class_id == $edit_product_class_id) {
/* changed
                    $arrShipmentProducts['shipment_quantity'][$select_shipping_id][$relation_index] ++;
changed */
                    $arrShipmentProducts['shipment_quantity'][$select_shipping_id][$relation_index]
                              = $arrShipmentProducts['shipment_quantity'][$select_shipping_id][$relation_index] + $pre_shipment_quantity;

// changed
                    break;
                }
            }
 


追記:6/24
(3)新しい商品を選ぶ
  現状:数量が1にされる。
     本来は、「変更前の商品の数量」になるのではないかと思う。

修正案
data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

    public function changeShipmentProducts(&$arrShipmentProducts, $arrProductInfo, $shipping_id, $no)
    {
        $arrShipmentProducts['shipment_product_class_id'][$shipping_id][$no]    = $arrProductInfo['product_class_id'];
        $arrShipmentProducts['shipment_product_code'][$shipping_id][$no]        = $arrProductInfo['product_code'];
        $arrShipmentProducts['shipment_product_name'][$shipping_id][$no]        = $arrProductInfo['name'];
        $arrShipmentProducts['shipment_classcategory_name1'][$shipping_id][$no] = $arrProductInfo['classcategory_name1'];
        $arrShipmentProducts['shipment_classcategory_name2'][$shipping_id][$no] = $arrProductInfo['classcategory_name2'];
        $arrShipmentProducts['shipment_price'][$shipping_id][$no]               = $arrProductInfo['price02'];
/* deleted
        $arrShipmentProducts['shipment_quantity'][$shipping_id][$no]            = 1;
deleted */
    }
 
bbkids
投稿日時: 2014/6/23 2:32
対応状況: −−−
常連
登録日: 2014/2/19
居住地:
投稿: 48
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
sumida 様

いつもお世話になっております。
「LC_Page_Admin_Order_Edit.php」って
本当不具合多いですね、まだこんなに残っていたとは。

上記2点について、
当方も試してみたところsumida様のご指摘通り再現致しました。
またそれに対するsumida様の修正案を、適用させて頂きました。

当方が確認できる範囲では、sumida様の修正コードで
いずれも問題は修正されているようです。

いつも助かります。
本当に有難う御座いました。
sumida
投稿日時: 2014/6/24 21:42
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
とりあえず、最初の投稿に、

(3)新しい商品に変更した場合についても追記しました。
bbkids
投稿日時: 2014/6/25 8:58
対応状況: −−−
常連
登録日: 2014/2/19
居住地:
投稿: 48
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
ケース(3)の追加修正案について当方も適用させて頂きました。
変更で、新しい商品を選んだ場合、変更前の数量になるように修正されました。

有難う御座いました。


本件のケース(1)〜(3)もそうですが、先日sumida様の修正案でお助け頂いた
受注管理>受注登録で他の商品が削除されてしまう」問題もなのですが、
コミッターの方は何故にバグ認定(チケット作成)してくれないのでしょうか?

またsumida様のご作成頂いた修正案で当方が試した限り、該当不具合は改善
されたように思われます。
やはり何故にコミッターの方はそれらの検証又は別の修正案を作成して頂けないものなのでしょうか?

オープンソースの開発コミュは不慣れなもので、詳しくありませんが、そう言うものなのでしょうか?
sumida
投稿日時: 2014/6/25 11:36
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
bbkids 様

コミッターの方も、みなさん本業の傍らでの作業となるので、
漏れなくという訳には行かないようですよ。
red
投稿日時: 2014/7/2 21:49
対応状況: −−−
登録日: 2010/2/15
居住地: 東京都
投稿: 1498
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
フォーラムの全記事を読んでいくのはそれだけで多大な時間がかかりますのでなかなか難しいのが現状です。
気づけば出来る範囲で作業しますので、直接でも連絡していただければありがたいです


----------------
EC-CUBEのカスタマイズ、トラブル解決承ります
お気軽にお問い合わせ下さい
https://www.ec-cube.net/integrate/partner/partner.php?partner_id=690

snitta
投稿日時: 2014/7/2 23:47
対応状況: −−−
一人前
登録日: 2013/10/3
居住地: 島根県
投稿: 100
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
sumida 様

引用:
(1)変更前と同じ商品を選ぶ
  現状:数量が+1される。
     本来は、数量が変化してはいけないと思う。
     これについては、既にチケット#2486に登録されています。

(2)既にある商品と同じ商品を選ぶ
  現状:数量が+1される。
     本来は、「変更前の商品の数量+既にある商品の数量」になるのではないかと思う。

(3)新しい商品を選ぶ
  現状:数量が1にされる。
     本来は、「変更前の商品の数量」になるのではないかと思う。


(1)は http://svn.ec-cube.net/open_trac/ticket/2486#comment:5 にて対応いたしました。
(2)及び(3)は仕様なのか不具合なのか判断できず、挙動も変わってしまうため取り込んでいません。
ご確認程よろしくお願いいたします。


----------------
Seiji Nitta
zenith6@gmail.com
https://github.com/zenith6/

sumida
投稿日時: 2014/7/3 8:59
対応状況: −−−
仙人
登録日: 2013/2/10
居住地: 広島県呉市ときどき瀬戸内海
投稿: 641
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
snitta 様

対応ありがとうございます。
なお、個人的な見解として、
「商品の変更」とういう作業に対して、使用者が意図していない数量欄を勝手に書き換えてしまうのは、不具合だと思います。
snitta
投稿日時: 2014/7/3 10:50
対応状況: −−−
一人前
登録日: 2013/10/3
居住地: 島根県
投稿: 100
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
数量リセットは商品追加変更が実装された当初からそうなっていますね。
http://svn.ec-cube.net/open_trac/changeset/17773
http://svn.ec-cube.net/open_trac/browser/branches/version-2_4/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php?rev=17773#L671

この辺りの事情は分からないので詳しい方に判断をお任せいたします^^;


----------------
Seiji Nitta
zenith6@gmail.com
https://github.com/zenith6/

bbkids
投稿日時: 2014/7/3 13:20
対応状況: −−−
常連
登録日: 2014/2/19
居住地:
投稿: 48
Re: 管理画面:受注情報編集において、既存の商品に変更した場合、数量が+1される。
red 様

受注管理>受注登録で他の商品が削除されてしまう

につきましても、バグ認定(チケット作成)お願い致します。
お手数お掛け致します。

【追記】
すみません。
既にチケット作成して頂けていたようです。
対応有難う御座いました。
(1) 2 »
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン



統計情報

総メンバー数は74,636名です
総投稿数は104,066件です

投稿数ランキング

1
seasoft
7333
2
468
3217
3
AMUAMU
2712
4
nanasess
2202
5
umebius
2078
6
yuh
1664
7
red
1498
8
h_tanaka
1188
9
tsuji
942
10
fukap
907
11
shutta
835
12
tao_s
794
13 ramrun 789
14 karin 689
15 sumida 641
16
homan
633
17 DELIGHT 572
18
patapata
502
19
flealog
485
20 tonton 437


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.