プロジェクト

全般

プロフィール

Bug(バグ) #2353

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

Mutsumi Imamura12年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2011-08-16
期日:
進捗率:

100%

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

説明

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

スクリーンショット(2011-08-16_14.42.29).PNG 表示 (88.1 KB) Mutsumi Imamura, 2011-08-16 14:53


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #2364: ログインしてくださいというアラートが表示され続ける Fixed(完了) 2011-08-16 2011-10-05
関連している OpenPNE 3 - Backport(バックポート) #2365: ログインしてくださいというアラートが表示され続ける Fixed(完了) 2011-08-16
関連している OpenPNE 3 - Bug(バグ) #2373: メール通知テンプレート設定にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16
関連している opCommunityTopicPlugin - Bug(バグ) #2376: イベント参加画面にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16

関係しているリビジョン

リビジョン a4bd10e2 (差分)
Maki Takahashi12年以上前に追加

(refs #2353) chenged sefFlash() persist param to false in member/login

履歴

#1 Maki Takahashi12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット

#2 Maki Takahashi12年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 にセット
  • 3.6 で発生するかYes にセット

#3 Shingo Yamada12年以上前に更新

  • 優先度Normal(通常) から High(高め) に変更

#4 Shingo Yamada12年以上前に更新

  • 360対象RC1 にセット

#5 Maki Takahashi12年以上前に更新

  • 説明 を更新 (diff)

#6 Maki Takahashi12年以上前に更新

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

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

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

#7 Yuma Sakata12年以上前に更新

  • 360対象 を削除 (RC1)

#8 Yuma Sakata12年以上前に更新

  • 360対象RC1 にセット

#9 Rimpei Ogawa12年以上前に更新

  • ステータスPending Review(レビュー待ち) から 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-  }

#10 Maki Takahashi12年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

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

#11 Maki Takahashi12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • mailActions::executeTemplate() については #2373
  • opCommunityTopicPluginEventCommentActions::executeCreate()については #2376

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

#12 Maki Takahashi12年以上前に更新

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

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

#13 Yuma Sakata12年以上前に更新

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

テストOKです。

#14 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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