Project

General

Profile

Backport(バックポート) #2365

ログインしてくださいというアラートが表示され続ける

Added by Maki Takahashi almost 9 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-08-16
Due date:
% Done:

100%


Description

Overview (現象)

未ログイン状態で閲覧にログインが必要なページにアクセスすると「ログインしてください」というアラートが表示されるが、その後、閲覧にログインが不要なページ(例:Web全体に公開の日記)にアクセスするとアラートが表示されたままになってしまう。

閲覧にログインが不要なページなのにこのようなアラートが表示されるのは不適切であると考えられる。

再現バージョン

  • OpenPNE3.7 masterブランチ
  • OpenPNE3.6beta13-dev
  • OpenPNE3.4.15-dev

再現手順

1.メンバーAで未ログインの状態でログイン必要なページにアクセスする
 (例:未ログイン状態で diary/new にアクセスする)
2.ログインしてくださいというアラートが表示されることを確認する
3.手順3の状態でweb全体に公開されている日記にアクセスする
4.「ログインしてください」の表示がされたままになる

Causes (原因)

ログインアクション(member/login)において、「ログインしてください」メッセージを
$this->getUser()->setFlash()を利用してセットしているが
上記メソッドは次のアクションまでセットされたメッセージを保持するため
ログイン画面から別のページ(ログイン画面に再度アクセスした場合も含む)に行った場合でも
メッセージが表示され続けてしまっている

Way to fix (修正内容)

$this->getUser()->setFlash()の第3引数($persist)をfalseにする


Related issues

Related to OpenPNE 3 - Bug(バグ) #2353: ログインしてくださいというアラートが表示され続ける Fixed(完了) 2011-08-16

Associated revisions

Revision 23d39a23 (diff)
Added by Maki Takahashi over 8 years ago

(fixes #2365, BP from #2353) chenged sefFlash() persist param to false in member/login

History

#1 Updated by Maki Takahashi over 8 years ago

  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Anonymous over 8 years ago

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

更新履歴 23d39a23f5a139c2caa5adfed8f00afa78d14094 で適用されました。

#3 Updated by Kousuke Ebihara over 8 years ago

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

#4 Updated by Yuma Sakata over 8 years ago

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

テストOKです。

Also available in: Atom PDF