バグ報告 > 管理機能 > 【2.12.2】「商品管理>商品並び替え」が正常に行えない |
管理機能
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
casa |
投稿日時: 2013/4/13 21:12
対応状況: −−−
|
長老 登録日: 2011/10/3 居住地: 投稿: 210 |
【2.12.2】「商品管理>商品並び替え」が正常に行えない お世話になります。
---------- EC-CUBE: 2.12.2 MySQL: 5.5.16 ---------- 以下に同様の問題が報告されていますが、解決策が取り込まれていないようでしたので投稿致します。取り込めない理由・懸念などがありましたらご教授ください。 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4833&forum=11 問題点1: 商品削除時に商品カテゴリ(dtb_product_categories)が削除されていない 問題点2: 商品登録画面で、登録していたカテゴリを 「削除 ->」 ボタンで外したときに、商品カテゴリを削除したあとrankを詰めていない 問題点1の結果、商品カテゴリに不要なレコードが残ります。 問題点2の結果、商品カテゴリのrankが飛び飛びになります。 これらの状態になると題名の通り、商品並び替えが正常に行えなくなります(指定した順位に移動しない、上へ/下へを何度かクリックしないと移動しない)。 当方では以下のように修正しました。 ■SC_Helper_DB
■LC_Page_Admin_Products
2.12.3の場合 doDelete の処理が若干違うので注意が必要ですが、問題ないと思います。 ご確認をお願い致します。 |
casa |
投稿日時: 2013/4/13 21:23
対応状況: −−−
|
長老 登録日: 2011/10/3 居住地: 投稿: 210 |
Re: 【2.12.2】「商品管理>商品並び替え」が正常に行えない 開発版において、以下のチェンジセットでも doDelete が変更されておりました。その場合は次のようになるかと思います。
http://svn.ec-cube.net/open_trac/changeset/22582 ■LC_Page_Admin_Products
申し訳ありませんが、こちらのバージョンでは動作確認までは行えておりませんのでご了承ください。 # 話がズレますが、上記 doDelete内の $objRecomment->deleteByProductIDs から # $objDb->sfDeleteRankRecord が呼ばれており、ここで再度トランザクションを張っているのですが # 問題ないでしょうか。 |
AMUAMU |
投稿日時: 2013/4/16 3:49
対応状況: −−−
|
神 登録日: 2009/5/2 居住地: 東京都 投稿: 2712 |
Re: 【2.12.2】「商品管理>商品並び替え」が正常に行えない ご報告、提案ありがとうございます!
下記でチケットを立てました。 http://svn.ec-cube.net/open_trac/ticket/2227 次バージョンには含まれるようにしたい所です。 引用: # 話がズレますが、上記 doDelete内の $objRecomment->deleteByProductIDs から 2.12系のSC_Queryではトランザクションのネストが可能になっていますので問題無いと思われます。
|
casa |
投稿日時: 2013/4/16 9:17
対応状況: −−−
|
長老 登録日: 2011/10/3 居住地: 投稿: 210 |
Re: 【2.12.2】「商品管理>商品並び替え」が正常に行えない チケット発行頂きありがとうございます。
引用: 2.12系のSC_Queryではトランザクションのネストが可能になっていますので問題無いと思われます。 こちらについてもご教授いただきありがとうございました。大変勉強になりました。 また、別スレッドを立てようか迷ったのですがこちらに記載します。 上記でも記載した開発版の doDelete処理ですが、以下のforeachループは不要ではないでしょうか。
getCol()を呼んでいるので、戻りは連想配列ではなくただの配列だと思われます。 |
AMUAMU |
投稿日時: 2013/4/16 17:54
対応状況: −−−
|
神 登録日: 2009/5/2 居住地: 東京都 投稿: 2712 |
Re: 【2.12.2】「商品管理>商品並び替え」が正常に行えない >上記でも記載した開発版の doDelete処理ですが、以下のforeachループは不要ではないでしょうか。
確かに不要そう・・・ あとで本体見てみます
|
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |