プロジェクト

全般

プロフィール

Backport(バックポート) #4015

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

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

Youichi Kimura7年以上前に追加. 5年以上前に更新.

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

100%


説明

現象

メールアドレス変更処理が完了しても、仮登録用のデータ (***_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         }

修正内容


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #4017: メールアドレス設定から意図的に他のメンバーと同じメールアドレスを設定できる Fixed(完了) 2016-09-23

関係しているリビジョン

リビジョン 6855f0ee (差分)
Youichi Kimura7年以上前に追加

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

(cherry picked from commit 0b12caf5ad1009c17cdc86e2a3c030c0afb3c4ab)

リビジョン b88cbd94 (差分)
Youichi Kimura7年以上前に追加

remove unnecessary fetching *_token record (refs #4015, BP from #3077)

(cherry picked from commit 864bc2a4f25c2e4b942d2a6e837100c37f8a1e47)

リビジョン 94e802b6 (差分)
isao sano約6年前に追加

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

リビジョン 79298c6e (差分)
isao sano約6年前に追加

remove unnecessary fetching *_token record (refs #4015, BP from #3077)

リビジョン 2b7ab886
kaoru n5年以上前に追加

Merge pull request #374 from upsilon/t-4015-4017

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

履歴

#1 Youichi Kimura7年以上前に更新

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

#2 Youichi Kimura7年以上前に更新

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

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

#4 Shinichi Urabe約7年前に更新

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

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

#5 kaoru n約7年前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.26 に変更

#6 isao sano約7年前に更新

現在、セットで修正されている #4017 が差し戻し中のため、試験できません。

#7 isao sano約7年前に更新

#8 kaoru n約7年前に更新

  • 対象バージョンOpenPNE 3.6.26 から OpenPNE 3.6.x に変更

#9 kaoru n6年以上前に更新

  • 親チケット#3077 にセット

#10 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.32 に変更

#11 Chiharu Nakajima5年以上前に更新

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

動作確認OKです

#12 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.6.32 から OpenPNE 3.6.x に変更

#13 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.33 に変更

#14 kaoru n5年以上前に更新

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

マージしました

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