OpenPNE 3

2系から3系にコンバートするとき、
2系のDB上(c_profile)では値がないときdefaultとして0がセットされており、
3系ではmax, minともに0に変換されます。

これはコンバートの問題ではなく、max=0,min=0であったときは、
文字数制限をかけないなどの対応をするのが適切かと思います。

コンバート後の挙動を確認しました。

少なくとも2系では、最小値、最大値に関する制限が全くなく、min > max であるような指定や、文字列の長さとして負値を設定するようなことが可能になっています。

今回の問題は、最小値、最大値を未指定にしていた場合、コンバートすると3系で min, max が 0, 0 となってしまうというものですが、これは文字列の長さに関しては対応の仕様があるものの、数値の下限上限に関しては自然な対応が思いつきません。

数値に関して、2系で明示的に (min, max) = (1, 5) と指定していれば、数値として 1～5 が許されるプロフィール項目を意味します。 (min, max) = (1, 1) であれば 1 のみが許される項目になります。ここで、 (min, max) = (0, 0) が指定されていれば 0 のみが許される項目と考えるのが自然かと思われますが、これを (min, max) = (NULL, NULL) とみなすという対応は自然でしょうか。

と、ここまで書きましたが、

2系から3系にコンバートするとき、

2系のDB上(c_profile)では値がないときdefaultとして0がセットされており、
3系ではmax, minともに0に変換されます。

と示されている通り、2系では明示的に指定していても、(min, max) = (0, 0) の場合は下限上限なしという判断になっていました。ここで2つの対応方法が考えられます。

• (1) 2系の設計が悪いだけなので、3系では特に何も対応しない（コンバート直後は min, max が 0, 0 になるが、それをプロフィール設定で変更してもらうことを仕様とする）。
• (2) コンバート直後に「0文字以内で入力してください」というエラーが出ることを問題であると考え、(min, max) = (0, 0) だけは例外的に範囲指定がされていないものとして扱う。

文字列の長さに関して (min, max) = (0, 0) になるのが問題であると思っていたので、後者の対応を考えていましたが、数値の下限上限を考えると前者の方が好ましいように思いました（後者による対応は、数値 0 のみを入力として許すプロフィール項目の作成手段を提供しないことになります）。

コンバート直後に「0文字以内で入力してください」というエラーが出ることが問題かどうかの認識によって対応方法が異なるということを示しておきます。

対応方法を更に示します。

• (3) コンバート時に (min, max) = (0, 0) であるレコードを (NULL, NULL) に変換して保存する
• (4) コンバート後に、どこかのタイミングで (0, 0) であるプロフィール項目の最小値最大値を持つレコードの値を (NULL, NULL) に変更する

(2)と(4) を併せた対応を考えていますが、修正すべきソースコードの箇所が分からず調査中です。

対応方針

lib/util/opFormItemGenerator.class.php
generateValidator() メソッド内

    if ($field['ValueType'] === 'integer' || $field['FormType'] === 'date')
    {
      if (!empty($field['ValueMin']))
      {
        $option['min'] = $field['ValueMin'];
      }
      if (!empty($field['ValueMax']))
      {
        $option['max'] = $field['ValueMax'];
      }
    }
    else
    {
      if (isset($field['ValueMin']))
      {
        $option['min_length'] = $field['ValueMin'];
      }
      if (isset($field['ValueMax']))
      {
        $option['max_length'] = $field['ValueMax'];
      }
    }

後半 else ブロック内で、isset() に加えて、max > 0 という条件を与えます。
更に、2系あるいは #940 修正前の3系にて、大小関係が不正な (min, max) が指定されたプロフィール項目がある場合にも対応できるよう、 min <= max という if 文をラップします。

if (isset($field['ValueMax'])) ブロック内の末に以下の記述を追記します。

        if (1 > (int)$field['ValueMax'] || (isset($field['ValueMin']) && (int)$field['ValueMin'] > (int)$field['ValueMax']))
        {
          unset($option['min_length']);
          unset($option['max_length']);
        }