ログイン前だとバナーのリンクが機能しない

[imamura623]

■現象

バナーに関連付けられたURLに遷移しない。

■原因

ログインしていない状態でセキュアな画面にアクセスしているため、
正常なURLに遷移しない。

■修正内容

$before_afterの値で条件分岐し ログイン前とログイン後で別のアクションを指定する必要がある。

• ログイン前⇒do_o_click_banner
• ログイン後⇒do_h_click_banner

({if $before_after == "after"})
<a href="({t_url m=pc a=do_h_click_banner})&amp;target_c_banner_id=({$INC_FOOTER_inc_side_banner.c_banner_id})&amp;sessid=({$PHPSESSID})" target="_blank">
({else})
<a href="({t_url m=pc a=do_o_click_banner})&amp;target_c_banner_id=({$INC_FOOTER_inc_side_banner.c_banner_id})" target="_blank">
({/if})
<img src="({t_img_url filename=$INC_FOOTER_inc_side_banner.image_filename})" alt="({$INC_FOOTER_inc_side_banner.nickname})" /></a>

ポータルのプレビュー画面はサイドバー・フッターをログイン前として表示する必要がある。
管理画面でログインした状態なので、ログイン前の情報で上書きするなどの対処が必要。

■関連情報

​http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=4119

[kiwa]

修正案が入っているのでキーワード外しますね。

[mogi]

以下のリビジョンで修正しました。ご確認ください。

• r11051

[mogi]

張り忘れていたリンクがあったので、改めてこちらで張りなおします。

• 2.12 r11053
• trunk r11051　

[ogawa]

以下の点について修正してください。

• トップバナーにも同様の問題があると思います
• webapp_biz/modules/pc/templates/common/layout.tpl が修正されていません
• ログイン前かログイン後かの判定には $before_after を使った方がいいかもしれません

[nagasawa]

r11074 で修正しました。

[ogawa]

• 1ファイルしか修正されていません（トップバナーは他のファイルも修正必要）
• trunk しか修正されていません
• チケットの担当者が nobody のままです

[kiwa]

修正やります。

[kiwa]

以下のリビジョンで修正しました。ご確認ください。

• 2.12 - r11077, r11080
• trunk - r11078, r11079

[ebihara]

portal_page_user_top, portal_page_preview では pc モジュールの init.inc を利用することでテンプレート変数 $before_after のアサインをおこなっていますが、この $before_after の値は現在実行中のアクションクラスの isSecure() メソッドの返り値に依存します。

portal_page_user_top の場合は isSecure() が false を返し、 $before_after の値が想定通りの "before" となりますが、 portal_page_preview は isSecure() が true を返すために $before_after の値が "after" となってしまいます。そのため、管理者が SNS にログインしていない状態で portal_page_preview のバナーをクリックした場合、正常に遷移をおこなうことができません。

portal モジュールの場合はほとんどログイン前として扱われるのが望ましいため、以下のどちらかのアプローチで修正する必要があると思います。

• portal モジュール内では常にログイン前バナーを表示する
• portal モジュールの init.inc で、 pc モジュールの init.inc を読み込んだ後、アサインされた $before_after の値を強制的に "before" に上書きする

[nagasawa]

修正します。

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11107
• trunk - r11105

[nagasawa]

必要なものまで消してる

[nagasawa]

以下で修正しました。

• 2.12 - r11108
• trunk - r11109

[kiwa]

portal_page_previewのサイドバナーにログイン前バナーが表示されません。{$INC_FOOTER_inc_side_banner}に値が入っていないです。

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11122
• trunk - r11121

[ebihara]

r11122, r11121 の変更は取り消してください。 portal_page_preview をログイン済みの管理者以外に閲覧させるべきではありません。

[nagasawa]

r11123,r11124で変更を取り消し、以下のリビジョンで修正しました。

• 2.12 - r11128
• trunk - r11127

[ogawa]

comment:18 は、ログイン後のバナー画像（任意HTMLは対象外）が反映されてしまう問題を含んでいます。

サイドバナーに関しては comment:21 で修正されていますが、同様の問題がフッターのHTML挿入にもあります。プレビューページでログイン後のフッターが表示されてしまいます（INC_FOOTER_inc_page_footer）

それと、チケットのDescriptionの書式を修正してください。

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11142
• trunk - r11141

[ogawa]

comment:11 以前の修正に対する指摘です。

1. 以下の2ファイルの差分を極力減らしてください。サイドバナー部分の記述がかなり違っています
   • webapp/modules/portal/templates/user_top.tpl
   • webapp/modules/portal/templates/preview.tpl
2. 以下の2ファイルの差分を極力減らしてください。BIZの方にはログイン前のページはないかもしれませんが、今回の修正も入れて合わせておいた方がよいと思います
   • webapp/modules/pc/templates/common/layout.tpl
   • webapp_biz/modules/pc/templates/common/layout.tpl

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11149
• trunk - r11148

[ogawa]

webapp/modules/pc/templates/common/layout.tpl　に newschedule と todolist は入れないでください。差異を極力減らすといっても webapp_biz のみにあるものを webapp に入れるのは好ましくないです。

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11152
• trunk - r11150

[ogawa]

空行の連続は必要ないので削除してください。

また指摘が漏れていましたが、以下のテンプレート内でインデントにハードタブと半角スペースが混在しているのでどちらかに統一してください。

• webapp/modules/pc/templates/common/layout.tpl
• webapp_biz/modules/biz/templates/common/layout.tpl

[nagasawa]

以下のリビジョンで修正しました。

• 2.12 - r11154
• trunk - r11153

[ogawa]

OKです。

[kiwa]

確認しました。問題ありません。

動作テスト中に別の現象を発見しましたが、範囲外のため別のチケットにします。→ #3759