EC-CUBE 開発コミュニティ

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

管理機能

投稿者	スレッド
sumida	投稿日時: 2014/6/21 21:59 対応状況: −−−
仙人登録日: 2013/2/10 居住地: 広島県呉市ときどき瀬戸内海投稿: 641	管理画面：受注情報編集において、既存の商品に変更した場合、数量が+1される。管理画面：受注情報編集において、「変更」ボタンをクリックし、商品を選択する。（１）変更前と同じ商品を選ぶ　　現状：数量が＋１される。　　　　　本来は、数量が変化してはいけないと思う。　　　　　これについては、既にチケット#2486に登録されています。修正案 data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php `//既にあるデータは１つだけ数量を１増やす $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])) {` （２）既にある商品と同じ商品を選ぶ　　現状：数量が＋１される。　　　　　本来は、「変更前の商品の数量＋既にある商品の数量」になるのではないかと思う。この（２）についてのスレッドです。修正案 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; } } 　追記：６／２４（３）新しい商品を選ぶ　　現状：数量が１にされる。　　　　　本来は、「変更前の商品の数量」になるのではないかと思う。修正案 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 居住地: 投稿: 64	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される。とりあえず、最初の投稿に、（３）新しい商品に変更した場合についても追記しました。

bbkids	投稿日時: 2014/6/25 8:58 対応状況: −−−
常連登録日: 2014/2/19 居住地: 投稿: 64	Re: 管理画面：受注情報編集において、既存の商品に変更した場合、数量が+1される。ケース（３）の追加修正案について当方も適用させて頂きました。変更で、新しい商品を選んだ場合、変更前の数量になるように修正されました。有難う御座いました。本件のケース（１）〜（３）もそうですが、先日sumida様の修正案でお助け頂いた「受注管理＞受注登録で他の商品が削除されてしまう」問題もなのですが、コミッターの方は何故にバグ認定(チケット作成)してくれないのでしょうか？またsumida様のご作成頂いた修正案で当方が試した限り、該当不具合は改善されたように思われます。やはり何故にコミッターの方はそれらの検証又は別の修正案を作成して頂けないものなのでしょうか？オープンソースの開発コミュは不慣れなもので、詳しくありませんが、そう言うものなのでしょうか？

sumida	投稿日時: 2014/6/25 11:36 対応状況: −−−
仙人登録日: 2013/2/10 居住地: 広島県呉市ときどき瀬戸内海投稿: 641	Re: 管理画面：受注情報編集において、既存の商品に変更した場合、数量が+1される。 bbkids 様コミッターの方も、みなさん本業の傍らでの作業となるので、漏れなくという訳には行かないようですよ。

red	投稿日時: 2014/7/2 21:49 対応状況: −−−
神登録日: 2010/2/15 居住地: 東京都投稿: 1574	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 様引用：（１）変更前と同じ商品を選ぶ　　現状：数量が＋１される。　　　　　本来は、数量が変化してはいけないと思う。　　　　　これについては、既にチケット#2486に登録されています。（２）既にある商品と同じ商品を選ぶ　　現状：数量が＋１される。　　　　　本来は、「変更前の商品の数量＋既にある商品の数量」になるのではないかと思う。（３）新しい商品を選ぶ　　現状：数量が１にされる。　　　　　本来は、「変更前の商品の数量」になるのではないかと思う。（１）は http://svn.ec-cube.net/open_trac/ticket/2486#comment:5 にて対応いたしました。（２）及び（３）は仕様なのか不具合なのか判断できず、挙動も変わってしまうため取り込んでいません。ご確認程よろしくお願いいたします。 ---------------- Seiji Nitta [email protected] 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 [email protected] https://github.com/zenith6/

bbkids	投稿日時: 2014/7/3 13:20 対応状況: −−−
常連登録日: 2014/2/19 居住地: 投稿: 64	Re: 管理画面：受注情報編集において、既存の商品に変更した場合、数量が+1される。 red 様受注管理＞受注登録で他の商品が削除されてしまうにつきましても、バグ認定（チケット作成）お願い致します。お手数お掛け致します。【追記】すみません。既にチケット作成して頂けていたようです。対応有難う御座いました。

(1) 2 »