プロジェクト

全般

プロフィール

Bug(バグ) #3077

メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない

Shinichi Urabeほぼ12年前に追加. 5年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2012-06-14
期日:
進捗率:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

説明

現象

メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) がMemberConfig に残ってしまい、削除されない

原因

opDoctrineRecord::save() で正常に保存されても返り値がないため、
opMemberAction::executeConfigComplete() の以下の if 文内の処理が実施されない。

262         if ($config->save())
263         {
264           $pre->delete();
265           $token = Doctrine::getTable('MemberConfig')->retrieveByNameAndMemberId($type.'_token', $memberId);
266           $token->delete();
267         }

修正内容


子チケット

Backport(バックポート) #4015: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されないFixed(完了)Youichi Kimura

Backport(バックポート) #4016: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されないFixed(完了)Youichi Kimura

関係しているリビジョン

リビジョン 0b92e1dd (差分)
Youichi Kimura約6年前に追加

fix member/configConfirm action doesn't delete *_pre and *_token records (fixes #3077)

リビジョン 51f875c9 (差分)
Youichi Kimura約6年前に追加

remove unnecessary fetching *_token record (refs #3077)

リビジョン 59665caf
kaoru n5年以上前に追加

Merge pull request #503 from isaosano/t-4012

メールアドレス変更に関する複数の修正 (fixes #3077, #4012)

履歴

#1 Shinichi Urabeほぼ12年前に更新

  • 題名メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) MemberConfig に残ってしまい、削除されない から メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない に変更
  • 説明 を更新 (diff)

#2 開 石切山11年以上前に更新

  • 3.8 で発生するかYes (はい) にセット

#3 Youichi Kimura7年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura にセット
  • 進捗率0 から 50 に変更

下記の Pull Request にて修正しました (#3077, #4012 の修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/372

#4 Youichi Kimura7年以上前に更新

  • 関連している Backport(バックポート) #4015: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない を追加

#5 Youichi Kimura7年以上前に更新

  • 関連している Backport(バックポート) #4016: メールアドレス変更処理が完了しても、仮登録用のデータ (***_pre, ***_token) が MemberConfig に残ってしまい、削除されない を追加

#6 Youichi Kimura7年以上前に更新

  • 対象バージョンOpenPNE 3.9.0-old にセット

#8 Shinichi Urabe7年以上前に更新

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

レビューは OK ですが、修正が #4012 とセットであるため、 #4012 が OK となってからテストを進めてください

#9 isao sano約7年前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#10 isao sano約7年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

#11 isao sanoほぼ7年前に更新

再現を確認しました。

#12 isao sano約6年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#13 isao sano約6年前に更新

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

https://github.com/openpne/OpenPNE3/pull/497 にてプルリクエストしました。

#14 Rimpei Ogawa5年以上前に更新

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

#4012 修正部分については差し戻しとなっています
https://redmine.openpne.jp/issues/4012#note-13

#15 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.3 に変更

#16 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE-3.9.3 から OpenPNE 3.9.0 に変更

#17 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.4 に変更

#18 Chiharu Nakajima5年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

動作確認OKです

#19 kaoru n5年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

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