プロジェクト

全般

プロフィール

Backport(バックポート) #2044

完了

pc_frontend から利用可能な認証プラグインがない状態でメール投稿がおこなえない

Masato Nagasawa さんが13年以上前に追加. 13年以上前に更新.

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

100%

予定工数:

説明

Overview (現象)

pc_frontend から利用可能な認証プラグインがない状態でメール投稿がおこなえない。

再現手順

1. 管理画面において、すべての認証プラグインの「PC版有効設定」を「無効にする」に変更する
2. メール投稿をおこなう
3. サーバ側に以下のようなエラーメッセージが記録されることを確認する

PHP Fatal error:  Cannot instantiate abstract class opAuthAdapter in /path/to/OpenPNE/lib/user/sfOpenPNESecurityUser.class.php on line 109

Causes (原因)

携帯メール経由で OpenPNE へアクセスした際に、 opSecurityUser がそのアクセスを携帯からのものとみなしておらず、 PC の設定を確認しに行くために、 opSecurityUser を初期化した際に利用可能な認証プラグインがないと判断され、エラーとなってしまう。

mobile_mail_frontend のアクションを実行するのだから、携帯として扱うのが自然。また、そもそも mobile_mail_frontend で opSecurityUser を使う意味はないはずなので、 sfBasicSecurityUser に変更するなどの対応も検討する。

Way to fix (修正内容)

myUser の親クラスを opSecurityUser から sfBasicSecurityUser に変更する。


関連するチケット 2 (0件未完了2件完了)

関連している OpenPNE 3 - Bug(バグ) #1841: pc_frontend から利用可能な認証プラグインがない状態でメール投稿がおこなえないFixed(完了)Masato Nagasawa2010-12-10

操作
関連している OpenPNE 3 - Bug(バグ) #2344: 携帯で新規登録が出来ないFixed(完了)Shinichi Urabe2011-08-10

操作

Masato Nagasawa さんが13年以上前に更新

  • 対象バージョンOpenPNE 3.4.12 から OpenPNE 3.4.13 に変更

Masato Nagasawa さんが13年以上前に更新

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

更新履歴 72c1860b9ea9e5b0d2888923d0057ba92baf9588 で適用されました。

Masato Nagasawa さんが13年以上前に更新

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

Masato Nagasawa さんが13年以上前に更新

  • ステータスPending Review(レビュー待ち) から New(新規) に変更
  • 担当者 を削除 (Masato Nagasawa)

3.4.12 では取り込みを行わないので、一旦取り消します。

Masato Nagasawa さんが13年以上前に更新

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

Masato Nagasawa さんが13年以上前に更新

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

更新履歴 0f89576cf9cd1c8364eb39f885d5a0bd20c9f734 で適用されました。

Yuma Sakata さんが13年以上前に更新

テストOKです。

Minoru Takai さんが13年以上前に更新

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

修正方針については、実装者によって検討されている経緯を見るかぎりは妥当そうだと判断できます。

修正内容についてですが、コーディング規約の観点からも問題はなさそうだと判断できるのでコードチェックはOKです。

Minoru Takai さんが13年以上前に更新

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

テストは既に行なわれているため、 note-7 のチェックを以てテスト完了とし、チケットをクローズします。

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