Backport（バックポート） #2138
トップバナー（ログイン前）とサイドバナー（ログイン前）が表示されない
100%
Description
Overview (現象)¶
http://sns.openpne.jp/communityTopic/6399 より転記
3.6beta5 トップバナー（ログイン前）とサイドバナー（ログイン前）が表示されません。 トップバナー（ログイン前）の掲載場所にはトップバナー（ログイン後）が表示され、サイドバナー（ログイン前）には何も表示されません。 FirefoxとChromeとIE8で確認しましたのでブラウザの影響ではないようです。 よろしくお願いいたします。
追記コメント
ガジェット設定でサイドバナーをweb全体に公開にしたら、ログインページにもサイドバナー（ログイン後）が表示されました。 トップバナー（ログイン前）とサイドバナー（ログイン前）は項目があるだけで未実装？
Causes (原因)¶
バナー表示は、ログイン/未ログインを sfUser::getMember() がインスタンスを返すかどうかで判断している。
しかし、3.6系では外部公開機能に対応するため未ログイン時には opAnonymousMember を返すように変更されている、
このことが原因でログイン判定がうまく機能しなくなり、正しく表示されていない。
追記コメントにあるサイドバナーを"web全体に公開"にした場合は表示された件は、
正常な挙動であり、"全員に公開"の場合ログインしていない状態では表示されない。
Way to fix (修正内容)¶
opAnonymousMember のインスタンスでないかどうかをログイン判定の条件に加える。
Related issues
Associated revisions
History
#1
Updated by Kousuke Ebihara over 12 years ago
Updated by - Target version changed from OpenPNE3.6beta11 to OpenPNE3.6beta12
親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。
#2
Updated by Kousuke Ebihara over 12 years ago
- Status changed from New（新規） to Accepted（着手）
- Assignee changed from Masato Nagasawa to Kousuke Ebihara
バックポート引き受けます
#3
Updated by Kousuke Ebihara over 12 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
#4
Updated by Naoya Tozuka over 12 years ago
Updated by - Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
BP修正確認しました。OKです。
#5
Updated by isao sano over 12 years ago
Updated by 以下の手順を行った際、サイドバナーの公開範囲が「全体に公開」にも関わらず、サイドバナー(ログイン前)が表示されます。
1. 管理画面より、サイドバナーガジェット設定(/pc_backend.php/design/gadget/type/sideBanner)を「全員に公開」に設定する
2. サイドバナー（ログイン前）設定(/design/banner/id/2)に遷移する
3. 「任意HTMLで表示する」にチェックを入れ、文字を入力する。
4. 管理画面より、招待メールを送る。
5. 招待メールを開き、書かれているURLに遷移する。
※仮登録の状態以外では、サイドバナーの公開範囲が「全体に公開」になっている場合、サイドバナー(ログイン前)は表示されません。
#6
Updated by isao sano over 12 years ago
現在テストで「remotes/origin/release-3.6beta12」のブランチを使用しています。
git logを行ったときに関連するリビジョンのうち最新の三つ
リビジョン 2c4078735674cd8458027c17043a4969d8f86bf4
Masato Nagasawa が7日前に追加
fixed banner is not displayed when logged in (refs #2138, BP from #1678)
Conflicts:
apps/smartphone_frontend/modules/default/templates/_sideBanner.php
apps/smartphone_frontend/templates/_header.php
リビジョン 7876788547d90d2cde3aead2a615f48288603ca6 Kousuke Ebihara が7日前に追加 changed way to check whether an instance of member is opAnonymousMember in side banner gadget (refs #2138, BP from #1678)
リビジョン a115e44d798277d68456c4bfe68eeb6f1ac6da02 Kousuke Ebihara が7日前に追加 fixed "$sf_user->getMember()->getRawValue()" causes fatal error when registration process (refs #2138, BP from #1678)
は見つけることが出来たのですが、以下を見つけることが出来ません。
リビジョン 96e7df62da6dcfb12f34e41ef9bfacbdc0260320
Masato Nagasawa が9日前に追加
fixed banner is not displayed when logged in (refs #2138, BP from #1678)
Conflicts:
apps/smartphone_frontend/modules/default/templates/_sideBanner.php
apps/smartphone_frontend/templates/_header.php
リビジョン ca79d6dbb97583eabf31ecf9fffcd18f62adbda0 Kousuke Ebihara が9日前に追加 changed way to check whether an instance of member is opAnonymousMember in side banner gadget (refs #2138, BP from #1678)
リビジョン 7388c67527f07c534e3026b6dd81339be42697e2 Kousuke Ebihara が9日前に追加 fixed "$sf_user->getMember()->getRawValue()" causes fatal error when registration process (refs #2138, BP from #1678)
#7
Updated by isao sano over 12 years ago
- Status changed from Pending Testing（テスト待ち） to Rejected（差し戻し）
- % Done changed from 70 to 50
#8
Updated by Kousuke Ebihara over 12 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について、 dc11cddc121b1ded3f9daa32d2d310596d4174fd で対応しました。
#9
Updated by Naoya Tozuka over 12 years ago
メモ¶
http://redmine.openpne.jp/issues/2138#note-6 の件ですが
- 2c4078735674cd8458027c17043a4969d8f86bf4
- 7876788547d90d2cde3aead2a615f48288603ca6
- a115e44d798277d68456c4bfe68eeb6f1ac6da02
および
が release-3.6beta12 へのコミットで、
- 96e7df62da6dcfb12f34e41ef9bfacbdc0260320
- ca79d6dbb97583eabf31ecf9fffcd18f62adbda0
- 7388c67527f07c534e3026b6dd81339be42697e2
および
は stable-3.6.x への（同一パッチの）コミットになります。（本チケットでは stable-3.6.x へのコミットについては無視して構わないと思います）
#10
Updated by Naoya Tozuka over 12 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#11
Updated by isao sano over 12 years ago
- Status changed from Pending Testing（テスト待ち） to Rejected（差し戻し）
- % Done changed from 70 to 50
http://redmine.openpne.jp/issues/2138#note-5
の件、再現いたします。
使用しているブランチはremotes/origin/release-3.6beta12です
以上認識がずれていました。
横線にて削除いたします。
#8での修正は確認いたしました。問題ありません。
#12
Updated by Kousuke Ebihara over 12 years ago
- Status changed from Rejected（差し戻し） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について読み違えていました（登録中にログイン後バナーが表示されてしまうのが問題であるとの指摘であると認識していた）。
ただし、この指摘はサイドバナーガジェットに限定した問題ではありません。サイドバナーガジェットに限定した問題ではない以上、このチケットで取り扱うべきではありません。そこで、この問題は別チケットで（3.6 までに）対応することにします。
#13
Updated by isao sano over 12 years ago
- Status changed from Pending Testing（テスト待ち） to Fixed（完了）
- % Done changed from 70 to 100
テスト完了いたしました。
問題ありませんので完了とします。
#14
Updated by Minoru Takai over 12 years ago
Updated by Kousuke Ebihara は書きました:
http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について読み違えていました（登録中にログイン後バナーが表示されてしまうのが問題であるとの指摘であると認識していた）。
ただし、この指摘はサイドバナーガジェットに限定した問題ではありません。サイドバナーガジェットに限定した問題ではない以上、このチケットで取り扱うべきではありません。そこで、この問題は別チケットで（3.6 までに）対応することにします。
この件については #1944 で修正する可能性があることを示しておきます。
#15
Updated by Minoru Takai over 12 years ago
親チケットよりも先にこのチケットが対応されており、 note-8 から note-11 での内容に誤解があったことを示しておきます。 note-8 での修正は、動作的には修正前と修正後で何も変化がないはずです。 親チケットのレビュー があるのでそちらも参考にしてください。