Project

General

Profile

Bug(バグ) #3543

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

Added by Youichi Kimura almost 6 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2014-01-07
Due date:
% Done:

0%

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

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 to OpenPNE 3 - Backport(バックポート) #3723: MySQLのStrict Modeが有効な状態でメンバーの新規登録を行うとエラーが発生する Fixed(完了) 2014-01-07
Copied to OpenPNE 3 - Backport(バックポート) #3724: MySQLのStrict Modeが有効な状態でメンバーの新規登録を行うとエラーが発生する Fixed(完了) 2014-01-07

Associated revisions

Revision d18ed005 (diff)
Added by Youichi Kimura almost 6 years ago

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

History

#1 Updated by Youichi Kimura almost 6 years ago

  • Description updated (diff)

#2 Updated by Youichi Kimura almost 6 years ago

  • Description updated (diff)

#3 Updated by Youichi Kimura almost 6 years ago

  • Description updated (diff)

#4 Updated by Youichi Kimura almost 6 years ago

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

更新履歴 d18ed005dd5430d53141271dec226b2bfcd20b0d で適用されました。

#5 Updated by isao sano about 5 years ago

#6 Updated by isao sano about 5 years ago

#7 Updated by Shinichi Urabe almost 5 years ago

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

レビューOK

#10 Updated by kaoru n over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
  • % Done changed from 70 to 0

OpenPNE 3.8.14 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF