Project

General

Profile

Bug(バグ) #3077

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

Added by Shinichi Urabe about 7 years ago. Updated 7 months ago.

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

100%

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

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         }

修正内容


Subtasks

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

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

History

#1 Updated by Shinichi Urabe about 7 years ago

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

#2 Updated by 開 石切山 almost 7 years ago

  • 3.8 で発生するか set to Yes (はい)

#3 Updated by Youichi Kimura almost 3 years ago

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

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

#4 Updated by Youichi Kimura almost 3 years ago

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

#5 Updated by Youichi Kimura almost 3 years ago

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

#6 Updated by Youichi Kimura almost 3 years ago

  • Target version set to OpenPNE 3.9.0-old

#8 Updated by Shinichi Urabe over 2 years ago

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

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

#9 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

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

#10 Updated by isao sano over 2 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

#11 Updated by isao sano over 2 years ago

再現を確認しました。

#12 Updated by isao sano over 1 year ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#13 Updated by isao sano over 1 year ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#14 Updated by Rimpei Ogawa 12 months ago

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

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

#15 Updated by kaoru n 12 months ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.3

#16 Updated by kaoru n 11 months ago

  • Target version changed from OpenPNE-3.9.3 to OpenPNE 3.9.0

#17 Updated by kaoru n 8 months ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.4

#18 Updated by Chiharu Nakajima 7 months ago

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

動作確認OKです

#19 Updated by kaoru n 7 months ago

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

Also available in: Atom PDF