Project

General

Profile

Backport(バックポート) #2138

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

Added by Masato Nagasawa over 9 years ago. Updated about 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-10-14
Due date:
% Done:

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

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

Associated revisions

Revision 96e7df62 (diff)
Added by Masato Nagasawa over 9 years ago

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

Revision ca79d6db (diff)
Added by Kousuke Ebihara over 9 years ago

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

Revision 7388c675 (diff)
Added by Kousuke Ebihara over 9 years ago

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

Revision 2c407873 (diff)
Added by Masato Nagasawa over 9 years ago

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

Revision 78767885 (diff)
Added by Kousuke Ebihara over 9 years ago

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

Revision a115e44d (diff)
Added by Kousuke Ebihara over 9 years ago

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

Revision dc11cddc (diff)
Added by Kousuke Ebihara over 9 years ago

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

Revision ecd53be6 (diff)
Added by Kousuke Ebihara over 9 years ago

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

History

#1 Updated by Kousuke Ebihara over 9 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE3.6beta12

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

#2 Updated by Kousuke Ebihara over 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee changed from Masato Nagasawa to Kousuke Ebihara

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

#3 Updated by Kousuke Ebihara over 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#4 Updated by Naoya Tozuka over 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

#5 Updated by isao sano over 9 years ago

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

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

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

#6 Updated by isao sano over 9 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 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

#8 Updated by Kousuke Ebihara over 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#9 Updated by Naoya Tozuka over 9 years ago

メモ

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

および

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

および

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

#10 Updated by Naoya Tozuka over 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

メモ

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

#11 Updated by isao sano over 9 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 9 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 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

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

#14 Updated by Minoru Takai about 9 years ago

Kousuke Ebihara は書きました:

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

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

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

#15 Updated by Minoru Takai about 9 years ago

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

Also available in: Atom PDF