Project

General

Profile

Backport(バックポート) #1469

opFormItemGenerator::generateValidator() の 数値用バリデータ生成時 ValueMax, ValueMin が 0の場合は 範囲指定されない

Added by Shogo Kawahara about 12 years ago. Updated almost 12 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-07-28
Due date:
% Done:

100%


Description

Overview

opFormItemGenerator::generateValidator() の 数値用バリデータ生成時 ValueMax, ValueMin が 0の場合は 範囲指定されない


再現手順

  • 範囲が 0 〜 1 である数値プロフィール項目を作成。
  • プロフィール編集で -1 を入力
  • 正常にプロフィールが登録できてしまう。

対象

OpenPNE3.0.x 〜 OpenPNE3.7.x-dev

Causes

184     if ($field['ValueType'] === 'integer' || $field['FormType'] === 'date')
185     {   
186         if (!empty($field['ValueMin']))
187         {
188           $option['min'] = $field['ValueMin'];
189         }
190         if (!empty($field['ValueMax']))
191         {
192           $option['max'] = $field['ValueMax'];
193         }
194     } 

ValueMinおよびValueMax を empty() を利用して判定しているためです。

Way to fix


Related issues

Related to OpenPNE 3 - Bug(バグ) #1449: opFormItemGenerator::generateValidator() の 数値用バリデータ生成時 ValueMax, ValueMin が 0の場合は 範囲指定されない Fixed(完了) 2010-07-28

Associated revisions

Revision d2665b4e (diff)
Added by Shogo Kawahara almost 12 years ago

fixed opFormItemGenerator doesn't set min or max option to sfValidateInteger when ValueMin or ValueMax is "0" (fixes #1469, BP from #1449)

Revision 2d62d074 (diff)
Added by Shogo Kawahara almost 12 years ago

fixed MemberProfileForm for readability of code (fixes #1469, BP from #1449)

Revision bbc075e1 (diff)
Added by Shogo Kawahara almost 12 years ago

fixed opFormItemGenerator::generateValidator() to check option values (fixes #1469, BP from #1449)

History

#1 Updated by Shinichi Urabe almost 12 years ago

  • Target version changed from OpenPNE 3.4.7 to OpenPNE 3.4.8

#2 Updated by Shinichi Urabe almost 12 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Shinichi Urabe

#3 Updated by Shogo Kawahara almost 12 years ago

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

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

#4 Updated by Shogo Kawahara almost 12 years ago

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

#5 Updated by Shogo Kawahara almost 12 years ago

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

#6 Updated by Mutsumi Imamura almost 12 years ago

動作確認しました。テストOKです。

#7 Updated by Rimpei Ogawa almost 12 years ago

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

#8 Updated by Shinichi Urabe almost 12 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストは完了しているので閉じます

Also available in: Atom PDF