プロジェクト

全般

プロフィール

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 に変更する。


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1841: pc_frontend から利用可能な認証プラグインがない状態でメール投稿がおこなえない Fixed(完了) 2010-12-10
関連している OpenPNE 3 - Bug(バグ) #2344: 携帯で新規登録が出来ない Fixed(完了) 2011-08-10

関係しているリビジョン

リビジョン 72c1860b (差分)
Masato Nagasawaほぼ13年前に追加

fixed opSecurityUser to sfBasicSecurityUser for the mobile_mail_frontend app (fixes #2044 BP from #1841)

Conflicts:

apps/mobile_mail_frontend/lib/myUser.class.php

リビジョン da5ac61e (差分)
Masato Nagasawaほぼ13年前に追加

Revert "fixed opSecurityUser to sfBasicSecurityUser for the mobile_mail_frontend app (fixes #2044 BP from #1841)"

This reverts commit 72c1860b9ea9e5b0d2888923d0057ba92baf9588.

リビジョン 0f89576c (差分)
Masato Nagasawaほぼ13年前に追加

fixed opSecurityUser to sfBasicSecurityUser for the mobile_mail_frontend app (fixes #2044 BP from #1841)

Conflicts:

apps/mobile_mail_frontend/lib/myUser.class.php

履歴

#1 Masato Nagasawaほぼ13年前に更新

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

#2 Masato Nagasawaほぼ13年前に更新

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

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

#3 Masato Nagasawaほぼ13年前に更新

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

#4 Masato Nagasawaほぼ13年前に更新

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

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

#5 Masato Nagasawaほぼ13年前に更新

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

#6 Masato Nagasawaほぼ13年前に更新

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

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

#7 Yuma Sakataほぼ13年前に更新

テストOKです。

#8 Minoru Takaiほぼ13年前に更新

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

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

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

#9 Minoru Takaiほぼ13年前に更新

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

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

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