プロジェクト

全般

プロフィール

Backport(バックポート) #1469

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

Shogo Kawahara13年以上前に追加. 13年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-07-28
期日:
進捗率:

100%


説明

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


関連するチケット

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

関係しているリビジョン

リビジョン d2665b4e (差分)
Shogo Kawahara13年以上前に追加

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

リビジョン 2d62d074 (差分)
Shogo Kawahara13年以上前に追加

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

リビジョン bbc075e1 (差分)
Shogo Kawahara13年以上前に追加

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

履歴

#1 Shinichi Urabe13年以上前に更新

  • 対象バージョンOpenPNE 3.4.7 から OpenPNE 3.4.8 に変更

#2 Shinichi Urabe13年以上前に更新

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

#3 Shogo Kawahara13年以上前に更新

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

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

#4 Shogo Kawahara13年以上前に更新

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

#5 Shogo Kawahara13年以上前に更新

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

#6 Mutsumi Imamura13年以上前に更新

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

#7 Rimpei Ogawa13年以上前に更新

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

#8 Shinichi Urabe13年以上前に更新

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

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

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