プロジェクト

全般

プロフィール

Bug(バグ) #4013

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

Youichi Kimura約2年前に追加. 7ヶ月前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2016-09-23
期日:
進捗率:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

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

修正内容を記入


子チケット

Backport(バックポート) #4148: メールアドレス変更を行うと member_config に name='captcha' のレコードが追加されるFixed(完了)Youichi Kimura

Backport(バックポート) #4149: メールアドレス変更を行うと member_config に name='captcha' のレコードが追加されるFixed(完了)Youichi Kimura

履歴

#1 Youichi Kimura約2年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Youichi Kimura にセット

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

#2 Youichi Kimura約2年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#3 kaoru nishizoe1年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

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

#5 isao sano8ヶ月前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#6 isao sano8ヶ月前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#7 Rimpei Ogawa7ヶ月前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#8 kaoru nishizoe7ヶ月前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.1 に変更

#9 kaoru nishizoe7ヶ月前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF