Bug（バグ） #2104: opFormItemGenerator の trim 指定の参照要素名が間違っている - OpenPNE 3 - OpenPNE Issue Tracking System

操作

リンクをコピー

Bug（バグ） #2104

完了

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

Masato Nagasawa さんが13年以上前に追加. 約9年前に更新.

ステータス:

Fixed（完了）

優先度:

Urgent（急いで）

担当者:

Masato Nagasawa

対象バージョン:

OpenPNE 3.8beta1

開始日:

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設定については修正範囲が大きいため対応しません）

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

操作

リンクをコピー

Masato Nagasawa さんが13年以上前に更新

ステータス を New（新規） から Accepted（着手） に変更
担当者 を Masato Nagasawa にセット

操作

リンクをコピー

Masato Nagasawa さんが13年以上前に更新

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

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

操作

リンクをコピー

Minoru Takai さんが13年以上前に更新

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

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

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

操作

リンクをコピー