Project

General

Profile

Actions

Bug(バグ) #2353

closed

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

Added by Mutsumi Imamura over 13 years ago. Updated over 9 years ago.

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

100%

Estimated time:
3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

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にする


Files


Related issues 4 (0 open4 closed)

Related to OpenPNE 3 - Backport(バックポート) #2364: ログインしてくださいというアラートが表示され続けるFixed(完了)Mutsumi Imamura2011-08-162011-10-05

Actions
Related to OpenPNE 3 - Backport(バックポート) #2365: ログインしてくださいというアラートが表示され続けるFixed(完了)Maki Takahashi2011-08-16

Actions
Related to OpenPNE 3 - Bug(バグ) #2373: メール通知テンプレート設定にてエラーのアラートが表示され続けるFixed(完了)Maki Takahashi2011-08-16

Actions
Related to opCommunityTopicPlugin - Bug(バグ) #2376: イベント参加画面にてエラーのアラートが表示され続けるFixed(完了)Maki Takahashi2011-08-16

Actions
Actions #1

Updated by Maki Takahashi over 13 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Maki Takahashi
Actions #2

Updated by Maki Takahashi over 13 years ago

  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか set to Yes
Actions #3

Updated by Shingo Yamada over 13 years ago

  • Priority changed from Normal(通常) to High(高め)
Actions #4

Updated by Shingo Yamada over 13 years ago

  • 360対象 set to RC1
Actions #5

Updated by Maki Takahashi over 13 years ago

  • Description updated (diff)
Actions #6

Updated by Maki Takahashi over 13 years ago

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

commit: a4bd10e2 にて修正いたしました。

再現手順を実行し、「ログインしてください」アラートが表示されている状態で
全体に公開の日記にアクセスした際に、アラートが表示されないことを確認しています。

Actions #7

Updated by Yuma Sakata over 13 years ago

  • 360対象 deleted (RC1)
Actions #8

Updated by Yuma Sakata over 13 years ago

  • 360対象 set to RC1
Actions #9

Updated by Rimpei Ogawa over 13 years ago

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

チケット表題の問題についての修正内容はレビューしましたが問題ありません。

ただ、masterブランチの a4bd10e2 時点のソースコード(バンドルプラグイン含む)で調べたところ、以下の2箇所で同様の原因でエラーメッセージが画面遷移後に残る問題が発生する可能性があります。

mailActions::executeTemplate()

apps/pc_backend/modules/mail/actions/actions.class.php
68:      $this->getUser()->setFlash('error', (string)$this->form->getErrorSchema());
69-    }
70-  }

opCommunityTopicPluginEventCommentActions::executeCreate()

plugins/opCommunityTopicPlugin/lib/action/opCommunityTopicPluginEventCommentActions.class.php
67:        $this->getUser()->setFlash('error', $e->getMessage());
68-      }   
69-    }
70-
71-    $this->setTemplate('../../communityEvent/templates/show');
72-
73-    return sfView::SUCCESS;
74-  }

Actions #10

Updated by Maki Takahashi over 13 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

note-9 について、それぞれチケットを作成いたします。

Actions #11

Updated by Maki Takahashi over 13 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • mailActions::executeTemplate() については #2373
  • opCommunityTopicPluginEventCommentActions::executeCreate()については #2376

にそれぞれチケットを作成いたしました。

Actions #12

Updated by Maki Takahashi over 13 years ago

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

本チケットの修正内容につきましては、すでにレビューいただき問題ないとのことですので
テスト待ちにいたします

Actions #13

Updated by Yuma Sakata about 13 years ago

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

テストOKです。

Actions #14

Updated by kaoru n over 9 years ago

  • 3.8 で発生するか set to Unknown (未調査)
Actions

Also available in: Atom PDF