Uploaded image for project: 'Teeda'
  1. Teeda
  2. TEEDA-378

TNumberLengthValidator の整数部の最少桁数 (integralMin) は初期値 1,小数部の最少桁数 (fractionMin) は初期値 0 にしました.

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.0.10
    • Fix Version/s: 1.0.11
    • Component/s: Teeda Extension
    • Labels:
      None
    • Environment:
      Windows Vista / JDK1.6.0_02

      Description

      NumberLengthアノテーションとTNumberLengthValidatorのfractionMinのデフォルト値がInteger.MIN_VALUEとなっているが、ZEROとすべき。

        Activity

        Hide
        manhole manhole added a comment -

        ZEROでないと、どういうケースでお困りですか?

        Show
        manhole manhole added a comment - ZEROでないと、どういうケースでお困りですか?
        Hide
        yanashin Shinichi Yanagisawa added a comment -

        @NumberLength(integralMax = 10, fractionMax = 2)
        private BigDecimal arg1;
        というアノテーションを指定たarg1に、たとえば「11.111」という値を入力した場合
        バリデーションエラー時のエラーメッセージが
        「"arg1"の小数部は-2,147,483,648~2桁でなくてはいけません。」
        になってしまいます。
        (-2,147,483,648がInteger.MIN_VALUEに対応している)
        アノテーションのパラメタにいちいちfractionMax=0をつけるのはめんどうですよね。

        Show
        yanashin Shinichi Yanagisawa added a comment - @NumberLength(integralMax = 10, fractionMax = 2) private BigDecimal arg1; というアノテーションを指定たarg1に、たとえば「11.111」という値を入力した場合 バリデーションエラー時のエラーメッセージが 「"arg1"の小数部は-2,147,483,648~2桁でなくてはいけません。」 になってしまいます。 (-2,147,483,648がInteger.MIN_VALUEに対応している) アノテーションのパラメタにいちいちfractionMax=0をつけるのはめんどうですよね。
        Hide
        koichik koichik added a comment -

        対応しました.
        http://svn.seasar.org/browse/?view=rev&root=teeda&revision=3592

        整数部の最少桁数 (integralMin) は初期値 1,小数部の最少桁数 (fractionMin) は初期値 0 にしました.

        Show
        koichik koichik added a comment - 対応しました. http://svn.seasar.org/browse/?view=rev&root=teeda&revision=3592 整数部の最少桁数 (integralMin) は初期値 1,小数部の最少桁数 (fractionMin) は初期値 0 にしました.
        Hide
        manhole manhole added a comment -

        納得しました。
        って、もうkoichikさんが対応されているのですね。^^

        Show
        manhole manhole added a comment - 納得しました。 って、もうkoichikさんが対応されているのですね。^^
        Hide
        shot shot added a comment -

        Tigerのアノテーションが対応されてませんです^^;

        Show
        shot shot added a comment - Tigerのアノテーションが対応されてませんです^^;
        Hide
        shot shot added a comment -

        対応しました.

        Show
        shot shot added a comment - 対応しました.

          People

          • Assignee:
            shot shot
            Reporter:
            yanashin Shinichi Yanagisawa
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development