laravelで画像のアップロード

Question

###前提・実現したいこと
PHPとLaravelで画像のアップロードをしたいのですが、
getClientOriginalName()でエラーが出ます。

###発生している問題・エラーメッセージ

FatalErrorException in AuthController.php line 67:
Call to a member function getClientOriginalName() on string

###該当のソースコード
まずコントローラーです。

<?php

namespace App\Http\Controllers\Auth;

use App\User;
use Validator;
use App\Http\Controllers\Controller;
use Illuminate\Foundation\Auth\ThrottlesLogins;
use Illuminate\Foundation\Auth\AuthenticatesAndRegistersUsers;
use Image;

class AuthController extends Controller
{
    /*
    |--------------------------------------------------------------------------
    | Registration & Login Controller
    |--------------------------------------------------------------------------
    |
    | This controller handles the registration of new users, as well as the
    | authentication of existing users. By default, this controller uses
    | a simple trait to add these behaviors. Why don't you explore it?
    |
    */

    use AuthenticatesAndRegistersUsers, ThrottlesLogins;

    /**
     * Where to redirect users after login / registration.
     *
     * @var string
     */
    protected $redirectTo = '/';

    /**
     * Create a new authentication controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('guest', ['except' => 'logout']);
    }

    /**
     * Get a validator for an incoming registration request.
     *
     * @param  array  $data
     * @return \Illuminate\Contracts\Validation\Validator
     */
    protected function validator(array $data)
    {
        return Validator::make($data, [
            'name' => 'required|max:255',
            'email' => 'required|email|max:255|unique:users',
            'password' => 'required|confirmed|min:6',
        ]);
    }

    /**
     * Create a new user instance after a valid registration.
     *
     * @param  array  $data
     * @return User
     */
    protected function create(array $data)
    {
        $fileName = $data['avatar']->getClientOriginalName();
        Image::make($data['avatar'])->save(public_path() . '/images/' . $fileName);

        return User::create([
            'name' => $data['name'],
            'email' => $data['email'],
            'password' => bcrypt($data['password']),
            'avatar' => $fileName,
        ]);
    }
}

フォームです。

@extends('layout')

@section('content')
<div class="bgcolor-white pt1em pb1em" id="contents">
	<div id="main_cnt_wrapper">
		<div id="yjContentsBody">
			<div class="yjContainer">
				<div class="form_box">
					<h2>m<span>新規登録</span></h2>
                    {!! Form::open(array('files' => true)) !!}
                     @if (count($errors) > 0)
    					<div id="error_explanation">
    						<ul>
    							@foreach ($errors->all() as $error)
                                    <li>{{ $error }}</li>
    							@endforeach
    						</ul>
    					</div>
					@endif
					<div class="label">
						{{ Form::label('email') }}
                        {{ Form::email('email', '', ['placeholder' => 'メールアドレスを入力']) }}
					</div>
					<div class="label">
						{{ Form::label('password') }}
                        {{ Form::password('password', ['placeholder' => 'パスワードを入力']) }}
					</div>
					<div class="label">
						{{ Form::label('password_confirmation') }}
                        {{ Form::password('password_confirmation', ['placeholder' => 'パスワードを入力（確認）']) }}
					</div>
                    <div class="label">
                        {{ Form::label('name') }}
                        {{ Form::text('name') }}
                    </div>
                    <div class="field">
                        {{ Form::file('avatar') }}
                    </div>

					<div class="submit">
						<div class="actions">
							{{ Form::submit('Create User', ['class' => 'btn btn--block']) }}
						</div>
					</div>

					{!! Form::close() !!}

					<div class="more_link_box">
						<strong>すでにアカウントを持っていますか？</strong>
						<a href="/login">Log in</a>
					</div>
				</div>
			</div>
		</div>
	</div>
@endsection

###補足情報(言語/FW/ツール等のバージョンなど)
windows
php 5.6.8
laravel 5.2
composer 1.4.2

Comments

[m.ts10806]

create()はコントローラの1メソッドですよね？なるべくコード全体をご提示ください。エラーメッセージ内容から、create()に渡す部分が間違っている可能性もあります。

[yaeyama]

そのまんま全部乗っけました。ユーザー登録時にユーザーアイコンも登録するということらしいのですが、いかがでしょうか。

Answer 1

マイナーバージョンの記載がありませんが同様の構成のLaravel5.1として、エラーの原因はAuthControllerのcreateメソッドの$dataの段階で、既にファイルデータでは無くなっているからかと思います。
AuthControllerでuseしているIlluminate\Foundation\Auth\RegistersUsersを参照してください。
postRegisterメソッドで処理を行えば多分解決するかと思います。
アップロードファイルの操作は、以下なども参考にしてください。
Laravel 5.1 ファイルシステム／クラウドストレージ ディスクインスタンス取得

Comments

[yaeyama]

laravelは5.2でした。
まだまだ勉強中で申し訳ないのですが、"postRegisterメソッドで処理を行う"とは具体的にどういったことなのでしょうか。

[aro10]

手元の5.1での記述での確認ですが、認証系コントローラの初期設定でroutes.phpの
Route::post('auth/register', 'Auth\AuthController@postRegister');
という記述で登録処理のコントローラの受け口がpostRegisterメソッドだと思うので、
トレイトのIlluminate\Foundation\Auth\RegistersUsersのpostRegisterメソッドをオーバーライドして$request変数を使うと、ファイルデータなども扱えるかと思います。
また5.3以降でアップロードファイルの保存処理が簡単になっているので、業務システムのメンテナンス等でなく勉強中であれば最新の5.5を使ってもいいかもしれません。

[yaeyama]

参考書が5.2なのでとりあえずはこのままでいきたいです。
routes.phpではRoute::auth()と書いていたのですが、
その場合でも可能でしょうか？

Answer 2

createの引数を変更できますか？
以下はどうでしょうか？

protected function create(Request $request) {
    $fileName = $request->file('avatar')->getClientOriginalName();
    ...
}

{!! Form::open(array('files' => true)) !!}

とされているので問題ないと思いますが、

<form> のなかのenctypeは"multipart/form-data"になっているか確認を。