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

Opened 12 years ago

Last modified 12 years ago

商品一覧のレイアウト取得に不具合

Reported by: Seasoft Owned by: somebody
Priority: Milestone: EC-CUBE2.4.1
Component: フロント Version: 2.4.0
Keywords: Cc:
修正済み:

Description (last modified by Seasoft) (diff)

下記のチェンジセットに疑問があります。

  • r17199 (正式版) (コミュニティはr17567)
  • branches/version-2/data/class/pages/products/LC_Page_Products_List.php (L78-81)
78:	 
79:	        // レイアウトデザインを取得 
80:	        $helper = new SC_Helper_PageLayout_Ex(); 
81:	        $helper->sfGetPageLayout($this, false, DEF_LAYOUT); 

下でも同様の処理をしており、そちらの処理の方が適切に思います。この状態ですと、上の処理が優先されるようで、ページレイアウトが商品一覧用に固定されず、パラメータ「DEF_LAYOUT」で構成されてしまいます。

Change History

comment:1 Changed 12 years ago by Seasoft

  • Description modified (diff)

comment:2 Changed 12 years ago by kajiwara

  • Milestone changed from EC-CUBE2.3.4 to EC-CUBE2.x.x

comment:3 Changed 12 years ago by kishik

2.4.0正式版でも確認。
この行が無くても152行目があるので正常にページは表示されます。
ここの処理はデータベースへの問合わせもあるので、
この行が無いほうが処理力がアップするはずです。

ご検討お願いします。

comment:4 Changed 12 years ago by Seasoft

  • Version changed from コミュニティ (eccube-comu) to 2.4.0
  • Description modified (diff)
  • Milestone changed from EC-CUBE2.x.x to EC-CUBE2.4.1

コミュニティは r18101 にて改修しました。

正式版 2.4.0 でも発生する不具合と判明しておりますので、マイルストーン・バージョンを変更します。(kishik 様、正式版での動作確認ありがとうございます。)

comment:5 Changed 12 years ago by kajiwara

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

Seasoft様、kishik様 本件、ご精査いただきありがとうございます。

確かに必要ないと思いますし、かなり若干ではあるものの、負荷軽減にも寄与できるとのことで、対応させていただきました。(r18113 にて)

デザインテンプレートの適応においても問題ないよう思われます。

また、DEF_LAYOUT としていた部分ですが、こちらは「その他の画面レイアウト」の設定であって、変更が前提となる部分かと思われますので、ご指摘の通り、適切ではなかったようです。

ご指摘いただき、ありがとうございました。

Note: See TracTickets for help on using tickets.