Project

General

Profile

Backport(バックポート) #4149

Bug(バグ) #4013: メールアドレス変更を行うと member_config に name='captcha' のレコードが追加される

メールアドレス変更を行うと member_config に name='captcha' のレコードが追加される

Added by Youichi Kimura almost 3 years ago. Updated over 1 year ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2016-09-23
Due date:
% Done:

100%


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 (修正内容)

修正内容を記入

History

#1 Updated by Youichi Kimura almost 3 years ago

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

下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/446

#2 Updated by Shinichi Urabe over 2 years ago

  • Parent task set to #4013

#4 Updated by Rimpei Ogawa over 1 year ago

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

#5 Updated by kaoru n over 1 year ago

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

#6 Updated by isao sano over 1 year ago

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

試験完了しました。
問題ありません。

#7 Updated by kaoru n over 1 year ago

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

マージしました

Also available in: Atom PDF