OpenPNE 3

元チケットではエラーメッセージに含まれる日付フォーマットの修正も行いますが、3.2.7 に関してはリリースが迫っているため note-7 までの修正内容に留めておきます。

このチケットでの対応内容を示しておきます（親チケットは完了しておらず、通常のBP対応とは異なっています）。

修正内容¶

• apps/pc_backend/modules/profile/templates/editSuccess.php
  

   <ul>
  -<li><?php echo __('Please input in format: %format% . For example: %example%', array('%format%' => 'YYYY/MM/DD HH:MM:SS', '%example%' => '2009/01/01 23:59:21')) ?></li>
  +<li><?php echo __('Please input in format: %format%. For example: %example%', array('%format%' => 'YYYY/MM/DD', '%example%' => '2010/07/18, 0001/01/01')) ?></li>
   <li><?php echo __('Besides, you can use any particular string that can be interpreted by strtotime() function of PHP.') ?></li>
   </ul>
  

  • 文言が翻訳されていない記述ミスを修正、日付の記述例を datetime から date に変更。
• lib/form/doctrine/ProfileForm.class.php
  

     public function bind($params)
     {
  -    if ($params['form_type'] === 'input' || $params['form_type'] === 'textarea')
  +    if ('input' === $params['form_type'] || 'textarea' === $params['form_type'])
  (*1) コーディング規約に反する記述を修正
       {
  -      $validator = new sfValidatorInteger(array('required' => false));
  -      $this->setValidator('value_min', $validator);
  -      $validator = new sfValidatorInteger(array('required' => false));
  -      $this->setValidator('value_max', $validator);
  +      $validatorArgs = array(
  +        'required' => false,
  +        'trim' => true,
  +      );
  +      $validatorMin = new sfValidatorInteger($validatorArgs);
  +      $validatorMax = new sfValidatorInteger($validatorArgs);
  +      if ('integer' !== $params['value_type'])
  +      {
  +        $validatorMin->setOption('min', 0);
  +        $validatorMax->setOption('min', 1);
  +      }
  (*2) 文字数制限の場合に最小値(min=0, max=1)を指定
  +
  +      $this->setValidator('value_min', $validatorMin);
  +      $this->setValidator('value_max', $validatorMax);
       }
  -    elseif ($params['form_type'] === 'date')
  +    elseif ('date' === $params['form_type'])
  (*1) コーディング規約に反する記述を修正
       {
  -      $validator = new opValidatorDate(array('required' => false));
  -      $this->setValidator('value_min', $validator);
  -      $validator = new opValidatorDate(array('required' => false));
  -      $this->setValidator('value_max', $validator);
  +      $validatorArgs = array(
  +        'required' => false,
  +        'trim' => true,
  +        'with_time' => true,
  +        'datetime_output' => 'Y/m/d',
  +        'min' => '0001/01/01',
  +      );
  (*3) 日付の場合は、 Y/m/d 表記で保存し、0年などを入力できないように指定
  +      $validatorMin = new opValidatorDate($validatorArgs);
  +      $validatorMax = new opValidatorDate($validatorArgs);
  +
  +      $this->setValidator('value_min', $validatorMin);
  +      $this->setValidator('value_max', $validatorMax);
       }
       elseif ($params['value_min'] || $params['value_max'])
       {
         throw new sfValidatorError($validator, 'invalid');
       }
  
  +    $this->validatorSchema->setPostValidator(new sfValidatorCallback(
  +      array('callback' => array($this, 'compareMinAndMax')),
  +      array('invalid' => 'Value must be less than or equal to Minimum value.')
  +    ));
  +
  (*4) 日付の入力値が必ず yyyy/mm/dd と変換されることを前提に、 min と max の日付の大小を比較
       return parent::bind($params);
     }
  
  +  public function compareMinAndMax(sfValidatorBase $validator, $params)
  +  {
  +    $value_min = $params['value_min'];
  +    $value_max = $params['value_max'];
  +    if (!is_null($value_min) && !is_null($value_max))
  +    {
  +      if ('date' !== $params['form_type'])
  +      {
  +        $value_min = (int)$value_min;
  +        $value_max = (int)$value_max;
  +      }
  +      if ($value_min > $value_max)
  +      {
  +        throw new sfValidatorErrorSchema($validator, array('value_max' => new sfValidatorError($validator, 'invalid')));
  +      }
  +    }
  +    return $params;
  +  }
  +
  

• i18n/messages.ja.xml
  

  +      <trans-unit id="">
  +        <source>"%value%" must be at least %min%.</source>
  +        <target>%min%以上でなければなりません。</target>
  +      </trans-unit>
  +      <trans-unit id="">
  +        <source>"%value%" must be at most %max%.</source>
  +        <target>%max%以下でなければなりません。</target>
  +      </trans-unit>
  +      <trans-unit id="">
  +        <source>Value must be less than or equal to Minimum value.</source>
  +        <target>最小値以下でなければなりません。</target>
  +      </trans-unit>
  

  • 比較に関する翻訳を追加。

備考¶

• #1147 (BP from #930) で、可能なはずの入力ができなくなっている問題 #1595 が生じていたことを知らずに、それを仕様だと思ってこのチケットで修正が行われている。
• 上記修正は、本来あるべき仕様とは異なる仕様における対応内容として扱われるべきである（ 3.2 で可能なことが 3.4 以降で不可能になっている可能性がある）
• 本来あるべき仕様では、日付の min, max の大小比較は常にできない可能性がある（ #940 参照）
  • 以下の文言は意味的に間違っている（「最小値以下(less than)」は「最小値以上(greater than)」の誤り）
    

    +        <source>Value must be less than or equal to Minimum value.</source>
    +        <target>最小値以下でなければなりません。</target>