プロジェクト

全般

プロフィール

Bug(バグ) #2104

opFormItemGenerator の trim 指定の参照要素名が間違っている

Masato Nagasawaほぼ13年前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Urgent(急いで)
担当者:
対象バージョン:
開始日:
2011-05-16
期日:
進捗率:

100%

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

説明

Overview (現象)

trim の設定が IsRequired の設定になっている。

Causes (原因)

IsRequired を指定しているが、恐らくこの指定は間違っている。

$option = array('required' => $field['IsRequired'], 'trim' => $field['IsRequired']);

Way to fix (修正内容)

$field['IsTrim'] に修正し、
トリムが必要な項目については「IsTrim: true」を追加する。
(ただし、ガジェットのyml設定については修正範囲が大きいため対応しません)


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #3448: opFormItemGenerator の trim 指定の参照要素名が間違っている Fixed(完了) 2011-05-16
関連している OpenPNE 3 - Backport(バックポート) #3468: opFormItemGenerator の trim 指定の参照要素名が間違っている Invalid(無効) 2011-05-16

関係しているリビジョン

リビジョン 9a9d99dd (差分)
Masato Nagasawaほぼ13年前に追加

fixed "IsTrim" option (fixes #2104)

リビジョン f6991781 (差分)
Masato Nagasawa約12年前に追加

fixed "IsTrim" option (fixes #2104)

履歴

#1 Masato Nagasawaほぼ13年前に更新

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

#2 Masato Nagasawaほぼ13年前に更新

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

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

#3 Minoru Takaiほぼ13年前に更新

この修正に対しての直接の指摘ではありませんが、 opFormItemGenerator クラス内のメソッドで $field 変数で参照できる値を用いる箇所が多々ありますが、 $field 変数がどのようなキーを持った連想配列であることを前提としているのかがよく分かりません。

この修正について言えば、これまで yml ファイルには IsTrim の値は記述されていなかったため $field['IsTrim'] は未定義でした。今回の修正では opFormItemGenerator::generateWidget() では $field['IsTrim'] が定義されている前提としていますが、 opFormItemGenerator::generateWidget() が使われる時点で $field['IsTrim'] が定義されている保証はありません。

この指摘は今回の修正に限った話ではないためこのチケットで扱わなくてもよいでしょうが、もし未定義のキーを参照する可能性があることを問題とするならば、 $field[$key] を参照する前に予めデフォルト値を持った配列をマージ($field += $field_default)してから $field を参照するなどした方がよいかもしれません。

#4 Shouta Kashiwagi約12年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

#5 Youichi Kimura約12年前に更新

  • 優先度Normal(通常) から Urgent(急いで) に変更

#6 Yuma Sakata約12年前に更新

  • 対象バージョン252 から OpenPNE 3.8beta1 に変更
  • 3.6 で発生するかYes から Unknown (未調査) に変更
  • 3.4 で発生するかUnknown (未調査) にセット

#7 Masato Nagasawa約12年前に更新

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

#8 Yuya Watanabe約12年前に更新

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

#9 Yuma Sakata約12年前に更新

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

テストOKです。

#10 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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