Project

General

Profile

Backport(バックポート) #4016

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

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

Added by Youichi Kimura about 8 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2012-06-14
Due date:
% Done:

100%


Description

現象

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

修正内容


Related issues

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

Associated revisions

Revision 0a3c5061 (diff)
Added by Youichi Kimura about 8 years ago

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

(cherry picked from commit 0b12caf5ad1009c17cdc86e2a3c030c0afb3c4ab)

Revision 295eac0c (diff)
Added by Youichi Kimura about 8 years ago

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

(cherry picked from commit 864bc2a4f25c2e4b942d2a6e837100c37f8a1e47)

Revision b6ad5f20 (diff)
Added by isao sano over 6 years ago

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

Revision 5f2e095e (diff)
Added by isao sano over 6 years ago

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

Revision 8c0cdae4
Added by kaoru n almost 6 years ago

Merge pull request #375 from upsilon/t-4016-4018

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

History

#1 Updated by Youichi Kimura about 8 years ago

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

#2 Updated by Youichi Kimura about 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Shinichi Urabe almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

#5 Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.23

#6 Updated by isao sano over 7 years ago

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

#7 Updated by isao sano over 7 years ago

#8 Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.23 to OpenPNE 3.8.x

#9 Updated by kaoru n almost 7 years ago

  • Parent task set to #3077

#10 Updated by kaoru n about 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.29

#11 Updated by Chiharu Nakajima about 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

動作確認OKです

#12 Updated by kaoru n about 6 years ago

  • Target version changed from OpenPNE 3.8.29 to OpenPNE 3.8.x

#13 Updated by kaoru n almost 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.30

#14 Updated by kaoru n almost 6 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF