Project

General

Profile

Actions

Backport(バックポート) #3724

closed

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

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

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

100%

Estimated time:

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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by isao sano about 10 years ago

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

Updated by isao sano about 10 years ago

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

Updated by isao sano about 10 years ago

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

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

Actions #4

Updated by Shinichi Urabe almost 10 years ago

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

Updated by isao sano almost 10 years ago

  • Assignee set to isao sano
Actions #6

Updated by isao sano almost 10 years ago

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

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

Actions #7

Updated by Shinichi Urabe almost 10 years ago

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

Also available in: Atom PDF