プロジェクト

全般

プロフィール

Backport(バックポート) #2138

トップバナー(ログイン前)とサイドバナー(ログイン前)が表示されない

Masato Nagasawaほぼ13年前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-10-14
期日:
進捗率:

100%


説明

Overview (現象)

http://sns.openpne.jp/communityTopic/6399 より転記

3.6beta5 トップバナー(ログイン前)とサイドバナー(ログイン前)が表示されません。

トップバナー(ログイン前)の掲載場所にはトップバナー(ログイン後)が表示され、サイドバナー(ログイン前)には何も表示されません。

FirefoxとChromeとIE8で確認しましたのでブラウザの影響ではないようです。
よろしくお願いいたします。

追記コメント

ガジェット設定でサイドバナーをweb全体に公開にしたら、ログインページにもサイドバナー(ログイン後)が表示されました。

トップバナー(ログイン前)とサイドバナー(ログイン前)は項目があるだけで未実装?

Causes (原因)

バナー表示は、ログイン/未ログインを sfUser::getMember() がインスタンスを返すかどうかで判断している。
しかし、3.6系では外部公開機能に対応するため未ログイン時には opAnonymousMember を返すように変更されている、
このことが原因でログイン判定がうまく機能しなくなり、正しく表示されていない。

追記コメントにあるサイドバナーを"web全体に公開"にした場合は表示された件は、
正常な挙動であり、"全員に公開"の場合ログインしていない状態では表示されない。

Way to fix (修正内容)

opAnonymousMember のインスタンスでないかどうかをログイン判定の条件に加える。


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1678: トップバナー(ログイン前)とサイドバナー(ログイン前)が表示されない Fixed(完了) 2010-10-14
関連している OpenPNE 3 - Bug(バグ) #2285: 仮登録中のユーザーで公開範囲外のガジェットが表示される Invalid(無効) 2011-07-21

関係しているリビジョン

リビジョン 96e7df62 (差分)
Masato Nagasawa12年以上前に追加

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

リビジョン ca79d6db (差分)
Kousuke Ebihara12年以上前に追加

changed way to check whether an instance of member is opAnonymousMember in side banner gadget (refs #2138, BP from #1678)

リビジョン 7388c675 (差分)
Kousuke Ebihara12年以上前に追加

fixed "$sf_user->getMember()->getRawValue()" causes fatal error when registration process (refs #2138, BP from #1678)

リビジョン 2c407873 (差分)
Masato Nagasawa12年以上前に追加

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

リビジョン 78767885 (差分)
Kousuke Ebihara12年以上前に追加

changed way to check whether an instance of member is opAnonymousMember in side banner gadget (refs #2138, BP from #1678)

リビジョン a115e44d (差分)
Kousuke Ebihara12年以上前に追加

fixed "$sf_user->getMember()->getRawValue()" causes fatal error when registration process (refs #2138, BP from #1678)

リビジョン dc11cddc (差分)
Kousuke Ebihara12年以上前に追加

fixed that a user in registration process could view after-login banner (refs #2138, BP from #1678)

リビジョン ecd53be6 (差分)
Kousuke Ebihara12年以上前に追加

fixed that a user in registration process could view after-login banner (refs #2138, BP from #1678)

履歴

#1 Kousuke Ebiharaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE3.6beta12 に変更

親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。

#2 Kousuke Ebihara12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Masato Nagasawa から Kousuke Ebihara に変更

バックポート引き受けます

#3 Kousuke Ebihara12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#4 Naoya Tozuka12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

BP修正確認しました。OKです。

#5 isao sano12年以上前に更新

以下の手順を行った際、サイドバナーの公開範囲が「全体に公開」にも関わらず、サイドバナー(ログイン前)が表示されます。

1. 管理画面より、サイドバナーガジェット設定(/pc_backend.php/design/gadget/type/sideBanner)を「全員に公開」に設定する
2. サイドバナー(ログイン前)設定(/design/banner/id/2)に遷移する
3. 「任意HTMLで表示する」にチェックを入れ、文字を入力する。
4. 管理画面より、招待メールを送る。
5. 招待メールを開き、書かれているURLに遷移する。

※仮登録の状態以外では、サイドバナーの公開範囲が「全体に公開」になっている場合、サイドバナー(ログイン前)は表示されません。

#6 isao sano12年以上前に更新

現在テストで「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 isao sano12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

#8 Kousuke Ebihara12年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について、 dc11cddc121b1ded3f9daa32d2d310596d4174fd で対応しました。

#9 Naoya Tozuka12年以上前に更新

メモ

http://redmine.openpne.jp/issues/2138#note-6 の件ですが

および

が release-3.6beta12 へのコミットで、

および

は stable-3.6.x への(同一パッチの)コミットになります。(本チケットでは stable-3.6.x へのコミットについては無視して構わないと思います)

#10 Naoya Tozuka12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

note-8 の修正対応箇所 (dc11cddc) 確認しました。OKです。

メモ

$sf_user->getMember() が instanceof opAnonymousMember の時、$sf_user->isSNSMember() は false を返す。
(←→ $m instanceof opAnonymousMember のとき、$m->getIsActive() = false)

#11 isao sano12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

http://redmine.openpne.jp/issues/2138#note-5
の件、再現いたします。
使用しているブランチはremotes/origin/release-3.6beta12です

以上認識がずれていました。
横線にて削除いたします。
#8での修正は確認いたしました。問題ありません。

#12 Kousuke Ebihara12年以上前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について読み違えていました(登録中にログイン後バナーが表示されてしまうのが問題であるとの指摘であると認識していた)。

ただし、この指摘はサイドバナーガジェットに限定した問題ではありません。サイドバナーガジェットに限定した問題ではない以上、このチケットで取り扱うべきではありません。そこで、この問題は別チケットで(3.6 までに)対応することにします。

#13 isao sano12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テスト完了いたしました。
問題ありませんので完了とします。

#14 Minoru Takai12年以上前に更新

Kousuke Ebihara は書きました:

http://redmine.openpne.jp/issues/2138#note-5 の指摘事項について読み違えていました(登録中にログイン後バナーが表示されてしまうのが問題であるとの指摘であると認識していた)。

ただし、この指摘はサイドバナーガジェットに限定した問題ではありません。サイドバナーガジェットに限定した問題ではない以上、このチケットで取り扱うべきではありません。そこで、この問題は別チケットで(3.6 までに)対応することにします。

この件については #1944 で修正する可能性があることを示しておきます。

#15 Minoru Takai12年以上前に更新

親チケットよりも先にこのチケットが対応されており、 note-8 から note-11 での内容に誤解があったことを示しておきます。 note-8 での修正は、動作的には修正前と修正後で何も変化がないはずです。 親チケットのレビュー があるのでそちらも参考にしてください。

他の形式にエクスポート: Atom PDF