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

[Seasoft]

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

• 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」で構成されてしまいます。

[kishik]

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

ご検討お願いします。

[Seasoft]

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

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

[kajiwara]

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

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

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

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

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