Ticket #2478 (closed バグ指摘: 修正済)

Opened 7 years ago

Last modified 6 years ago

おすすめ商品管理> 新規に選択した商品を登録前に並び替えしようとすると消えてしまう

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.13.3
Component: 管理画面 Version: 2.13.0
Keywords: Cc:
修正済み: yes

Description

コミュニティにて、下記報告を頂いた。

v2.13.1 おすすめ商品に新規に選択した商品を登録する前に並び替えしようとすると、対象の商品が消える
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=13912&forum=9

[対象バージョン]
v2.13.1(多分v2.13.0も)

[再現手順]
1.おすすめ商品管理で「商品を選択する」を押す
2.ポップアップで適当な商品を選択する
3.新規に選択した商品の並び替えリンク(上下どちらでも可)をクリック
4.商品が消える

[原因]
新規選択商品の場合、$arrItems[$smarty.section.cnt.iteration].best_id に値が無いため、JavaScriptがSyntaxErrorを起こすため
(Mac版Firefox、Safariで確認)

[対処]
Smarty/templates/admin/contents/recommend.tplの並び替えリンクを置くかを判定するifの条件にbest_idの有無を追加

 && $arrItems[$smarty.section.cnt.iteration].best_id

Change History

comment:1 Changed 6 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.13.2 to EC-CUBE2.13.3

comment:2 Changed 6 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from new to assigned

r23549 にて修正。

以下、参考。

管理画面:おすすめ商品管理画面での「並べ替え(上へ・下へ)」の不具合及び修正案
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=13629&forum=7&post_id=61787

(2)「商品を選択する」により商品を選択した直後(「この内容で登録する」を操作していない)の場合

    「上へ」「下へ」を表示している。

不具合:
    「上へ」「下へ」を非表示にすべき。
(SC_Helper_DB.php で、

        // 対象項目のランクを取得
        $rank = $objQuery->get('rank', $table, $where, array($id));

とあり、対象項目の存在を確認していない。)

修正案:

                <td>
                    <!--{* 移動 *}-->
<!--{* changed
                    <!--{if $smarty.section.cnt.iteration != 1 && $arrItems[$smarty.section.cnt.iteration].product_id}-->
changed *}-->
                    <!--{if $smarty.section.cnt.iteration != 1 && $arrItems[$smarty.section.cnt.iteration].best_id}-->
<!-- {* changed *}-->


                    <!--{/if}-->
<!--{* changed
                    <!--{if $smarty.section.cnt.iteration != $tpl_disp_max && $arrItems[$smarty.section.cnt.iteration].product_id}-->
changed *}-->
                    <!--{if $smarty.section.cnt.iteration != $tpl_disp_max && $arrItems[$smarty.section.cnt.iteration].best_id}-->
<!-- {* changed *}-->

v2.13 管理画面:(最後尾の)おすすめ商品の「下へ」ボタンが動作しない。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=14763&forum=9&post_id=66087

(1)「商品の選択」直後の「上へ」「下へ」ボタンが動作しない
「商品の選択」直後は、既にチケット2478にて明記されている不具合があります。
原因:
  ボタンの表示をproduct_idの有無で判断している。
  実処理は、DBにあるおすすめ商品に対して行っている。
  結果、DBにまだ登録されていない商品(「商品の選択」直後)では動作しない。
対策:
  とりあえず、ボタンの表示をbest_idの有無で判断する。
  これにより、「商品の選択」直後は、「上へ」「下へ」ボタンを表示させない。

comment:3 Changed 6 years ago by kim

  • Status changed from assigned to closed
  • Resolution set to 修正済

shutta 様

ありがとうございます。

修正確認いたしました。クローズいたします。

Note: See TracTickets for help on using tickets.