Actions
Backport(バックポート) #4149
closedBug(バグ) #4013: メールアドレス変更を行うと member_config に name='captcha' のレコードが追加される
メールアドレス変更を行うと member_config に name='captcha' のレコードが追加される
Start date:
2016-09-23
Due date:
% Done:
100%
Estimated time:
Description
Overview (現象)¶
メールアドレス変更を行うと、member_config に下記のような name = 'captcha' のレコードが追加される。
> SELECT * FROM member_config WHERE name = 'captcha' \G *************************** 1. row *************************** id: 145 member_id: 2 name: captcha value: Array value_datetime: NULL name_value_hash: 7fee6c903f71b6b14b0b6b6636ab3f5b created_at: 2016-08-31 15:30:37 updated_at: 2016-08-31 15:30:37
Causes (原因)¶
source:lib/form/MemberConfigForm/MemberConfigPcAddressForm.class.php@43190aa1#L28:
public function __construct(Member $member = null, $options = array(), $CSRFSecret = null)
{
parent::__construct($member, $options, $CSRFSecret);
if (sfConfig::get('op_is_use_captcha', false))
{
$this->embedForm('captcha', new opCaptchaForm());
}
}
MemberConfigPcAddressForm, MemberConfigMobileAddressForm では embedForm を使用して opCaptchaForm を追加している。一方で、saveConfig メソッドでは追加された captcha フィールドをそのまま member_config テーブルに格納しようとするため上記のような現象が発生する。
Way to fix (修正内容)¶
修正内容を記入
Updated by Youichi Kimura over 7 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/446
Updated by Rimpei Ogawa over 6 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
Updated by kaoru n over 6 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.27
Updated by isao sano over 6 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
試験完了しました。
問題ありません。
Updated by kaoru n over 6 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました
Actions