Project

General

Profile

Actions

Bug(バグ) #3077

closed

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

Added by Shinichi Urabe over 12 years ago. Updated about 6 years ago.

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

100%

Estimated time:
(Total: 0:00 h)
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 2 (0 open2 closed)

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

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

Actions
Actions #1

Updated by Shinichi Urabe over 12 years ago

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

Updated by 開 石切山 about 12 years ago

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

Updated by Youichi Kimura over 8 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

Actions #4

Updated by Youichi Kimura over 8 years ago

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

Updated by Youichi Kimura over 8 years ago

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

Updated by Youichi Kimura over 8 years ago

  • Target version set to OpenPNE 3.9.0-old
Actions #8

Updated by Shinichi Urabe about 8 years ago

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

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

Actions #9

Updated by isao sano almost 8 years ago

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

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

Actions #10

Updated by isao sano almost 8 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
Actions #11

Updated by isao sano over 7 years ago

再現を確認しました。

Actions #12

Updated by isao sano almost 7 years ago

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

Updated by isao sano almost 7 years ago

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

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

Actions #14

Updated by Rimpei Ogawa over 6 years ago

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

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

Actions #15

Updated by kaoru n over 6 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.3
Actions #16

Updated by kaoru n over 6 years ago

  • Target version changed from OpenPNE-3.9.3 to OpenPNE 3.9.0
Actions #17

Updated by kaoru n about 6 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.4
Actions #18

Updated by Chiharu Nakajima about 6 years ago

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

動作確認OKです

Actions #19

Updated by kaoru n about 6 years ago

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

Also available in: Atom PDF