質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.03%

input type="file"を含むフォームで、Validationクラスの必須チェックが上手くできない

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 681

score 64

目的

FuelPHPでファイルをアップロードするためのフォームを作成しています。
ファイルを選択せずにsubmitするのを防ぐために、type="file"のフィールドにValidationクラスの必須チェックadd_rule('required')を付与しています。

質問

ファイルを選択せずにsubmitしようとすると必須チェックが上手く機能するのは確認できましたが、ファイルを選択してもバリデーションチェックNGの処理(下記コードの(★★★)部分)に入ってしまいます。
原因がわかる方がいましたら、ご指摘いただけると助かります。

ブラウザで見たフォームのイメージ

下記のようにファイルを選択してsubmit(アップロード)した場合も、バリデーションチェックNGの処理(下記コードの(★★★)部分)に入ってしまいます。
イメージ説明

コントローラの内容

class Controller_Csv extends Controller_Template
{
  public function action_index()
  {
    $form_data = '';

    //フォームを作成(type="file"のフィールドを含む)
    $form = Fieldset::forge('csvform', array(
      'form_attributes' => array(
        'enctype'=>'multipart/form-data',
        'class'=>'p-form'
        )
      )
    );

    //フォームに項目を追加(type="file"と"submit"のみ)
    $form->add('csvfile', '', array('type'=>'file', 'placeholder'=>'CSVファイル'))
      ->add_rule('required'); //ファイル必須チェックを付与
    $form->add('submit', '', array('type'=>'submit', 'value'=>'アップロード'));

    //POST送信された場合
    if(Input::method() === 'POST')
    {      
      $val = $form->validation();

      //バリデーションチェックOKの場合
      if($val->run())
      {
        var_dump('OK');

      //バリデーションチェックNGの場合(★★★)
      }
      else
      {
        var_dump('NG');
      }
    }

    //ビューを作成
    $this->template->head = View::forge('layout/head');
    $this->template->header = View::forge('layout/header');
    $this->template->footer = View::forge('layout/footer');
    $this->template->content = View::forge('contents/csv');
    $this->template->set_global('csvform', $form->build(''), false);
    $this->template->set_global('form_data', $form_data);
  }
}

試したこと

type="file"の代わりにtype="text"の項目にした場合は、テキストボックスに入力してsubmitするとバリデーションOKの処理に入れました。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

check解決した方法

+1

Validationクラスではなく、Uploadクラスのis_validメソッドを使うことで解決しました。

・FuelPHP Uploadクラスのドキュメント
http://fuelphp.jp/docs/1.8/classes/upload/usage.html

具体的には、add_rule('required')をフィールドに付与するのをやめて、バリデーションの判定条件をUpload::is_valid()がtrueであることにしました(下記コード★の箇所)。

    //フォームに項目を追加(type="file"と"submit"のみ)
    $form->add('csvfile', '', array('type'=>'file', 'placeholder'=>'CSVファイル'));
    //->add_rule('required'); //(★)削除
    $form->add('submit', '', array('type'=>'submit', 'value'=>'アップロード'));

    //POST送信された場合
    if(Input::method() === 'POST')
    {      
      //$val = $form->validation(); //(★)削除

      //バリデーションチェックOKの場合
      //if($val->run()) (★)削除
      if(Upload::is_valid()) //(★)追加
      {
        var_dump('OK');

      //バリデーションチェックNGの場合
      }
      else
      {
        var_dump('NG');
      }
    }

Validationクラスのような警告は出せませんが、Sessionクラスのset_flashメソッドでエラーメッセージをViewに渡して表示する対応にて代替することにします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る