Project

General

Profile

Bug(バグ) #3428

Task(タスク) #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う

opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する

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

Status:
Won't fix(対応せず)
Priority:
Low(低め)
Target version:
Start date:
2013-10-07
Due date:
% Done:

0%

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

Description

Overview (現象)

source:lib/config/config/sns_config.yml などの opFormItemGenerator で自動生成するフィールドの定義では、しばしば IsRequired や IsTrim が省略されているが、opFormItemGenerator::generateValidator() の下記のコードはそれらの省略を考慮されておらず E_NOTICE レベルのエラーを発生させてしまう。

  public static function generateValidator($field, $choices = array())
  {     
    $field = self::arrayKeyCamelize($field);
    $option = array('required' => $field['IsRequired'], 'trim' => $field['IsTrim']);

Way to fix (修正内容)

isset() を使用し IsRequired 等が定義されていることを確認した上で値を使用するように修正する


Related issues

Related to OpenPNE 3 - Backport(バックポート) #3437: opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する Fixed(完了) 2013-10-07
Related to OpenPNE 3 - Backport(バックポート) #3463: opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する Fixed(完了) 2013-10-07
Duplicated by OpenPNE 3 - Bug(バグ) #3167: IsTrim の undefined Index notice Invalid(無効) 2012-08-18

Associated revisions

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

make IsRequired and IsTrim optional in opFormItemGenerator (fixes #3428)

History

#1 Updated by Youichi Kimura almost 6 years ago

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

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

#2 Updated by Shinichi Urabe almost 6 years ago

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

レビューOKです
今まで、trim, required が未定義のため null となっていたものに false が渡されることになりますが、 sfValidatorBase::clean() や opValidatorString::clean() の処理を確認する限り、動作は問題ないと考えます。

#3 Updated by 誠二 天重 almost 5 years ago

#4 Updated by Chiharu Nakajima over 4 years ago

  • Priority changed from Normal(通常) to Low(低め)

#5 Updated by Chiharu Nakajima over 4 years ago

このチケットの内容は下記の項目に該当する不具合であるため、一旦優先度を下げます。
・未対応でもユーザーに影響が少ない不具合(回避可能な不具合である等)

#7 Updated by isao sano over 2 years ago

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

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

Also available in: Atom PDF