Project

General

Profile

Backport(バックポート) #3724

MySQLのStrict Modeが有効な状態でメンバーの新規登録を行うとエラーが発生する

Added by isao sano about 5 years ago. Updated almost 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2014-01-07
Due date:
% Done:

100%


Description

現象

MySQLの Strict Mode が有効な環境でメンバーの新規登録を行う際に プリセット でかつ 日付 の項目を入力すると、次のようなエラーが発生する。

SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect datetime value: '' for column 'value_datetime' at row 1

エラーが発生した SQL は下記の通り。

12月 27 17:23:12 symfony [info] {Doctrine_Connection_Statement} execute : INSERT INTO member_profile (value, member_id, profile_id, value_datetime, created_at, updated_at) VALUES (?, ?, ?, ?, ?, ?) - (, 6, 2, , 2013-12-27 17:23:12, 2013-12-27 17:23:12)

再現手順

  1. MySQL の my.cnf に下記の設定を加える
    [mysqld]
    sql_mode=STRICT_ALL_TABLES
    
  2. 管理画面から招待メールを送信し、メールに記載されたURLから新規登録ページ (/member/register) を開く
  3. プロフィール入力 (/member/registerInput) へ進む
  4. 必須項目と 誕生日 を入力する
  5. 「送信」をクリックすると500エラーが発生する

原因

MemberProfile::preSave() 内の下記のコードが直接の原因となっている。

    elseif ('date' === $this->getFormType() && isset($modified['value']) && $this->getProfile()->isPreset())
    {   
      $this->_set('value_datetime', $this->_get('value'));
    }

config/doctrine/schema.ymlMemberProfile.value のデフォルト値が "" であると記述されているため、value に変更が加えられたか否かに係わらず MemberProfile::preSave() 内で isset($modified['value']) は常に true になる(UPDATE の場合を除く)。
すると MemberProfile.value_datetime に空文字列がそのままセットされてしまい、前述の Invalid datetime format エラーが発生してしまう。

なお、プリセットではない日付型のプロフィール項目については value_datetime は使用されない (参照: b6445a0a) ためこの問題は発生しない。

修正内容

MemberProfile::preSave() 内で value_datetime に対するから文字列の代入が行われないように $modified['value'] のチェックを行う。


Related issues

Copied from OpenPNE 3 - Bug(バグ) #3543: MySQLのStrict Modeが有効な状態でメンバーの新規登録を行うとエラーが発生する Won't fix(対応せず) 2014-01-07

Associated revisions

Revision 594dc40d (diff)
Added by isao sano about 5 years ago

fix MemberProfile::preSave() set empty string to value_datetime if value is empty (fixes #3724, BP from #3543)

Revision b03fcccd
Added by Shinichi Urabe almost 5 years ago

Merge pull request #220 from isaosano/t-3724

fix MemberProfile::preSave() set empty string to value_datetime if value is empty (fixes #3724, BP from #3543)

History

#1 Updated by isao sano about 5 years ago

  • Copied from Bug(バグ) #3543: MySQLのStrict Modeが有効な状態でメンバーの新規登録を行うとエラーが発生する added

#2 Updated by isao sano about 5 years ago

  • Status changed from New(新規) to Accepted(着手)

#3 Updated by isao sano about 5 years ago

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

https://github.com/openpne/OpenPNE3/pull/220
にてプルリクエストを行いました

#4 Updated by Shinichi Urabe about 5 years ago

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

#5 Updated by isao sano about 5 years ago

  • Assignee set to isao sano

#6 Updated by isao sano almost 5 years ago

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

単体試験完了しました。
マージお願いたします。

#7 Updated by Shinichi Urabe almost 5 years ago

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

Also available in: Atom PDF