Bug(バグ) #2104
opFormItemGenerator の trim 指定の参照要素名が間違っている
開始日:
2011-05-16
期日:
進捗率:
100%
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
説明
関連するチケット
履歴
#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です。