プロジェクト

全般

プロフィール

Backport(バックポート) #3724

完了

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

isao sano さんが約10年前に追加. 約10年前に更新.

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

100%

予定工数:

説明

現象

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'] のチェックを行う。


関連するチケット 1 (0件未完了1件完了)

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

操作

isao sano さんが約10年前に更新

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

isao sano さんが約10年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

isao sano さんが約10年前に更新

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

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

Shinichi Urabe さんが約10年前に更新

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

isao sano さんが約10年前に更新

  • 担当者isao sano にセット

isao sano さんが約10年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

Shinichi Urabe さんが約10年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

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