プロジェクト

全般

プロフィール

Bug(バグ) #3077

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

Shinichi Urabe6年以上前に追加. 約1ヶ月前に更新.

ステータス:
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

履歴

#1 Shinichi Urabe6年以上前に更新

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

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

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

#3 Youichi Kimura2年以上前に更新

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

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

#4 Youichi Kimura2年以上前に更新

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

#5 Youichi Kimura2年以上前に更新

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

#6 Youichi Kimura2年以上前に更新

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

#8 Shinichi Urabe約2年前に更新

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

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

#9 isao sanoほぼ2年前に更新

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

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

#10 isao sanoほぼ2年前に更新

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

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

再現を確認しました。

#12 isao sano約1年前に更新

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

#13 isao sano約1年前に更新

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

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

#14 Rimpei Ogawa6ヶ月前に更新

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

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

#15 kaoru nishizoe6ヶ月前に更新

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

#16 kaoru nishizoe5ヶ月前に更新

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

#17 kaoru nishizoe2ヶ月前に更新

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

#18 Chiharu Nakajima約1ヶ月前に更新

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

動作確認OKです

#19 kaoru nishizoe約1ヶ月前に更新

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

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